-->
Page 27 of 27

Re: 3.0 branch available for testing. Report bugs here.

PostPosted: Mon Jul 04, 2016 7:18 am
by raintime
Forlotto,

I see in your code you write:

if pinaction == "po" then gosub, [pomypin]

But the documentation example for GOSUB AND RETURN simply says:

gosub {label}

Why do you put a comma after gosub? Is it optional? required? shouldn't be there?
8-)

Re: 3.0 branch available for testing. Report bugs here.

PostPosted: Mon Jul 04, 2016 4:23 pm
by forlotto
Nope it is not a requirement you are correct thanks!
Was placed there because it is a needed thing with button code version 3 for instance so a bad habbit that was formed from messing with button code prior to working with this.
Good catch though!

I actually got this working finally well have an idea on how to correct it I have not really sat down and spent time with it however it works to some degree.

Re: 3.0 branch available for testing. Report bugs here.

PostPosted: Mon Jul 04, 2016 7:24 pm
by raintime
OK, but then we have another "bug" since the comma should have been detected by the syntax checker as an error! :D

Re: 3.0 branch available for testing. Report bugs here.

PostPosted: Mon Jul 04, 2016 9:08 pm
by forlotto
not so sure on that one I would have to test to see if it works both ways hold on....

That would be correct it will say No MSG Branch defined! When trying to request this is correct there is no syntax flag some things would be difficult to check for I would assume this may be one IDK?

But yep you are right after testing it technically is a bug so to speak in the syntax checker.

The largest bug we face is HTML currently ...