Can't figure out how to leave loop

[tgoVIPER]

So I've got the following function working great. It switches the scroll lock light on/off as my mic is muted/unmuted.

Code: Select all

mutemic()
{
SoundSet, +1, MASTER, mute,11
SoundGet, master_mute, MASTER, mute, 11
Sleep, 100
if master_mute = on
  KeyboardLED(1,"on")
else
  KeyboardLED(0,"off")
}
Return

I'd like to gussy it up a bit and make the scroll light blink, but I'm not sure how to get out of this loop.
It starts the way I want it to, it mutes the mic and starts the light blinking, but because it's now stuck in the loop it won't look for the off command.

Code: Select all

mutemic()
{
SoundSet, +1, MASTER, mute,11
SoundGet, master_mute, MASTER, mute, 11
Sleep, 100
if master_mute = on
{
    Loop
    {
      KeyboardLED(1,"on") ;
      Sleep, 500
      KeyboardLED(1,"off")
      Sleep, 500
    }
}
else
  KeyboardLED(0,"off")
}
Return

I'm hoping the solution is overly complicated so I don't feel too dumb.

Thanks in advance,

[neverlevel]

F2::reload?

probably not what you want.

could always use goto and add a sub up near the top to force the check

Code: Select all

mutemic()
{
SoundSet, +1, MASTER, mute,11
SoundGet, master_mute, MASTER, mute, 11
Sleep, 100
Recheck:
if master_mute = on
{
      KeyboardLED(1,"on") ;
      Sleep, 500
      KeyboardLED(1,"off")
      Sleep, 500
      Goto, recheck
    }
else
  KeyboardLED(0,"off")
}
Return

[tgoVIPER]

Not a bad idea, thank you. I tried it out, and am still stuck in the loop. I'll try removing the loop maybe?.

[eelrod]

If you wish for it to blink for the entire duration that the mic is muted, I would suggest using SetTimer instead of the Loop.

[SirRFI]

You can exit loops using break. However, you'd likely want to use while instead, or alternatively SetTimer.

[tgoVIPER]

If you wish for it to blink for the entire duration that the mic is muted, I would suggest using SetTimer instead of the Loop.

Good point, I left that out of the original post. Yes, I do want it to blink for as long as it's muted.
I'll look into how to use SetTimer. Thank you.

[tgoVIPER]

Code: Select all

mutemic()
{
SoundSet, +1, MASTER, mute,11
SoundGet, master_mute, MASTER, mute, 11
Sleep, 100

SetTimer, blink, 250

blink:
{
  if master_mute = on
  {
    KeyboardLED(1,"on") ;
    Sleep, 500
    KeyboardLED(1,"off")
    Sleep, 500
    return
  }
else
  KeyboardLED(0,"off")
}
return
}

Not sure if I'm using SetTimer right. It blinks the light 3 times for some reason when muted, then stops blinking. The microphone is still muted.

[tgoVIPER]

Still not sure how to properly implement the SetTimer in this situation.
Any ideas?

[SirRFI]

Your blink label is within a function, which is likely improper. Is master_mute defined somewhere?

[tgoVIPER]

Your blink label is within a function, which is likely improper. Is master_mute defined somewhere?

master_mute is defined in the SoundGet call.

I still haven't been able to figure it out.