Regexp.union behavior and Regexp.try_convert
Hi! Why Regexp.union('/../') trying to convert string with directly call rb_check_regexp_type(arg) (re.c:3139) instead of rb_reg_s_try_convert (re.c:3122)? I think second variant would be more correct, and i would be able to rewrite Regexp.try_convert in my code for changing behavior as i need.
For example: i have the code where was implemented String#to_regexp which breaks default behavior of Regexp.union and i got incorrect regexp. But i can't (for some reasons) to rewrite to_regexp method. Yes, stupid example, but it's real situation.
#2 Updated by Alexander Yunin almost 2 years ago
Bug description is incorrect, sorry. Now i want to say that implementation of String#to_regexp can breaks behavior of Regexp::union.
Regexp.union('/some_regexp/') #=> /regexp/
And it's not documented now.
#3 Updated by Tomoyuki Chikanaga almost 2 years ago
- Target version set to 2.1.0
- Status changed from Feedback to Assigned
- Assignee set to Zachary Scott
- Category set to doc
That makes sense.
I think it's spec that Regexp.union internally convert arguments with to_regexp, but it is undocumented behavior. I think it's a documentation issue.
#4 Updated by Zachary Scott almost 2 years ago
- % Done changed from 0 to 100
- Status changed from Assigned to Closed