Merge pull request #1212 from monaka/pr-fixup-string

Fix-up string

mrbgems/mruby-string-ext/test/string.rb

@@ -14,6 +14,8 @@ assert('String#getbyte') do
 end
 
 assert('String#dump') do
+  ("\1" * 100).dump     # should not raise an exception - regress #1210
+  "\0".inspect == "\"\\000\"" and
   "foo".dump == "\"foo\""
 end
 

src/string.c

@@ -2438,8 +2438,11 @@ mrb_str_inspect(mrb_state *mrb, mrb_value str)
           continue;
       }
       else {
-        int n = sprintf(buf, "\\%03o", c & 0377);
-        mrb_str_buf_cat(mrb, result, buf, n);
+        buf[0] = '\\';
+        buf[3] = '0' + c % 8; c /= 8;
+        buf[2] = '0' + c % 8; c /= 8;
+        buf[1] = '0' + c % 8;
+        mrb_str_buf_cat(mrb, result, buf, 4);
         continue;
       }
     }

test/t/string.rb

@@ -415,10 +415,7 @@ assert('String#each_byte') do
   bytes1 == bytes2
 end
 
-assert('String#dump') do
-  ("\1" * 100).dump     # should not raise an exception - regress #1210
-end
-
 assert('String#inspect') do
   ("\1" * 100).inspect  # should not raise an exception - regress #1210
+  "\0".inspect == "\"\\000\""
 end