Igor Bukanov wrote:
Hi!
http://bugzilla.mozilla.org/show_bug.cgi?id=258844 contains a working patch against Rhino CVS from mozilla.org to enable Continuation support there.
Great work, Igor. Thanks a lot!
A minor remark though, about the syntax for creating the continuation. AFAICS, this is currently done with "Continuation()". I personally find it misleading, as the continuation represents the point where the current function was called, and not the point where the "Continuation()" statement occured.
Long ago, I proposed to Chris to allow a syntax suggested in [1] (2nd post) which is "arguments.continuation" which clearly relates the continuation to the point where the function is called, and he implemented it in [2] (search for "TokenStream.GETPROP").
This may seem as nitpicking, but if this is to go into the official Rhino and if Cocoon has to cope with minor incompatibilities like catch(continue|break|return), it would be good to have a clean syntax for continuations.
What do you think?
Sylvain
PS: I don't currently have time myself for testing the new Rhino, but hope others do!
[1] http://lambda-the-ultimate.org/classic/message8089.html
[2] http://svn.cocoondev.org/repos/rhino+cont/trunk/rhino1_5R4pre/src/org/mozilla/javascript/continuations/ContinuationInterpreter.java
-- Sylvain Wallez Anyware Technologies http://www.apache.org/~sylvain http://www.anyware-tech.com { XML, Java, Cocoon, OpenSource }*{ Training, Consulting, Projects }
