diff options
author | nobu <nobu@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2008-12-08 09:31:44 +0000 |
---|---|---|
committer | nobu <nobu@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2008-12-08 09:31:44 +0000 |
commit | 92e0f99d375648b9f12dd25a556eace54a3aa6af (patch) | |
tree | b376296d9d2364db4007faa8ea21f15d42a11e22 | |
parent | 774f924db9bd7abc7284256704f4cc63e568e63e (diff) | |
download | ruby-92e0f99d375648b9f12dd25a556eace54a3aa6af.tar.gz |
* pack.c (pack_pack): fixed length for odd length string.
[ruby-dev:37283]
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@20580 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
-rw-r--r-- | ChangeLog | 5 | ||||
-rw-r--r-- | pack.c | 4 | ||||
-rw-r--r-- | test/ruby/test_pack.rb | 12 |
3 files changed, 19 insertions, 2 deletions
@@ -1,3 +1,8 @@ +Mon Dec 8 18:31:41 2008 Nobuyoshi Nakada <nobu@ruby-lang.org> + + * pack.c (pack_pack): fixed length for odd length string. + [ruby-dev:37283] + Mon Dec 8 11:28:14 2008 Nobuyoshi Nakada <nobu@ruby-lang.org> * ruby.c (proc_options): added version, copyright, usage, yydebug, @@ -620,7 +620,7 @@ pack_pack(VALUE ary, VALUE fmt) long i, j = 0; if (len > plen) { - j = (len - plen + 1)/2; + j = (len + 1) / 2 - (plen + 1) / 2; len = plen; } for (i=0; i++ < len; ptr++) { @@ -651,7 +651,7 @@ pack_pack(VALUE ary, VALUE fmt) long i, j = 0; if (len > plen) { - j = (len - plen + 1)/2; + j = (len + 1) / 2 - (plen + 1) / 2; len = plen; } for (i=0; i++ < len; ptr++) { diff --git a/test/ruby/test_pack.rb b/test/ruby/test_pack.rb index 33694b9f38..6101a30219 100644 --- a/test/ruby/test_pack.rb +++ b/test/ruby/test_pack.rb @@ -179,11 +179,23 @@ class TestPack < Test::Unit::TestCase assert_equal("", ["10ef"].pack("h0")) assert_equal("\x01\x0e", ["10ef"].pack("h3")) assert_equal("\x01\xfe\x0", ["10ef"].pack("h5")) + assert_equal("\xff\x0f", ["fff"].pack("h3")) + assert_equal("\xff\x0f", ["fff"].pack("h4")) + assert_equal("\xff\x0f\0", ["fff"].pack("h5")) + assert_equal("\xff\x0f\0", ["fff"].pack("h6")) + assert_equal("\xff\x0f\0\0", ["fff"].pack("h7")) + assert_equal("\xff\x0f\0\0", ["fff"].pack("h8")) assert_equal("\x10\xef", ["10ef"].pack("H*")) assert_equal("", ["10ef"].pack("H0")) assert_equal("\x10\xe0", ["10ef"].pack("H3")) assert_equal("\x10\xef\x0", ["10ef"].pack("H5")) + assert_equal("\xff\xf0", ["fff"].pack("H3")) + assert_equal("\xff\xf0", ["fff"].pack("H4")) + assert_equal("\xff\xf0\0", ["fff"].pack("H5")) + assert_equal("\xff\xf0\0", ["fff"].pack("H6")) + assert_equal("\xff\xf0\0\0", ["fff"].pack("H7")) + assert_equal("\xff\xf0\0\0", ["fff"].pack("H8")) assert_equal(["10ef"], "\x01\xfe".unpack("h*")) assert_equal([""], "\x01\xfe".unpack("h0")) |