1berry/ruby
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

stringio.c: non-ascii encoding

Browse Source 
* ext/stringio/stringio.c (strio_putc): fix for non-ascii
  encoding, like as IO#putc.  [ruby-dev:48114] [Bug #9765]

git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@45646 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
This commit is contained in:
nobu 2014-04-21 05:11:50 +00:00
parent 862618d224
commit 90e393b368
3 changed files with 29 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
ChangeLog
View File

@ -1,3 +1,8 @@
Mon Apr 21 14:11:48 2014 Nobuyoshi Nakada <nobu@ruby-lang.org>
* ext/stringio/stringio.c (strio_putc): fix for non-ascii
encoding, like as IO#putc. [ruby-dev:48114] [Bug #9765]
Sun Apr 20 12:57:15 2014 Masaya Tarui <tarui@ruby-lang.org> Sun Apr 20 12:57:15 2014 Masaya Tarui <tarui@ruby-lang.org>
* st.c (st_foreach_check): change start point of search at check * st.c (st_foreach_check): change start point of search at check

16
ext/stringio/stringio.c
View File

@ -1233,17 +1233,17 @@ static VALUE
strio_putc(VALUE self, VALUE ch) strio_putc(VALUE self, VALUE ch)
{ {
struct StringIO *ptr = writable(self); struct StringIO *ptr = writable(self);
int c = NUM2CHR(ch); VALUE str;
long olen;
check_modifiable(ptr); check_modifiable(ptr);
olen = RSTRING_LEN(ptr->string); if (RB_TYPE_P(ch, T_STRING)) {
if (ptr->flags & FMODE_APPEND) { str = rb_str_substr(ch, 0, 1);
ptr->pos = olen;
} }
strio_extend(ptr, ptr->pos, 1); else {
RSTRING_PTR(ptr->string)[ptr->pos++] = c; char c = NUM2CHR(ch);
OBJ_INFECT(ptr->string, self); str = rb_str_new(&c, 1);
}
strio_write(self, str);
return ch; return ch;
} }

16
test/stringio/test_stringio.rb
View File

@ -419,6 +419,22 @@ class TestStringIO < Test::Unit::TestCase
assert_equal("foo123", s) assert_equal("foo123", s)
end end
def test_putc_nonascii
s = ""
f = StringIO.new(s, "w")
f.putc("\u{3042}")
f.putc(0x3044)
f.close
assert_equal("\u{3042}D", s)
s = "foo"
f = StringIO.new(s, "a")
f.putc("\u{3042}")
f.putc(0x3044)
f.close
assert_equal("foo\u{3042}D", s)
end
def test_read def test_read
f = StringIO.new("\u3042\u3044") f = StringIO.new("\u3042\u3044")
assert_raise(ArgumentError) { f.read(-1) } assert_raise(ArgumentError) { f.read(-1) }