Forum: IronRuby Code Review: ParseTree5

Announcement (2017-05-07): www.ruby-forum.com is now read-only since I unfortunately do not have the time to support and maintain the forum any more. Please see rubyonrails.org/community and ruby-lang.org/en/community for other Rails- und Ruby-related community platforms.
Cb51033949ffccd982ae32c9f890f25a?d=identicon&s=25 Tomas Matousek (Guest)
on 2008-10-21 05:07
(Received via mailing list)
Attachment: ParseTree5.diff (500 KB)
tfpt review "/shelveset:ParseTree5;REDMOND\tomat"
Comment  :
  Bunch of tweaks to AST:
  - removes some nodes that are notneeded any more,
  - moves block argument out of Arguments to CallExpression,

  Fixes several bugs:
  - order of evaluation in "(x,y) while z",
  - passing parameters to a super method when a block is specified
"super {}",
  - yield should report an error when a block argument is given "yield
&p"

  Implements ParseTree library
  - there are few nodes that are to be finished,
  - in some cases the generated parse tree deosn't exactly match MRI
(some of which I consider bugs in MRI),
  - adds ir_parse_tree.rb that implements the same API as parse_tree.rb
calling to IronRuby for tree transformation,
  - only string parsing is supported now, getting AST from method/class
at runtime is not yet implemented.

Tomas
F983f0c990cba2fe743ef62a975ec99c?d=identicon&s=25 Curt Hagenlocher (Guest)
on 2008-10-21 05:51
(Received via mailing list)
In ir_parse_tree.rb, there's code near the bottom that's 'if
RUBY_VERSION > "1.9"'.  Is that version check a typo or does it actually
reflect differences between 1.9 and 1.9.1?

Otherwise, changes look good.  And exciting!
Cb51033949ffccd982ae32c9f890f25a?d=identicon&s=25 Tomas Matousek (Guest)
on 2008-10-22 02:37
(Received via mailing list)
ir_parse_tree.rb is just a copy of parse_tree.rb from ParseTree gem with
C code replaced by a call to IronRuby. I haven't modified anything other
than that.

Tomas
This topic is locked and can not be replied to.