tfpt review "/shelveset:StringGsubWithStringPatternAndBlock;REDMOND\dremy"
Comment  :
  This change fixes the rest of the gsub coverage minus one (String#gsub with 
pattern and block raises a RuntimeError if the string is modified while 
substituting).  We had a similar issue implementing squeeze so it looks like we 
will have to track inflight changes to the underlying string (frozen already 
does this, I wonder if there is some variant we can use).

  This changes fixes the following specs:

  String#gsub with pattern and block sets $~ for access from the block
  String#gsub with pattern and block restores $~ after leaving the block
  String#gsub with pattern and block sets $~ to MatchData of last match and nil 
when there's none for access from outside




Attachment: StringGsubWithStringPatternAndBlock.diff
Description: StringGsubWithStringPatternAndBlock.diff

_______________________________________________
Ironruby-core mailing list
Ironruby-core@rubyforge.org
http://rubyforge.org/mailman/listinfo/ironruby-core

Reply via email to