diff options
author | naruse <naruse@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2013-04-02 08:44:16 +0000 |
---|---|---|
committer | naruse <naruse@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2013-04-02 08:44:16 +0000 |
commit | 5fd798d1ed52dde16f71ea7540c96a82b0d921da (patch) | |
tree | ee9ff7bcb84b72704ccd0b423d445e79e2014fec | |
parent | a1a6fd90bb4045b95963788fb923a9a54fdaf502 (diff) | |
download | ruby-5fd798d1ed52dde16f71ea7540c96a82b0d921da.tar.gz |
* re.c (rb_reg_to_s): suppress duplicated charclass warning.
Regexp#to_s suppress extra its whole regexp options by calling
onig_new with its source, but it doesn't call rb_reg_preprocess.
Therefore its Unicode escapes (\u{XXXX}) are given as is,
and it may cause duplicated charclass warning for example
"[\u{33}]" (3 is duplicated) or "[\u{a}\u{b}]" (u is duplicated).
[ruby-core:53649] [Bug #8151]
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@40063 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
-rw-r--r-- | ChangeLog | 10 | ||||
-rw-r--r-- | re.c | 3 | ||||
-rw-r--r-- | test/ruby/test_regexp.rb | 3 |
3 files changed, 16 insertions, 0 deletions
@@ -1,3 +1,13 @@ +Tue Apr 2 17:38:20 2013 NARUSE, Yui <naruse@ruby-lang.org> + + * re.c (rb_reg_to_s): suppress duplicated charclass warning. + Regexp#to_s suppress extra its whole regexp options by calling + onig_new with its source, but it doesn't call rb_reg_preprocess. + Therefore its Unicode escapes (\u{XXXX}) are given as is, + and it may cause duplicated charclass warning for example + "[\u{33}]" (3 is duplicated) or "[\u{a}\u{b}]" (u is duplicated). + [ruby-core:53649] [Bug #8151] + Tue Apr 2 16:00:06 2013 NARUSE, Yui <naruse@ruby-lang.org> * vm_dump.c (rb_print_backtrace): separate to ease showing C backtrace. @@ -571,12 +571,15 @@ rb_reg_to_s(VALUE re) } if (*ptr == ':' && ptr[len-1] == ')') { Regexp *rp; + VALUE verbose = ruby_verbose; + ruby_verbose = Qfalse; ++ptr; len -= 2; err = onig_new(&rp, ptr, ptr + len, ONIG_OPTION_DEFAULT, enc, OnigDefaultSyntax, NULL); onig_free(rp); + ruby_verbose = verbose; } if (err) { options = RREGEXP(re)->ptr->options; diff --git a/test/ruby/test_regexp.rb b/test/ruby/test_regexp.rb index e17cd6f962..3fa0fcf83c 100644 --- a/test/ruby/test_regexp.rb +++ b/test/ruby/test_regexp.rb @@ -915,6 +915,9 @@ class TestRegexp < Test::Unit::TestCase bug7471 = '[ruby-core:50344]' assert_warning('', bug7471) { Regexp.new('[\D]') =~ "\u3042" } + + bug8151 = '[ruby-core:53649]' + assert_warning(/\A\z/, bug8151) { Regexp.new('(?:[\u{33}])').to_s } end def test_property_warn |