diff options
author | matz <matz@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2008-10-21 12:56:48 +0000 |
---|---|---|
committer | matz <matz@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2008-10-21 12:56:48 +0000 |
commit | 42a6550027acfae936ba4038ca3b2a3e5ed479b2 (patch) | |
tree | a45b61004977fc8f10e1906bc4ad2c7f6a21595c | |
parent | ac466de7ff7bb62f86e2252d479d9db8c1f98500 (diff) | |
download | ruby-42a6550027acfae936ba4038ca3b2a3e5ed479b2.tar.gz |
* ext/zlib/zlib.c (rb_gzwriter_write): conversion should be done
using to_str, not to_s.
* ext/zlib/zlib.c (rb_gzwriter_write): need proper conversion
according to gz encoding.
* ext/zlib/zlib.c (rb_gzreader_ungetc): convert string encoding
before unget.
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@19872 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
-rw-r--r-- | ChangeLog | 11 | ||||
-rw-r--r-- | ext/zlib/zlib.c | 11 | ||||
-rw-r--r-- | test/zlib/test_zlib.rb | 2 |
3 files changed, 20 insertions, 4 deletions
@@ -1,3 +1,14 @@ +Tue Oct 21 21:52:00 2008 Yukihiro Matsumoto <matz@ruby-lang.org> + + * ext/zlib/zlib.c (rb_gzwriter_write): conversion should be done + using to_str, not to_s. + + * ext/zlib/zlib.c (rb_gzwriter_write): need proper conversion + according to gz encoding. + + * ext/zlib/zlib.c (rb_gzreader_ungetc): convert string encoding + before unget. + Tue Oct 21 21:33:36 2008 Nobuyoshi Nakada <nobu@ruby-lang.org> * lib/test/unit.rb (assert_nothing_raised): raise with backtrace. diff --git a/ext/zlib/zlib.c b/ext/zlib/zlib.c index 8d349c49ee..bc0716b835 100644 --- a/ext/zlib/zlib.c +++ b/ext/zlib/zlib.c @@ -2747,10 +2747,10 @@ rb_gzwriter_write(VALUE obj, VALUE str) { struct gzfile *gz = get_gzfile(obj); - if (TYPE(str) != T_STRING) { - str = rb_obj_as_string(str); + StringValue(str); + if (gz->enc2 && gz->enc2 != rb_ascii8bit_encoding()) { + str = rb_str_conv_enc(str, rb_enc_get(str), gz->enc2); } - str = rb_str_export(str); gzfile_write(gz, (Bytef*)RSTRING_PTR(str), RSTRING_LEN(str)); return INT2FIX(RSTRING_LEN(str)); } @@ -3075,6 +3075,11 @@ static VALUE rb_gzreader_ungetc(VALUE obj, VALUE s) { struct gzfile *gz = get_gzfile(obj); + + StringValue(s); + if (gz->enc2 && gz->enc2 != rb_ascii8bit_encoding()) { + s = rb_str_conv_enc(s, rb_enc_get(s), gz->enc2); + } gzfile_ungets(gz, (const Bytef*)RSTRING_PTR(s), RSTRING_LEN(s)); return Qnil; } diff --git a/test/zlib/test_zlib.rb b/test/zlib/test_zlib.rb index 5746f036ea..5a4a87a0fb 100644 --- a/test/zlib/test_zlib.rb +++ b/test/zlib/test_zlib.rb @@ -597,7 +597,7 @@ if defined? Zlib assert_equal("foo", Zlib::GzipReader.open(t.path) {|gz| gz.read }) o = Object.new - def o.to_s; "bar"; end + def o.to_str; "bar"; end Zlib::GzipWriter.open(t.path) {|gz| gz.print(o) } assert_equal("bar", Zlib::GzipReader.open(t.path) {|gz| gz.read }) end |