msgpack 0.7.5 → 0.7.6

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 6ba9f040ee3d1216597e2beefef6251fa120b7a0
-  data.tar.gz: c248b8969af05080d2e68c33d4bec95faa5867e8
+  metadata.gz: 8f31066b6f1e423eee182e7ea7be472797607bf3
+  data.tar.gz: aed0b52b3aaaba5b423c91c678853f71cf3145ec
 SHA512:
-  metadata.gz: 82627fad314167164461fd41ddd01b9f46cad8261090d20c49b1789d80579ea5b1035ba3be8cb285bfd517d437ebc24b8de4763423e1afd33f192b7c59c1d714
-  data.tar.gz: 4ce0ecaed760e8a94ea3514717729c476a79d0848b614823b109c0a9dfee60ae0b50d4655fbe8685b70c17fcaa8dc889a6accec3ce588f5597884032b2f7509c
+  metadata.gz: a1c7a45deda8a37da5d082cc753253b665974bd55415c28d46fb973f6c6f7c038cf526e9824fa1c94c4b4f13d802b26d82ad1ff6424dccfb2809822a9db81b1e
+  data.tar.gz: a075febabd943210c5388589daa4cd8a036c4f459f68fad0da055944cac065f37af5f92685efbf79306cca0cec64059884f9b7b38942ea681ea31d9e541bf564

data/ChangeLog

@@ -1,3 +1,12 @@
+2016-05-10 version 0.7.6:
+
+* Fix bug to raise IndexOutOfBoundsException for Bignum value with small int in JRuby
+
+2016-04-06 version 0.7.5:
+
+* Improved support for i386/armel/armhf environments
+* Fix bug for negative ext type id and some architectures (arm*)
+
 2016-01-08 version 0.7.4:
 
 * Improved compatibility of Packer between CRuby and JRuby about argument IO-ish object values.

data/ext/java/org/msgpack/jruby/Encoder.java

@@ -3,6 +3,7 @@ package org.msgpack.jruby;
 
 import java.math.BigInteger;
 import java.nio.ByteBuffer;
+import java.util.Arrays;
 
 import org.jruby.Ruby;
 import org.jruby.RubyObject;
@@ -124,14 +125,18 @@ public class Encoder {
   }
 
   private void appendBignum(RubyBignum object) {
-    BigInteger value = ((RubyBignum) object).getBigIntegerValue();
-    if (value.bitLength() > 64 || (value.bitLength() > 63 && value.signum() < 0)) {
-      throw runtime.newArgumentError(String.format("Cannot pack big integer: %s", value));
+    BigInteger value = object.getBigIntegerValue();
+    if (value.compareTo(RubyBignum.LONG_MIN) < 0 || value.compareTo(RubyBignum.LONG_MAX) > 0) {
+      if (value.bitLength() > 64 || (value.bitLength() > 63 && value.signum() < 0)) {
+        throw runtime.newArgumentError(String.format("Cannot pack big integer: %s", value));
+      }
+      ensureRemainingCapacity(9);
+      buffer.put(value.signum() < 0 ? INT64 : UINT64);
+      byte[] b = value.toByteArray();
+      buffer.put(b, b.length - 8, 8);
+    } else {
+      appendInteger(object);
     }
-    ensureRemainingCapacity(9);
-    buffer.put(value.signum() < 0 ? INT64 : UINT64);
-    byte[] b = value.toByteArray();
-    buffer.put(b, b.length - 8, 8);
   }
 
   private void appendInteger(RubyInteger object) {

data/lib/msgpack/version.rb

@@ -1,3 +1,3 @@
 module MessagePack
-  VERSION = "0.7.5"
+  VERSION = "0.7.6"
 end

data/spec/msgpack_spec.rb

@@ -159,4 +159,13 @@ describe MessagePack do
       packed.should start_with("\xDA\x00\x20")
     end
   end
+
+  context 'when a Bignum has a small value' do
+    tests['numbers'].take(10).each do |desc, unpacked, packed|
+      it("encodes #{desc} to the smallest representation") do
+        bignum = (1 << 64).coerce(unpacked)[0]
+        MessagePack.pack(bignum).should eq(packed)
+      end
+    end
+  end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: msgpack
 version: !ruby/object:Gem::Version
-  version: 0.7.5
+  version: 0.7.6
 platform: ruby
 authors:
 - Sadayuki Furuhashi
@@ -10,7 +10,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-04-06 00:00:00.000000000 Z
+date: 2016-05-10 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler