diff options
| -rw-r--r-- | ChangeLog | 6 | ||||
| -rw-r--r-- | re.c | 9 | ||||
| -rw-r--r-- | test/ruby/test_m17n_comb.rb | 2 |
3 files changed, 13 insertions, 4 deletions
@@ -1,3 +1,9 @@ +Sun Feb 17 20:45:10 2008 Tanaka Akira <akr@fsij.org> + + * re.c (rb_reg_prepare_re): add enable_warning parameter. + (rb_reg_adjust_startpos): disable warning by rb_reg_prepare_re. + (rb_reg_search): follow rb_reg_prepare_re parameter change. + Sun Feb 17 20:12:41 2008 Yusuke Endoh <mame@tsg.ne.jp> * test/ruby/test_regexp.rb: add tests to achieve over 90% test @@ -1041,7 +1041,7 @@ rb_reg_preprocess(const char *p, const char *end, rb_encoding *enc, rb_encoding **fixed_enc, onig_errmsg_buffer err); static void -rb_reg_prepare_re(VALUE re, VALUE str) +rb_reg_prepare_re(VALUE re, VALUE str, int enable_warning) { int need_recompile = 0; rb_encoding *enc; @@ -1068,7 +1068,8 @@ rb_reg_prepare_re(VALUE re, VALUE str) RREGEXP(re)->ptr->enc != enc) { need_recompile = 1; } - if ((RBASIC(re)->flags & REG_ENCODING_NONE) && + if (enable_warning && + (RBASIC(re)->flags & REG_ENCODING_NONE) && enc != rb_ascii8bit_encoding() && rb_enc_str_coderange(str) != ENC_CODERANGE_7BIT) { rb_warn("regexp match /.../n against to %s string", @@ -1119,7 +1120,7 @@ rb_reg_adjust_startpos(VALUE re, VALUE str, int pos, int reverse) OnigEncoding enc; UChar *p, *string; - rb_reg_prepare_re(re, str); + rb_reg_prepare_re(re, str, 0); if (reverse) { range = -pos; @@ -1158,7 +1159,7 @@ rb_reg_search(VALUE re, VALUE str, int pos, int reverse) return -1; } - rb_reg_prepare_re(re, str); + rb_reg_prepare_re(re, str, 1); if (!reverse) { range += RSTRING_LEN(str); diff --git a/test/ruby/test_m17n_comb.rb b/test/ruby/test_m17n_comb.rb index ea93de1a44..1b4f504f39 100644 --- a/test/ruby/test_m17n_comb.rb +++ b/test/ruby/test_m17n_comb.rb @@ -109,6 +109,8 @@ class TestM17NComb < Test::Unit::TestCase s("\xe0\xa0\xa1"), #"aa".force_encoding("utf-16be"), + #"aaaa".force_encoding("utf-32be"), + #"aaa".force_encoding("utf-32be"), ] def combination(*args) |