diff options
-rw-r--r-- | ChangeLog | 5 | ||||
-rw-r--r-- | bootstraptest/test_knownbug.rb | 8 | ||||
-rw-r--r-- | string.c | 4 | ||||
-rw-r--r-- | test/ruby/test_m17n.rb | 6 |
4 files changed, 13 insertions, 10 deletions
@@ -1,3 +1,8 @@ +Sun Dec 9 11:29:23 2007 Nobuyoshi Nakada <nobu@ruby-lang.org> + + * string.c (tr_trans): get rid of segfaults when has mulitbytes but + source sets have no mulitbytes. + Sun Dec 9 04:01:28 2007 Tanaka Akira <akr@fsij.org> * encoding.c (rb_enc_mbclen): return 1 if underlying implementation diff --git a/bootstraptest/test_knownbug.rb b/bootstraptest/test_knownbug.rb index a5c712bcdf..a9cb8eb1a5 100644 --- a/bootstraptest/test_knownbug.rb +++ b/bootstraptest/test_knownbug.rb @@ -162,11 +162,3 @@ assert_equal 'true', %q{ s = "\xa3\xb0\xa3\xb1\xa3\xb1\xa3\xb3\xa3\xb4".force_encoding("euc-jp") s.squeeze == "\xa3\xb0\xa3\xb1\xa3\xb3\xa3\xb4".force_encoding("euc-jp") } - -assert_normal_exit %q{ - "\x81\x41".force_encoding("shift_jis").tr("A", "B") -} - -assert_normal_exit %q{ - "\x81\x41".force_encoding("shift_jis").tr_s("A", "B") -} @@ -3466,7 +3466,7 @@ tr_trans(VALUE str, VALUE src, VALUE repl, int sflag) v = trans[c] >= 0 ? trans[c] : Qnil; } else { - v = rb_hash_aref(hash, INT2NUM(c)); + v = hash ? rb_hash_aref(hash, INT2NUM(c)) : Qnil; } if (!NIL_P(v)) { if (!cflag) { @@ -3531,7 +3531,7 @@ tr_trans(VALUE str, VALUE src, VALUE repl, int sflag) v = trans[c] >= 0 ? trans[c] : Qnil; } else { - v = rb_hash_aref(hash, INT2NUM(c)); + v = hash ? rb_hash_aref(hash, INT2NUM(c)) : Qnil; } if (!NIL_P(v)) { if (!cflag) { diff --git a/test/ruby/test_m17n.rb b/test/ruby/test_m17n.rb index f923173e5a..542c3d2b00 100644 --- a/test/ruby/test_m17n.rb +++ b/test/ruby/test_m17n.rb @@ -483,4 +483,10 @@ class TestM17N < Test::Unit::TestCase assert_raise(SyntaxError) { eval(s(%{/\\u{6666}#{}\\xc2\\xa0/})) } assert_nothing_raised { eval(u(%{/\\u{6666}#{}\\xc2\\xa0/})) } end + + def test_tr + s = "\x81\x41".force_encoding("shift_jis") + assert_equal(s.tr("A", "B"), s) + assert_equal(s.tr_s("A", "B"), s) + end end |