How serious is this error ?

[JohnOne]

Not sure as it says "will lead to problems" and also "could have problems"

It seems to be ok, just wondering if I could do anything to put it right.

-###2 Obfuscation Error: Found Execute() statement which will lead to problems running your obfuscated script.

>### current Func: _WinAPI_SetLayeredWindowAttributes

F:\Program Files\AutoIt3\include\WinAPI.au3(5364,1) Warning for line:$i_transcolor = Execute('0x00' & StringMid($i_transcolor, 5, 2) & StringMid($i_transcolor, 3, 2) & StringMid($i_transcolor, 1, 2))

-#############################################################################################

-#### Obfuscator Found 1 Error(s)!!!! This means your script could have problems running properly. ####

-#############################################################################################

+> Obfuscator v1.0.27.0 finished obfuscating 7494 lines, stripped 10574 comment lines. created:F:\Temp\NEWGUI\Test_Obfuscated.au3

[Melba23]

JohnOne,

Obfuscator throws a wobbly with a number of commands. This is because it believes that obfuscation might cause the contents of the commands might not be properly determined at runtime if the rest of the script is obfuscated. Go and look at the Obfuscator topic in the Examples section - there are lots of examples with explanations from Jos.

In your case, I believe that there is not a problem as the variables are all actual variable names and not string constructs. Besides if you say it works....

M23

Edit: The Obfuscator thread is here.

Edited February 24, 2010 by Melba23

[JohnOne]

Cheers M23.

I'll take a look at that now.

[jchd]

BTW I question the mere use of Execute in this particular case.

$i_transcolor = Execute('0x00' & StringMid($i_transcolor, 5, 2) & StringMid($i_transcolor, 3, 2) & StringMid($i_transcolor, 1, 2))

Also, is the null a good thing in a string?

[james3mg]

I agree, I don't think that Execute() is probably necessary at all. However, I don't think there is a null at all...look carefully and I *think* what the OP is doing is creating a 0xAARRGGBB type color, where the AA part is set to 00. See after 0x00 he's adding three sets of two characters to the end, as a string. I think the Execute() was meant to kind of "finalize" the string into an actual number, which is usually unnecessary, as AutoIt does its own conversions from string to hex as necessary. I suppose you could always Hex(Dec($i_transcolor),8) if they wanted to ensure it was stored as an actual number, lol

[jchd]

Oops I missed the tiny single quotes. Not having a second coffee to open up the second eye was the reason. Sorry!