Return's parameter error, even inside a function

[pilger]

Hey guys,

I have the following code (and that's all the code, really):

Code: Select all

WaitPixelColor (x, y, color, limit:=20)
{
	loop %limit%
	{

		Sleep, 500
		PixelGetColor, Color, %x%, %y%

		if (Color = %color%) {
			Return true
		}
	}
	Return false
}

I get a "Error: Return's parameter should be blank except inside a function." when I try to reload the script. Both returns seem to be inside the function. What could be going wrong here!?

Thanks in advance!

[TheDewd]

Your function isn't being recognized because you've added a space between the name and the parameters:

WaitPixelColor (x, y, color, limit:=20)
Should be
WaitPixelColor(x, y, color, limit:=20)

[pilger]

Thank you very much. That did the trick. Dumb mistake!

[lexikos]

Maybe you figured it out after fixing the function definition, but...

if (Color = %color%)

I'm pretty sure this part isn't working as intended. My guess is that you made two mistakes:

1. Color and color are intended to be two different variables, so you can compare the result of PixelGetColor to the original value of the parameter. However, variable names in AutoHotkey are not case-sensitive, so PixelGetColor is overwriting the parameter value.
2. %color% is a double-deref. See FAQ: When exactly are variable names enclosed in percent signs?