Error dialog keeps appearing, any fix? Topic is solved

[tatagi]

hi, this is my script

Code: Select all

MaxPresses := 2



~Space::

Presses += 1

SetTimer, ResetPresses, -600

If (Pressed <= MaxPresses)

	GoSub, Presses%Presses%

return



ResetPresses:

Presses := 0

return



Presses1:

TrayTip, %A_ThisLabel%, You pressed Space 1 time

return



Presses2:

TrayTip, %A_ThisLabel%, You pressed Space 2 times

return

it works as normal but here's the problem
if I press "space" three or more times rapidly within 600ms, the error dialog pops up saying this

1.PNG (22.44 KiB) Viewed 429 times

I can minimize this problem by adjusting the settimer value down to 200ms or something
but is there a way I can stop it from showing completely?

[mikeyww]

Two ways:

1. Add a line that skips the Gosub if the value is invalid.

2. Reset the value after it reaches the maximum.

[tatagi]

thank you for the reply
but how to add a line to skip gosub?

I tried to put else after GoSub, Presses%Presses% which did nothing

[mikeyww]

I think I would change Pressed to Presses.

Code: Select all

~Space::
Presses++
SetTimer, ResetPresses, -600
If Presses between 1 and 2
 TrayTip, Presses, Times pressed: %Presses%
Return

ResetPresses:
Presses := 0
Return

[tatagi]

I think I would change Pressed to Presses.

Code: Select all

~Space::
Presses++
SetTimer, ResetPresses, -600
If Presses between 1 and 2
 TrayTip, Presses, Times pressed: %Presses%
Return

ResetPresses:
Presses := 0
Return

oh, that seems to work but I am sorry to say my real script is not traytip, it is two different actions that I want it to take.
I am not sure this makes sense but

Code: Select all

~Space::
Presses++
SetTimer, ResetPresses, -600
{
If Presses 1
do something  (for example ctrl s)
If Presses 2
do another thing (for example ctrl o)
return

ResetPresses:
Presses := 0
Return

is this possible? thx

[mikeyww]

Code: Select all

~Space::
Presses++
SetTimer, ResetPresses, -600
Switch Presses {
 Case 1: Send ^s
 Case 2: Send ^o
}
Return

ResetPresses:
Presses := 0
Return

[tatagi]

Code: Select all

~Space::
Presses++
SetTimer, ResetPresses, -600
Switch Presses {
 Case 1: Send ^s
 Case 2: Send ^o
}
Return

ResetPresses:
Presses := 0
Return

omg there's problem. in the example case, pressing space twice gives me both "save as" and "open dialog"
how to avoid this? I want two work separately.

and another problem is if I add #warn for integrity check, error saying:

[mikeyww]

Code: Select all

~Space::
KeyWait, Space
KeyWait, Space, DT.3
If ErrorLevel {      ; Pressed once
 Send ^s
 SoundBeep, 1700
} Else Send ^o       ; Pressed twice
Return