Backport #7558

Small docs bug for Regexp::union

Added by Alex Gaynor almost 3 years ago. Updated almost 3 years ago.

Status:Third Party's Issue
Assignee:Usaku NAKAMURA


The docs for Regexp::union ( state:

Regexp.union(/dogs/, /cats/) #=> /(?-mix:dogs)|(?i-mx:cats)/

This is, however, incorrect, /cats/ doesn't have the /i flag, either it should be added or the expected result should be changed.


#1 Updated by Steve Klabnik almost 3 years ago

I went to go fix this, and I found that it's fixed in trunk:

I'm not sure if this justifies a backport into 1.9.x, but I'd be willing to write the patch if someone can remind me of the procedure.

#2 Updated by Zachary Scott almost 3 years ago

For what it's worth, here is the original patch:

Not sure what revision that is though. And seeing as how it was originally committed 5 years ago, I'm surprised this bug still exists.

Steve, only documentation I found on backporting:

#3 Updated by Usaku NAKAMURA almost 3 years ago

  • Category deleted (doc)
  • Status changed from Open to Assigned
  • Assignee set to Usaku NAKAMURA
  • Tracker changed from Bug to Backport
  • Project changed from Ruby trunk to Backport193

#4 Updated by Usaku NAKAMURA almost 3 years ago

  • Status changed from Assigned to Third Party's Issue

seems it's only on

Also available in: Atom PDF