Helper ast node sequences must not wrongly grab ownership of the enclosing

Helper ast node sequences must not wrongly grab ownership of the enclosing
scope (fix issue 736).
Add test.
Committed: https://code.google.com/p/dart/source/detail?r=2324

[hausner, 2011-12-09 21:19:21]

LGTM w/ tiny comment.

http://codereview.chromium.org/8893008/diff/1/runtime/vm/parser.cc
File runtime/vm/parser.cc (right):

http://codereview.chromium.org/8893008/diff/1/runtime/vm/parser.cc#newcode3609
runtime/vm/parser.cc:3609: // are not initialized. If several variables are
declared and initialized,
This comment is outdated, since variables are now always initialized (possibly
implicitly). So NULL is never returned. Can you update it while you're working
in the neighborhood?

[regis, 2011-12-09 21:31:22]

Thanks

http://codereview.chromium.org/8893008/diff/1/runtime/vm/parser.cc
File runtime/vm/parser.cc (right):

http://codereview.chromium.org/8893008/diff/1/runtime/vm/parser.cc#newcode3609
runtime/vm/parser.cc:3609: // are not initialized. If several variables are
declared and initialized,
On 2011/12/09 21:19:21, hausner wrote:
> This comment is outdated, since variables are now always initialized (possibly
> implicitly). So NULL is never returned. Can you update it while you're working
> in the neighborhood?

Done.