Hi,

There were some commits about changing String#succ behavior in 1_8 (in
incompatible way), introducing at least 8 rubyspec failures. Luckily,
those changes were eventually reverted, so I just wanted to say thanks
for doing the right thing.

On a related note, the MRI tests for String were actually also updated
to show that new (now reverted) behavior, but the tests are not
reverted back.

The following simple patch removes offending tests:

diff --git a/test/ruby/test_string.rb b/test/ruby/test_string.rb
index 6bac7f1..4f2070a 100644
--- a/test/ruby/test_string.rb
+++ b/test/ruby/test_string.rb
@@ -27,7 +27,6 @@ class TestString < Test::Unit::TestCase
     assert_equal("1000", "999".succ)
     assert_equal("2.000", "1.999".succ)

-    assert_equal("No.10", "No.9".succ)
     assert_equal("2000aaa",  "1999zzz".succ)
     assert_equal("AAAAA000", "ZZZZ999".succ)
     assert_equal("*+", "**".succ)
@@ -75,10 +74,6 @@ class TestString < Test::Unit::TestCase
     assert_equal("*+", a.succ!)
     assert_equal("*+", a)

-    a = "No.9"
-    assert_equal("No.10", a.succ!)
-    assert_equal("No.10", a)
-
     assert_equal("aaaaaaaaaaaa", "zzzzzzzzzzz".succ!)
     assert_equal("aaaaaaaaaaaaaaaaaaaaaaaa", "zzzzzzzzzzzzzzzzzzzzzzz".succ!)
   end

Thanks,
  --Vladimir