diff options
| author | Jeremy Evans <[email protected]> | 2020-11-23 14:40:59 -0800 |
|---|---|---|
| committer | Benoit Daloze <[email protected]> | 2020-12-02 18:42:02 +0100 |
| commit | 9e73177d5362c1986814f411961b712967dc5f97 () | |
| tree | f0ddbddc3e24423cd39128c3bdebf07d7e6608d1 /test | |
| parent | 7546be2ca0de45bcc2a72d9b4ae0cd44a1322031 (diff) | |
Do not reduce quantifiers if it affects which text will be matched
Quantifier reduction when using +?)* and +?)+ should not be done as it affects which text will be matched. This removes the need for the RQ_PQ_Q ReduceType, so remove the enum entry and related switch case. Test that these are the only two patterns affected by testing all quantifier reduction tuples for both the captured and uncaptured cases and making sure the matched text is the same for both. Fixes [Bug #17341]
Notes: Merged: https://.com/ruby/ruby/pull/3808
| -rw-r--r-- | test/ruby/test_regexp.rb | 14 |
1 files changed, 14 insertions, 0 deletions
@@ -1196,6 +1196,20 @@ class TestRegexp < Test::Unit::TestCase } end def test_once pr1 = proc{|i| /#{i}/o} assert_equal(/0/, pr1.call(0)) |