AutoHotkey Community

[Whitespliff]

Hello,

I've been writing a script for me, everything went ok but now I'm stuck.
I get this error:


Adding return doesn't help b/c I didn't use Gosub as far as I know.

Full script can be found here

Any help is welcome...
Tnx
_________________
In a world without walls and fences, who needs Windows and Gates?

[Laszlo]

It is the "}", which does not have a "{" before.

[Whitespliff]

If I delete that I get this error:


It has something to do with GUI 4, I used a MsgBox before & everything was ok then.
_________________
In a world without walls and fences, who needs Windows and Gates?

[Laszlo]

Maybe the pairing is wrong. Check that each "{" has a pairing "}" later. If you cannot find the problem post your code here.

[koro]

Try moving all the labels outside the nested braces using gosub-return (This will make the code much clearer too) . When something goes to "4ButtonYes", for instance, the program finds three consecutive closing braces which were never open (since you called that subroutine from other part of the code). (i don't see the script gosub-ing or goto-ing to that part ever, so i don't see what's the purpose of the labels, but i assume the script is incomplete yet).

Btw, nothing after the "Return" line in the "IfMsgBox, No" block will be executed, of course, so one more reason why i don't understand all that big block.

Edit: I double checked and your braces do match, there's no missing brace, but the way they are used is rather weird, so try what i said above. (and perhaps try using indentation to have a cleaner view of how the blocks are structured)

[Laszlo]

Koro is right. You should not put subroutines inside blocks delimited by "{" and "}", but your GUI labels are put there. Try moving them at the end of the script.

[Whitespliff]

[PhiLho]

Line 101, there is an extra }
Using a good editor like SciTE helps to find out these problems, because it shows pairing (or non-pairing) parentheses and braces, and allows to jump to the opening/closing counter-part.
BTW, improving your indenting should help you finding these problems.
_________________
vPhiLho := RegExReplace("Philippe Lhoste", "^(\w{3})\w*\s+\b(\w{3})\w*$", "$1$2")

[Whitespliff]

An extra } ?

If I put an extra } there or if I delete it I get an error that 1 is missing.
For now I'm using the MsgBox again, I can't get GUI 4 to work.

If there is anybody that can help me, plz do

Tnx for suggesting SciTE, it is a lot easier this way
_________________
In a world without walls and fences, who needs Windows and Gates?

[koro]

[PhiLho]

[Whitespliff]

I think I've got it.
Tnx a lot for the help & tnx for suggesting SciLexer.dll with AutoHotkey support!

PS: code with indentations is a lot easier to read yes
_________________
In a world without walls and fences, who needs Windows and Gates?