Exit command

[ari]

Code: Select all

Let>WSI_TIMEOUT=3
WaitScreenImage>c:\imagedir\image_1.bmp,0,EXACT

If>WSI_TIMEOUT=true
    Exit>
Endif

I am assuming there is something wrong with this code, as the macro continues onward (instead of quitting) executing commands after the Endif even if the image.bmp does not appear in 3 seconds

(when the image is present, the macro executes normally and correctly)

P.S. I bought MS14 this week, thanks for the great product

[hagchr]

I think you need to use WSI_TIMEDOUT instead of WSI_TIMEOUT for the second check (small spelling difference - WSI_TIMEOUT sets the timeout in seconds and WSI_TIMEDOUT shows if the function "timed out" or not - TRUE/FALSE).

[CyberCitizen]

You Need To Specify An Exit Code.

Exit>0

[JRL]

You Need To Specify An Exit Code.

Actually, I often leave off the return code and Exit> works just fine. Maybe I'm being foolish and there will come a time when those lines with no parameter specified will fail.

Is the return code for the Exit> function optional by design or does it just happen work that way now?

I think hagchr hit the nail on the head. You have to use the correct variable. I also wanted to emphasize that all Macro Scheduler variable results are case sensitive. "true" is not equal to "TRUE". Since the results for "WSI_TIMEDOUT" are upper case, either the test needs to be for an upper case "TRUE", If>WSI_TIMEOUT=TRUE, Or the result needs to be preconditioned using the UpperCase> function or the LowerCase> function.

Code: Select all

Let>WSI_TIMEOUT=3
WaitScreenImage>c:\imagedir\image_1.bmp,0,EXACT

LowerCase>WSI_TIMEDOUT,WSI_TIMEDOUT
If>WSI_TIMEDOUT=true
    Exit>
Endif

[Marcus Tettmar]

You Need To Specify An Exit Code.

Exit>0

That's not the issue. Ari is checking if WSI_TIMEOUT is true. It won't be. It's already been set to 3. Ari needs to check WSI_TIMEDOUT as hagchr says.

[ari]

I was wrong on 2 fronts, timeD out, and the lower case TRUE/FALSE! Thanks, very educational and this makes my scripts a lot more useful (vs. watching closely and shift+esc when the script is done)

[CyberCitizen]

My apologies, I only had a quick glance while on the train via my phone. That was what jumped out at me. Should I looked at it further.