Thanks Alessandro. I've applied your good looking changes to SVN.

Cheers,

Sebastian


Alessandro Sala schrieb:
> Hello Sebastian,
>> I've changed the behavior a bit to support such expressions. Hopefully 
>> this does not break other code. However I've not used your patch. 
>> Thank you anyway for your work.
>>   
> thanks, unfortunately it seems the problem is only half-solved; now a
> construct similar to:
> 
> if (a == b)
>   a += 3;
> else
>   b += 2;
> 
> gets compiled in:
> 
> if(a==b)a+=3;elseb+=2;
> 
> so it is still missing a space between "else" and "b+=2". The 
> compilation succeeds only if the
> statement following "else" is another "if" or a call statement.
> 
> So I'm attaching another patch (simpler than my previous one, now that 
> you added the hasChild() method
> to the Node class), hoping that it will be useful to better show the 
> solution: similarly to what you have done
> for the "statement" preceding the "else", it simply checks whether the 
> "elseStatement" has a "block" child: if not,
> it adds a space after "else".
> 
> Cheers,
> Alessandro
> 
> 
> ------------------------------------------------------------------------
> 
> -------------------------------------------------------------------------
> Take Surveys. Earn Cash. Influence the Future of IT
> Join SourceForge.net's Techsay panel and you'll get the chance to share your
> opinions on IT & business topics through brief surveys -- and earn cash
> http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
> 
> 
> ------------------------------------------------------------------------
> 
> _______________________________________________
> qooxdoo-devel mailing list
> [email protected]
> https://lists.sourceforge.net/lists/listinfo/qooxdoo-devel


-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys -- and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
_______________________________________________
qooxdoo-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/qooxdoo-devel

Reply via email to