[ ruby-Bugs-1688 ] [PATCH] Bugfixes to ripper HEAD - broken depend, and regexp_literal lacks options

Bugs item #1688, was opened at 2005-03-25 22:01
You can respond by visiting: 
http://rubyforge.org/tracker/?func=detail&atid=1698&aid=1688&group_id=426

Category: Core
Group: None
Status: Open
Resolution: None
Priority: 5
Submitted By: Mitchell Charity (mcharity)
Assigned to: Nobody (None)
Summary: [PATCH] Bugfixes to ripper HEAD - broken depend, and regexp_literal lacks options

Initial Comment:
Two bugfixes to ripper, CVS HEAD.
Patch attached.

(1) Bug fix, ext/ripper/depend

Problem: ext/ripper/lib/ripper/core.rb is not being rebuilt.

Solution: Renamed the target from "rb" to the file name.
Created a new target "rb" for backwards compatibility.

(2) Bug fix, parse.y, and thus ext/ripper/ripper.y

Problem: In parse.y, ripper's regepx_literal dispatch is not given the
regexp options.

Solution: Added the options ($3) to the dispatch.

Before
  $ ruby -rripper -rpp -e 'pp Ripper.sexp("/pat/mi")'
  [:program,
   [:stmts_add,
    [:stmts_new],
    [:regexp_literal,
     [:xstring_add, [:xstring_new], [:@tstring_content, "pat", [1, 1]]]]]]
After
  $ ruby -rripper -rpp -e 'pp Ripper.sexp("/pat/mi")'
  [:program,
   [:stmts_add,
    [:stmts_new],
    [:regexp_literal,
     [:xstring_add, [:xstring_new], [:@tstring_content, "pat", [1, 1]]],
     [:@regexp_end, "/mi", [1, 4]]]]]


----------------------------------------------------------------------

You can respond by visiting: 
http://rubyforge.org/tracker/?func=detail&atid=1698&aid=1688&group_id=426