oj 3.3.9 → 3.3.10

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 4f086bef3216ed10da36f2302af180253dfdd2db
-  data.tar.gz: e1dfb9a7d1ec308b19ec532adb2dc320fcf6bf8b
+  metadata.gz: 687f7d149d0cf1490c71c850e9b899211bf93dff
+  data.tar.gz: e60c06c52ab2946307618cdc78a180ed2333f19b
 SHA512:
-  metadata.gz: d3c5a71807363fcd682474d42760199bc8f4709505648c63aebbc9dadbc91677ae27f82c4712973d5fe01935998dcff46c611c92a920b39016d217a6c3cfd3a7
-  data.tar.gz: fb39fbaae9c45f79ff49f30ab624838d3c7b2061d7aa6d891dc2c05e8f7cffd2dcd48bf08618264a12f6ad0092206de7a12b9c11773dae8f7fc0c38c8c27c22f
+  metadata.gz: '0287e158af4e2637f70aeaef9bd90fbdd92270ba8b94d58eb77f3371671ead837638440677a67f00880b1b0024098ea31530f843950c434154f72dfd844d8854'
+  data.tar.gz: 279a77556142e021b88e6e2b1b75ae93a11aa373dad3098f994559004ecfa4089ebd3514cd412b08004896438d014fde5a2c9dc27f87ad3558627920146870df

data/ext/oj/custom.c

@@ -42,6 +42,8 @@ bigdecimal_dump(VALUE obj, int depth, Out out) {
     } else if (0 == strcasecmp("-Infinity", str)) {
 	str = oj_nan_str(obj, out->opts->dump_opts.nan_dump, out->opts->mode, false, &len);
 	oj_dump_raw(str, len, out);
+    } else if (No == out->opts->bigdec_as_num) {
+	oj_dump_cstr(str, len, 0, 0, out);
     } else {
 	oj_dump_raw(str, len, out);
     }

data/ext/oj/dump.c

@@ -830,7 +830,7 @@ oj_dump_cstr(const char *str, size_t cnt, bool is_sym, bool escape1, Out out) {
     if (JXEsc == out->opts->escape_mode && 0 < str - orig && 0 != (0x80 & *(str - 1))) {
 	uint8_t	c = (uint8_t)*(str - 1);
 	int	i;
-	int	scnt = str - orig;
+	int	scnt = (int)(str - orig);
 	
 	// Last utf-8 characters must be 0x10xxxxxx. The start must be
 	// 0x110xxxxx for 2 characters, 0x1110xxxx for 3, and 0x11110xxx for

data/ext/oj/dump_object.c

@@ -42,7 +42,7 @@ dump_data(VALUE obj, int depth, Out out, bool as_ok) {
 	    const char		*str = rb_string_value_ptr((VALUE*)&rstr);
 	    int			len = (int)RSTRING_LEN(rstr);
 
-	    if (Yes == out->opts->bigdec_as_num) {
+	    if (No != out->opts->bigdec_as_num) {
 		oj_dump_raw(str, len, out);
 	    } else if (0 == strcasecmp("Infinity", str)) {
 		str = oj_nan_str(obj, out->opts->dump_opts.nan_dump, out->opts->mode, true, &len);

data/ext/oj/object.c

@@ -341,7 +341,7 @@ hat_value(ParseInfo pi, Val parent, const char *key, size_t klen, volatile VALUE
             volatile VALUE	sc;
 	    volatile VALUE	e1;
 	    int			slen;
-	    
+
 	    if (0 == len) {
 		oj_set_error_at(pi, oj_parse_error_class, __FILE__, __LINE__, "Invalid struct data");
 		return 1;
@@ -380,14 +380,14 @@ hat_value(ParseInfo pi, Val parent, const char *key, size_t klen, volatile VALUE
             } else {
 		int	i;
 
-		for (i = 0; i < slen; i++) {
+		for (i = 0; i < len - 1; i++) {
 		    rb_struct_aset(parent->val, INT2FIX(i), RARRAY_PTR(value)[i + 1]);
 		}
             }
 	    return 1;
 	} else if (3 <= klen && '#' == key[1]) {
 	    volatile VALUE	*a;
-	
+
 	    if (2 != len) {
 		oj_set_error_at(pi, oj_parse_error_class, __FILE__, __LINE__, "invalid hash pair");
 		return 1;
@@ -602,7 +602,7 @@ hash_set_value(ParseInfo pi, Val kval, VALUE value) {
 	    if (3 <= klen && '^' == *key && '#' == key[1] && T_ARRAY == rb_type(value)) {
 		long		len = RARRAY_LEN(value);
 		volatile VALUE	*a = RARRAY_PTR(value);
-	
+
 		if (2 != len) {
 		    oj_set_error_at(pi, oj_parse_error_class, __FILE__, __LINE__, "invalid hash pair");
 		    return;
@@ -683,7 +683,7 @@ array_append_cstr(ParseInfo pi, const char *str, size_t len, const char *orig) {
 		rb_ary_push(stack_peek(&pi->stack)->val, oj_circ_array_get(pi->circ_array, i));
 		return;
 	    }
-	    
+
 	}
     }
     rb_ary_push(stack_peek(&pi->stack)->val, str_to_value(pi, str, len, orig));

data/ext/oj/oj.c

@@ -38,6 +38,7 @@ ID	oj_array_end_id;
 ID	oj_array_start_id;
 ID	oj_as_json_id;
 ID	oj_begin_id;
+ID	oj_bigdecimal_id;
 ID	oj_end_id;
 ID	oj_exclude_end_id;
 ID	oj_error_id;
@@ -167,7 +168,7 @@ struct _Options	oj_default_options = {
     ObjectMode,	// mode
     Yes,	// class_cache
     UnixTime,	// time_format
-    Yes,	// bigdec_as_num
+    NotSet,	// bigdec_as_num
     AutoDec,	// bigdec_load
     No,		// to_hash
     No,		// to_json
@@ -545,13 +546,16 @@ oj_parse_options(VALUE ropts, Options copts) {
 	}
     }
     for (o = ynos; 0 != o->attr; o++) {
-	if (Qnil != (v = rb_hash_lookup(ropts, o->sym))) {
-	    if (Qtrue == v) {
+	if (Qtrue == rb_funcall(ropts, oj_has_key_id, 1, o->sym)) {
+	    v = rb_hash_lookup(ropts, o->sym);
+	    if (Qnil == v) {
+		*o->attr = NotSet;
+	    } else if (Qtrue == v) {
 		*o->attr = Yes;
 	    } else if (Qfalse == v) {
 		*o->attr = No;
 	    } else {
-		rb_raise(rb_eArgError, "%s must be true or false.", rb_id2name(SYM2ID(o->sym)));
+		rb_raise(rb_eArgError, "%s must be true, false, or nil.", rb_id2name(SYM2ID(o->sym)));
 	    }
 	}
     }
@@ -1525,13 +1529,15 @@ Init_oj() {
     oj_array_end_id = rb_intern("array_end");
     oj_array_start_id = rb_intern("array_start");
     oj_as_json_id = rb_intern("as_json");
-    oj_error_id = rb_intern("error");
     oj_begin_id = rb_intern("begin");
+    oj_bigdecimal_id = rb_intern("BigDecimal");
     oj_end_id = rb_intern("end");
+    oj_error_id = rb_intern("error");
     oj_exclude_end_id = rb_intern("exclude_end?");
     oj_file_id = rb_intern("file?");
     oj_fileno_id = rb_intern("fileno");
     oj_ftype_id = rb_intern("ftype");
+    oj_has_key_id = rb_intern("has_key?");
     oj_hash_end_id = rb_intern("hash_end");
     oj_hash_key_id = rb_intern("hash_key");
     oj_hash_set_id = rb_intern("hash_set");
@@ -1548,8 +1554,8 @@ Init_oj() {
     oj_replace_id = rb_intern("replace");
     oj_stat_id = rb_intern("stat");
     oj_string_id = rb_intern("string");
-    oj_to_hash_id = rb_intern("to_hash");
     oj_to_h_id = rb_intern("to_h");
+    oj_to_hash_id = rb_intern("to_hash");
     oj_to_json_id = rb_intern("to_json");
     oj_to_s_id = rb_intern("to_s");
     oj_to_sym_id = rb_intern("to_sym");
@@ -1561,7 +1567,6 @@ Init_oj() {
     oj_utc_offset_id = rb_intern("utc_offset");
     oj_utcq_id = rb_intern("utc?");
     oj_write_id = rb_intern("write");
-    oj_has_key_id = rb_intern("has_key?");
 
     rb_require("oj/bag");
     rb_require("oj/error");

data/ext/oj/oj.h

@@ -330,6 +330,7 @@ extern ID	oj_array_end_id;
 extern ID	oj_array_start_id;
 extern ID	oj_as_json_id;
 extern ID	oj_begin_id;
+extern ID	oj_bigdecimal_id;
 extern ID	oj_end_id;
 extern ID	oj_error_id;
 extern ID	oj_exclude_end_id;

data/ext/oj/parse.c

@@ -757,7 +757,7 @@ oj_num_as_value(NumInfo ni) {
 	}
     } else { // decimal
 	if (ni->big) {
-	    rnum = rb_funcall(oj_bigdecimal_class, oj_new_id, 1, rb_str_new(ni->str, ni->len));
+	    rnum = rb_funcall(rb_cObject, oj_bigdecimal_id, 1, rb_str_new(ni->str, ni->len));
 	    if (ni->no_big) {
 		rnum = rb_funcall(rnum, rb_intern("to_f"), 0);
 	    }
@@ -770,7 +770,7 @@ oj_num_as_value(NumInfo ni) {
 	    // 15 digits. This attempts to fix those few cases where this
 	    // occurs.
 	    if ((long double)INT64_MAX > d && (int64_t)d != (ni->i * ni->div + ni->num)) {
-		rnum = rb_funcall(oj_bigdecimal_class, oj_new_id, 1, rb_str_new(ni->str, ni->len));
+		rnum = rb_funcall(rb_cObject, oj_bigdecimal_id, 1, rb_str_new(ni->str, ni->len));
 		if (ni->no_big) {
 		    rnum = rb_funcall(rnum, rb_intern("to_f"), 0);
 		}

data/ext/oj/rails.c

@@ -212,6 +212,8 @@ dump_bigdecimal(VALUE obj, int depth, Out out, bool as_ok) {
 
     if ('I' == *str || 'N' == *str || ('-' == *str && 'I' == str[1])) {
 	oj_dump_nil(Qnil, depth, out, false);
+    } else if (Yes == out->opts->bigdec_as_num) {
+	oj_dump_raw(str, RSTRING_LEN(rstr), out);
     } else {
 	oj_dump_cstr(str, RSTRING_LEN(rstr), 0, 0, out);
     }
@@ -475,17 +477,65 @@ encoder_new(int argc, VALUE *argv, VALUE self) {
     return Data_Wrap_Struct(encoder_class, encoder_mark, encoder_free, e);
 }
 
+static VALUE
+resolve_classpath(const char *name) {
+    char	class_name[1024];
+    VALUE	clas;
+    char	*end = class_name + sizeof(class_name) - 1;
+    char	*s;
+    const char	*n = name;
+    ID		cid;
+
+    clas = rb_cObject;
+    for (s = class_name; '\0' != *n; n++) {
+	if (':' == *n) {
+	    *s = '\0';
+	    n++;
+	    if (':' != *n) {
+		return Qnil;
+	    }
+	    cid = rb_intern(class_name);
+	    if (!rb_const_defined_at(clas, cid)) {
+		return Qnil;
+	    }
+	    clas = rb_const_get_at(clas, cid);
+	    s = class_name;
+	} else if (end <= s) {
+	    return Qnil;
+	} else {
+	    *s++ = *n;
+	}
+    }
+    *s = '\0';
+    cid = rb_intern(class_name);
+    if (!rb_const_defined_at(clas, cid)) {
+	return Qnil;
+    }
+    clas = rb_const_get_at(clas, cid);
+
+    return clas;
+}
+
 static void
 optimize(int argc, VALUE *argv, ROptTable rot, bool on) {
     ROpt	ro;
-    
+
     if (0 == argc) {
-	int	i;
+	int		i;
+	NamedFunc	nf;
+	VALUE		clas;
 	
 	oj_rails_hash_opt = on;
 	oj_rails_array_opt = on;
 	oj_rails_float_opt = on;
 
+	for (nf = dump_map; NULL != nf->name; nf++) {
+	    if (Qnil != (clas = resolve_classpath(nf->name))) {
+		if (NULL == oj_rails_get_opt(rot, clas)) {
+		    create_opt(rot, clas);
+		}
+	    }
+	}
 	for (i = 0; i < rot->len; i++) {
 	    rot->table[i].on = on;
 	}

data/ext/oj/saj.c

@@ -381,7 +381,7 @@ read_num(ParseInfo pi, const char *key) {
 	
 	    *pi->s = '\0';
 	    if (pi->has_add_value) {
-		call_add_value(pi->handler, rb_funcall(oj_bigdecimal_class, oj_new_id, 1, rb_str_new2(start)), key);
+		call_add_value(pi->handler, rb_funcall(rb_cObject, oj_bigdecimal_id, 1, rb_str_new2(start)), key);
 	    }
 	    *pi->s = c;
 	} else {
@@ -399,7 +399,7 @@ read_num(ParseInfo pi, const char *key) {
 	
 	    *pi->s = '\0';
 	    if (pi->has_add_value) {
-		call_add_value(pi->handler, rb_funcall(oj_bigdecimal_class, oj_new_id, 1, rb_str_new2(start)), key);
+		call_add_value(pi->handler, rb_funcall(rb_cObject, oj_bigdecimal_id, 1, rb_str_new2(start)), key);
 	    }
 	    *pi->s = c;
 	} else {

data/lib/oj/version.rb

@@ -1,5 +1,5 @@
 
 module Oj
   # Current version of the module. 
-  VERSION = '3.3.9'
+  VERSION = '3.3.10'
 end

data/pages/Modes.md

@@ -94,7 +94,7 @@ information.
     | :array_nl              | String  |         |         |         |         |         |       x |         |
     | :ascii_only            | Boolean |       x |       x |       2 |       2 |       x |       x |         |
     | :auto_define           | Boolean |         |         |         |         |       x |       x |         |
-    | :bigdecimal_as_decimal | Boolean |         |         |         |         |       x |       x |         |
+    | :bigdecimal_as_decimal | Boolean |         |         |         |       3 |       x |       x |         |
     | :bigdecimal_load       | Boolean |         |         |         |         |         |       x |         |
     | :circular              | Boolean |       x |       x |       x |       x |       x |       x |         |
     | :class_cache           | Boolean |         |         |         |         |       x |       x |         |
@@ -104,17 +104,17 @@ information.
     | :escape_mode           | Symbol  |         |         |         |         |         |       x |         |
     | :float_precision       | Fixnum  |       x |       x |         |         |         |       x |         |
     | :hash_class            | Class   |         |         |       x |       x |         |       x |         |
-    | :indent                | Integer |       x |       x |       3 |       3 |       x |       x |       x |
+    | :indent                | Integer |       x |       x |       3 |       4 |       x |       x |       x |
     | :indent_str            | String  |         |         |       x |       x |         |       x |         |
     | :match_string          | Hash    |         |         |       x |       x |         |       x |         |
-    | :max_nesting           | Fixnum  |       4 |       4 |       x |         |       4 |       4 |         |
+    | :max_nesting           | Fixnum  |       4 |       4 |       x |         |       5 |       4 |         |
     | :mode                  | Symbol  |       - |       - |       - |       - |       - |       - |         |
     | :nan                   | Symbol  |         |         |         |         |         |       x |         |
     | :nilnil                | Boolean |         |         |         |         |         |       x |         |
     | :object_class          | Class   |         |         |       x |         |         |       x |         |
     | :object_nl             | String  |         |         |       x |       x |         |       x |         |
     | :omit_nil              | Boolean |       x |       x |       x |       x |       x |       x |         |
-    | :quirks_mode           | Boolean |         |         |       5 |         |         |       x |         |
+    | :quirks_mode           | Boolean |         |         |       6 |         |         |       x |         |
     | :second_precision      | Fixnum  |         |         |         |         |       x |       x |         |
     | :space                 | String  |         |         |       x |       x |         |       x |         |
     | :space_before          | String  |         |         |       x |       x |         |       x |         |
@@ -129,19 +129,23 @@ information.
 
  2. The :ascii_only options is an undocumented json gem option.
 
- 3. The integer indent value in the default options will be honored by since
+ 3. By default the bigdecimal_as decimal is not set and the default encoding
+    for Rails is as a string. Setting the value to true will encode a
+    BigDecimal as a number which breaks compatibility.
+
+ 4. The integer indent value in the default options will be honored by since
     the json gem expects a String type the indent in calls to 'to_json()',
     'Oj.generate()', or 'Oj.generate_fast()' expect a String and not an
     integer.
 
- 4. The max_nesting option is for the json gem and rails only. It exists for
+ 5. The max_nesting option is for the json gem and rails only. It exists for
     compatibility. For other Oj dump modes the maximum nesting is set to over
     1000. If reference loops exist in the object being dumped then using the
     `:circular` option is a far better choice. It adds a slight overhead but
     detects an object that appears more than once in a dump and does not dump
     that object a second time.
 
- 5. The quirks mode option is no longer supported in the most recent json
+ 6. The quirks mode option is no longer supported in the most recent json
     gem. It is supported by Oj for backward compatibility with older json gem
     versions.
 

data/test/bug.rb

@@ -5,20 +5,21 @@
   $LOAD_PATH.unshift File.expand_path("../../#{dir}", __FILE__)
 end
 
+require 'rails'
+require 'active_support'
+require 'active_support/json'
 require 'oj'
 
-S = Struct.new(
-               :chunk_seq,
-               :youngest,
-               :oldest,
-               :size
-               )
-
-data = S.new(
-             chunk_seq: 666000,
-             youngest: nil,
-             oldest: nil,
-             size: 1
-             )
-
-puts Oj.dump(data, mode: :object)
+x = {"a": BigDecimal("1.1"), "t": Time.now}
+
+#ActiveSupport.encode_big_decimal_as_string = false
+
+Oj.optimize_rails
+
+puts "to_json #{x.to_json}"
+
+Oj.default_options = { bigdecimal_as_decimal: true}
+#puts Oj.default_options
+
+puts "to_json #{x.to_json}"
+

data/test/test_compat.rb

@@ -1,5 +1,5 @@
 #!/usr/bin/env ruby
-# encoding: UTF-8
+# encoding: utf-8
 
 $: << File.dirname(__FILE__)
 $oj_dir = File.dirname(File.expand_path(File.dirname(__FILE__)))
@@ -274,13 +274,9 @@ class CompatJuice < Minitest::Test
   def test_bigdecimal
     # BigDecimals are dumped as strings and can not be restored to the
     # original value.
-    json = Oj.dump(BigDecimal.new('3.14159265358979323846'))
-    if RUBY_VERSION >= '2.4.0'
-      # 2.4.0 changes the exponent to lowercase
-      assert_equal('"0.314159265358979323846e1"', json)
-    else
-      assert_equal('"0.314159265358979323846E1"', json)
-    end
+    json = Oj.dump(BigDecimal('3.14159265358979323846'))
+    # 2.4.0 changes the exponent to lowercase
+    assert_equal('"0.314159265358979323846e1"', json.downcase)
   end
 
   def test_infinity

data/test/test_custom.rb

@@ -1,5 +1,5 @@
 #!/usr/bin/env ruby
-# encoding: UTF-8
+# encoding: utf-8
 
 $: << File.dirname(__FILE__)
 $oj_dir = File.dirname(File.expand_path(File.dirname(__FILE__)))
@@ -203,12 +203,9 @@ class CustomJuice < Minitest::Test
   end
 
   def test_bigdecimal
-    if '2.4.0' <= RUBY_VERSION
-      assert_equal('0.314159265358979323846e1', Oj.dump(BigDecimal.new('3.14159265358979323846'), :bigdecimal_load => true))
-    else
-      assert_equal('0.314159265358979323846E1', Oj.dump(BigDecimal.new('3.14159265358979323846'), :bigdecimal_load => true))
-    end
-    dump_and_load(BigDecimal.new('3.14159265358979323846'), false, :bigdecimal_load => true)
+    assert_equal('0.314159265358979323846e1', Oj.dump(BigDecimal('3.14159265358979323846'), bigdecimal_as_decimal: true).downcase())
+    assert_equal('"0.314159265358979323846e1"', Oj.dump(BigDecimal('3.14159265358979323846'), bigdecimal_as_decimal: false).downcase())
+    dump_and_load(BigDecimal('3.14159265358979323846'), false, :bigdecimal_load => true)
   end
 
   def test_object

data/test/test_file.rb

@@ -184,20 +184,20 @@ class FileJuice < Minitest::Test
   def test_bigdecimal_strict
     mode = Oj.default_options[:mode]
     Oj.default_options = {:mode => :strict}
-    dump_and_load(BigDecimal.new('3.14159265358979323846'), false)
+    dump_and_load(BigDecimal('3.14159265358979323846'), false)
     Oj.default_options = {:mode => mode}
   end
 
   def test_bigdecimal_null
     mode = Oj.default_options[:mode]
     Oj.default_options = {:mode => :null}
-    dump_and_load(BigDecimal.new('3.14159265358979323846'), false)
+    dump_and_load(BigDecimal('3.14159265358979323846'), false)
     Oj.default_options = {:mode => mode}
   end
 
   def test_bigdecimal_object
     Oj.default_options = {:mode => :object}
-    dump_and_load(BigDecimal.new('3.14159265358979323846'), false)
+    dump_and_load(BigDecimal('3.14159265358979323846'), false)
   end
 
   # Date

data/test/test_null.rb

@@ -208,11 +208,11 @@ class NullJuice < Minitest::Test
   # BigDecimal
   def test_bigdecimal_strict
     Oj.default_options = { :bigdecimal_load => true}
-    dump_and_load(BigDecimal.new('3.14159265358979323846'), false)
+    dump_and_load(BigDecimal('3.14159265358979323846'), false)
   end
 
   def test_bigdecimal_load
-    orig = BigDecimal.new('80.51')
+    orig = BigDecimal('80.51')
     json = Oj.dump(orig, :mode => :strict, :bigdecimal_as_decimal => true)
     bg = Oj.load(json, :mode => :strict, :bigdecimal_load => true)
     assert_equal(BigDecimal, bg.class)

data/test/test_object.rb

@@ -345,19 +345,19 @@ class ObjectJuice < Minitest::Test
 
   # BigDecimal
   def test_bigdecimal_object
-    dump_and_load(BigDecimal.new('3.14159265358979323846'), false)
+    dump_and_load(BigDecimal('3.14159265358979323846'), false)
   end
 
   def test_bigdecimal_load
-    orig = BigDecimal.new('80.51')
+    orig = BigDecimal('80.51')
     json = Oj.dump(orig, :mode => :object, :bigdecimal_as_decimal => true)
     bg = Oj.load(json, :mode => :object, :bigdecimal_load => true)
     assert_equal(BigDecimal, bg.class)
     assert_equal(orig, bg)
     # Infinity is the same for Float and BigDecimal
-    json = Oj.dump(BigDecimal.new('Infinity'), :mode => :object)
+    json = Oj.dump(BigDecimal('Infinity'), :mode => :object)
     assert_equal('Infinity', json)
-    json = Oj.dump(BigDecimal.new('-Infinity'), :mode => :object)
+    json = Oj.dump(BigDecimal('-Infinity'), :mode => :object)
     assert_equal('-Infinity', json)
   end
 
@@ -805,6 +805,14 @@ class ObjectJuice < Minitest::Test
     assert_equal(h2['b'].__id__, h2.__id__)
   end
 
+
+  def test_json_object_missing_fields
+    json = %{{ "^u": [ "ObjectJuice::Stuck",1]}}
+
+    obj = Oj.load(json, mode: :object)
+    assert_nil(obj['b'])
+  end
+
   def test_circular_array
     a = [7]
     a << a
@@ -895,11 +903,11 @@ class ObjectJuice < Minitest::Test
     json = Oj.dump(jam, :omit_nil => true, :mode => :object)
     assert_equal(%|{"^o":"ObjectJuice::Jam","x":{"a":1}}|, json)
   end
-  
+
   def test_odd_date
     dump_and_load(Date.new(2012, 6, 19), false)
   end
-  
+
   def test_odd_datetime
     dump_and_load(DateTime.new(2012, 6, 19, 13, 5, Rational(4, 3)), false)
     dump_and_load(DateTime.new(2012, 6, 19, 13, 5, Rational(7123456789, 1000000000)), false)

data/test/test_strict.rb

@@ -217,11 +217,11 @@ class StrictJuice < Minitest::Test
   # BigDecimal
   def test_bigdecimal_strict
     Oj.default_options = { :bigdecimal_load => true}
-    dump_and_load(BigDecimal.new('3.14159265358979323846'), false)
+    dump_and_load(BigDecimal('3.14159265358979323846'), false)
   end
 
   def test_bigdecimal_load
-    orig = BigDecimal.new('80.51')
+    orig = BigDecimal('80.51')
     json = Oj.dump(orig, :mode => :strict, :bigdecimal_as_decimal => true)
     bg = Oj.load(json, :mode => :strict, :bigdecimal_load => true)
     assert_equal(BigDecimal, bg.class)

data/test/test_various.rb

@@ -486,13 +486,13 @@ class Juice < Minitest::Test
   def test_bigdecimal_null
     mode = Oj.default_options[:mode]
     Oj.default_options = {:mode => :null}
-    dump_and_load(BigDecimal.new('3.14159265358979323846'), false)
+    dump_and_load(BigDecimal('3.14159265358979323846'), false)
     Oj.default_options = {:mode => mode}
   end
 
   def test_infinity
     n = Oj.load('Infinity', :mode => :object)
-    assert_equal(BigDecimal.new('Infinity').to_f, n);
+    assert_equal(BigDecimal('Infinity').to_f, n);
     x = Oj.load('Infinity', :mode => :compat)
     assert_equal('Infinity', x.to_s)
   end

data/test/test_wab.rb

@@ -157,11 +157,11 @@ class WabJuice < Minitest::Test
 
   # BigDecimal
   def test_bigdecimal_wab
-    dump_and_load(BigDecimal.new('3.14159265358979323846'), false)
+    dump_and_load(BigDecimal('3.14159265358979323846'), false)
   end
 
   def test_bigdecimal_load
-    orig = BigDecimal.new('80.51')
+    orig = BigDecimal('80.51')
     json = Oj.dump(orig, mode: :wab)
     bg = Oj.load(json, :mode => :wab, :bigdecimal_load => true)
     assert_equal(BigDecimal, bg.class)

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: oj
 version: !ruby/object:Gem::Version
-  version: 3.3.9
+  version: 3.3.10
 platform: ruby
 authors:
 - Peter Ohler
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-10-27 00:00:00.000000000 Z
+date: 2017-12-30 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rake-compiler