Hi Yong, Thanks a lot for the patches, they were spot on for fixing the problem and had new tests!
Git did have to change them when merging because of patches I had committed earlier, so they may cause conflicts for your local repository (even though they look like the same patches). Vladimir 2010/5/14 szergling <[email protected]>: > I have also encountered this problem. Enclosed is a patch, > with an additional fix to a similar issues inside arefs/arrays. > > I hope the patches are acceptable (don't know git well > enough). I would prefer for them to not depend on my > other patches from a few days ago. Will they still merge > ok? > > Yong. > > > On 5/4/10, Daniel Gackle <[email protected]> wrote: >> Another syntax error in the new PS. This form: >> >> (let ((blah (let ((x (foo))) >> (if (null x) y z))))) >> >> ... compiles to... >> >> var blah = x = foo(), x == null ? y : z; >> >> ... which is incorrect. Previously, PS produced this, which is correct: >> >> var blah = (x = foo(), x == null ? y : z); >> >> Daniel >> > > _______________________________________________ > parenscript-devel mailing list > [email protected] > http://common-lisp.net/cgi-bin/mailman/listinfo/parenscript-devel > > _______________________________________________ parenscript-devel mailing list [email protected] http://common-lisp.net/cgi-bin/mailman/listinfo/parenscript-devel
