oj 3.6.8 → 3.6.9

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: aba240d544c37d3f3c5e2a6c7fe51b29507ba354b76ede963bd868e620df4ece
-  data.tar.gz: 832dd821fa37c9ff2fd7b993c2529537a86b54ca2eded2a72f39ac8c6980bfd1
+  metadata.gz: 74bbbdb4c9cecc3d5008e135e276f134b3ccaa53d56d37e01f9039c80962abd8
+  data.tar.gz: 591d6d922cb44220f963d0432d246ab839001d8599bb9bc3ad562248df2c3b2a
 SHA512:
-  metadata.gz: e485a0613aa0306d2a5239fbccc510cfc67456760f6e01ad65c7075f2c42f0a2a6fd796dca4b35e8002a2ec7ee950801ae0ce2478ebae0e3e7715d8021eef4dc
-  data.tar.gz: fa9a0f9e185ae31e783515012f1b9e5c09bc4341e1cfe5f20f8341639d30fef5ae2d748b260fe12e3753eaf4d3e7260c5ee67d81a5706b2b950256065a5530a8
+  metadata.gz: 5cc9a42e0e64f59b4120cb39986d9b90f1b2f55585cc5c5df79e3be12b6d177407d58b3380006349b3dd6d42aece669da62d8c147c833567465377501ce11eca
+  data.tar.gz: 7b822b5752305070cd1fa9371c2bd5808ff3e127413482ef6a9a7af062c555cd79884a9a3ff0e3f5b3dc3675db84197b28a20259eb03969dbc930cb3f2d975c9

data/ext/oj/dump.c

@@ -691,9 +691,12 @@ oj_dump_str(VALUE obj, int depth, Out out, bool as_ok) {
 
 void
 oj_dump_sym(VALUE obj, int depth, Out out, bool as_ok) {
-    const char	*sym = rb_id2name(SYM2ID(obj));
+    // This causes a memory leak in 2.5.1. Maybe in other versions as well.
+    //const char	*sym = rb_id2name(SYM2ID(obj));
 
-    oj_dump_cstr(sym, strlen(sym), 0, 0, out);
+    volatile VALUE	s = rb_sym_to_s(obj);
+
+    oj_dump_cstr(rb_string_value_ptr((VALUE*)&s), RSTRING_LEN(s), 0, 0, out);
 }
 
 static void

data/lib/oj/version.rb

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

data/test/mem.rb

@@ -0,0 +1,35 @@
+#!/usr/bin/env ruby
+
+$: << '.'
+$: << '../lib'
+$: << '../ext'
+
+require 'objspace'
+require 'oj'
+require 'json'
+require 'get_process_mem'
+
+def record_allocation
+  GC.start
+  GC.start
+
+  mem = GetProcessMem.new
+  puts "Before - Process Memory: #{mem.mb} mb"
+  puts "Before - Objects count: #{ObjectSpace.each_object.count}"
+  puts "Before - Symbols count: #{Symbol.all_symbols.size}"
+
+  yield
+
+  GC.start
+  GC.start
+  GC.start
+
+  puts "After - Process Memory: #{mem.mb} mb"
+  puts "After - Objects count: #{ObjectSpace.each_object.count}"
+  puts "After - Symbols count: #{Symbol.all_symbols.size}"
+end
+
+record_allocation do
+  data = 1_000_000.times.map { |i| "string_number_#{i}".to_sym } # array of symbols
+  Oj.dump(data, mode: :strict)
+end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: oj
 version: !ruby/object:Gem::Version
-  version: 3.6.8
+  version: 3.6.9
 platform: ruby
 authors:
 - Peter Ohler
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2018-09-08 00:00:00.000000000 Z
+date: 2018-09-12 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rake-compiler
@@ -198,6 +198,7 @@ files:
 - test/json_gem/json_parser_test.rb
 - test/json_gem/json_string_matching_test.rb
 - test/json_gem/test_helper.rb
+- test/mem.rb
 - test/perf.rb
 - test/perf_compat.rb
 - test/perf_fast.rb
@@ -315,6 +316,7 @@ test_files:
 - test/test_scp.rb
 - test/perf_wab.rb
 - test/test_saj.rb
+- test/mem.rb
 - test/perf_compat.rb
 - test/helper.rb
 - test/isolated/test_mimic_alone.rb