[
http://jira.codehaus.org/browse/JANINO-116?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=136097#action_136097 ]
Arno Unkrig commented on JANINO-116:
------------------------------------
Hm... I think it is not correct to have a new syntax element {{StatementList}}. (Java has no "statement list" element.)
Why don't you create the {{StatementList}} tree _outside_ the AST, and when it is complete then "unroll" the {{StatementList}}s' statements into one {{Block}}? That' would just be natural.
> Need a Java.StatementList for more flexibility in generating ASTs
> -----------------------------------------------------------------
>
> Key: JANINO-116
> URL:
http://jira.codehaus.org/browse/JANINO-116> Project: Janino
> Issue Type: New Feature
> Reporter: Matt Fowles
> Assignee: Arno Unkrig
> Attachments: janino-full.patch
>
>
> StatementLists provide a way to put a place holder statement somewhere, and then fill it in later with only things that are needed. The big difference between this and a block is that it very explicitly does not introduce a new scope, so that it can be used to inject variables that will be visible.
> The attached patch includes an implementation of this request and tests for it. It also includes the fixes in:
>
http://jira.codehaus.org/browse/JANINO-111>
http://jira.codehaus.org/browse/JANINO-112>
http://jira.codehaus.org/browse/JANINO-113>
http://jira.codehaus.org/browse/JANINO-115> This flexibility is needed for the inclusion of Janino in my project. I am nearly finished integrating Janino as our backend bytecode generation system and it is currently passing our nightly test suite of over 9,000 tests with this patch.
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
http://jira.codehaus.org/secure/Administrators.jspa-
For more information on JIRA, see:
http://www.atlassian.com/software/jira
---------------------------------------------------------------------
To unsubscribe from this list, please visit:
http://xircles.codehaus.org/manage_email
