oj 3.11.0 → 3.16.5

data/test/perf_saj.rb

@@ -1,40 +1,37 @@
 #!/usr/bin/env ruby -wW1
-# encoding: UTF-8
+# frozen_string_literal: true
 
-$: << '.'
-$: << File.join(File.dirname(__FILE__), "../lib")
-$: << File.join(File.dirname(__FILE__), "../ext")
+$LOAD_PATH << '.'
+$LOAD_PATH << File.join(__dir__, '../lib')
+$LOAD_PATH << File.join(__dir__, '../ext')
 
 require 'optparse'
-require 'yajl'
+# require 'yajl'
 require 'perf'
 require 'json'
 require 'json/ext'
 require 'oj'
 
-$verbose = false
-$indent = 0
-$iter = 10000
-$gets = 0
-$fetch = false
-$write = false
-$read = false
+@verbose = false
+@indent = 0
+@iter = 10_000
+@gets = 0
+@fetch = false
+@write = false
+@read = false
 
 opts = OptionParser.new
-opts.on("-v", "verbose")                                  { $verbose = true }
-opts.on("-c", "--count [Int]", Integer, "iterations")     { |i| $iter = i }
-opts.on("-i", "--indent [Int]", Integer, "indentation")   { |i| $indent = i }
-opts.on("-g", "--gets [Int]", Integer, "number of gets")  { |i| $gets = i }
-opts.on("-f", "fetch")                                    { $fetch = true }
-opts.on("-w", "write")                                    { $write = true }
-opts.on("-r", "read")                                     { $read = true }
-opts.on("-h", "--help", "Show this display")              { puts opts; Process.exit!(0) }
-files = opts.parse(ARGV)
+opts.on('-v', 'verbose')                                  { @verbose = true }
+opts.on('-c', '--count [Int]', Integer, 'iterations')     { |i| @iter = i }
+opts.on('-i', '--indent [Int]', Integer, 'indentation')   { |i| @indent = i }
+opts.on('-g', '--gets [Int]', Integer, 'number of gets')  { |i| @gets = i }
+opts.on('-f', 'fetch')                                    { @fetch = true }
+opts.on('-w', 'write')                                    { @write = true }
+opts.on('-r', 'read')                                     { @read = true }
+opts.on('-h', '--help', 'Show this display')              { puts opts; Process.exit!(0) }
+opts.parse(ARGV)
 
 class AllSaj < Oj::Saj
-  def initialize()
-  end
-
   def hash_start(key)
   end
 
@@ -52,58 +49,53 @@ class AllSaj < Oj::Saj
 end # AllSaj
 
 class NoSaj < Oj::Saj
-  def initialize()
-  end
 end # NoSaj
 
 saj_handler = AllSaj.new()
 no_saj = NoSaj.new()
 
-$obj = {
+@obj = {
   'a' => 'Alpha', # string
   'b' => true,    # boolean
-  'c' => 12345,   # number
-  'd' => [ true, [false, {'12345' => 12345, 'nil' => nil}, 3.967, { 'x' => 'something', 'y' => false, 'z' => true}, nil]], # mix it up array
+  'c' => 12_345,   # number
+  'd' => [ true, [false, {'12345' => 12_345, 'nil' => nil}, 3.967, { 'x' => 'something', 'y' => false, 'z' => true}, nil]], # mix it up array
   'e' => { 'one' => 1, 'two' => 2 }, # hash
   'f' => nil,     # nil
-  'g' => 12345678901234567890123456789, # big number
+  'g' => 12_345_678_901_234_567_890_123_456_789, # big number
   'h' => { 'a' => { 'b' => { 'c' => { 'd' => {'e' => { 'f' => { 'g' => nil }}}}}}}, # deep hash, not that deep
   'i' => [[[[[[[nil]]]]]]]  # deep array, again, not that deep
 }
 
-Oj.default_options = { :indent => $indent, :mode => :compat }
+Oj.default_options = { :indent => @indent, :mode => :compat }
 
-$json = Oj.dump($obj)
-$failed = {} # key is same as String used in tests later
+@json = Oj.dump(@obj)
+@failed = {} # key is same as String used in tests later
 
 def capture_error(tag, orig, load_key, dump_key, &blk)
-  begin
-    obj = blk.call(orig)
-    raise "#{tag} #{dump_key} and #{load_key} did not return the same object as the original." unless orig == obj
-  rescue Exception => e
-    $failed[tag] = "#{e.class}: #{e.message}"
-  end
+  obj = blk.call(orig)
+  raise "#{tag} #{dump_key} and #{load_key} did not return the same object as the original." unless orig == obj
+rescue Exception => e
+  @failed[tag] = "#{e.class}: #{e.message}"
 end
 
 # Verify that all packages dump and load correctly and return the same Object as the original.
-capture_error('Yajl', $obj, 'encode', 'parse') { |o| Yajl::Parser.parse(Yajl::Encoder.encode(o)) }
-capture_error('JSON::Ext', $obj, 'generate', 'parse') { |o| JSON.generator = JSON::Ext::Generator; JSON::Ext::Parser.new(JSON.generate(o)).parse }
+# capture_error('Yajl', @obj, 'encode', 'parse') { |o| Yajl::Parser.parse(Yajl::Encoder.encode(o)) }
+capture_error('JSON::Ext', @obj, 'generate', 'parse') { |o| JSON.generator = JSON::Ext::Generator; JSON::Ext::Parser.new(JSON.generate(o)).parse }
 
-if $verbose
-  puts "json:\n#{$json}\n"
+if @verbose
+  puts "json:\n#{@json}\n"
 end
 
-
 puts '-' * 80
-puts "Parse Performance"
+puts 'Parse Performance'
 perf = Perf.new()
-perf.add('Oj::Saj', 'all') { Oj.saj_parse(saj_handler, $json) }
-perf.add('Oj::Saj', 'none') { Oj.saj_parse(no_saj, $json) }
-perf.add('Yajl', 'parse') { Yajl::Parser.parse($json) } unless $failed.has_key?('Yajl')
-perf.add('JSON::Ext', 'parse') { JSON::Ext::Parser.new($json).parse } unless $failed.has_key?('JSON::Ext')
-perf.run($iter)
-
-unless $failed.empty?
-  puts "The following packages were not included for the reason listed"
-  $failed.each { |tag,msg| puts "***** #{tag}: #{msg}" }
+perf.add('Oj::Saj', 'all') { Oj.saj_parse(saj_handler, @json) }
+perf.add('Oj::Saj', 'none') { Oj.saj_parse(no_saj, @json) }
+# perf.add('Yajl', 'parse') { Yajl::Parser.parse(@json) } unless @failed.has_key?('Yajl')
+perf.add('JSON::Ext', 'parse') { JSON::Ext::Parser.new(@json).parse } unless @failed.key?('JSON::Ext')
+perf.run(@iter)
+
+unless @failed.empty?
+  puts 'The following packages were not included for the reason listed'
+  @failed.each { |tag, msg| puts "***** #{tag}: #{msg}" }
 end

data/test/perf_scp.rb

@@ -1,61 +1,58 @@
 #!/usr/bin/env ruby -wW1
-# encoding: UTF-8
+# frozen_string_literal: true
 
-$: << '.'
-$: << File.join(File.dirname(__FILE__), "../lib")
-$: << File.join(File.dirname(__FILE__), "../ext")
+$LOAD_PATH << '.'
+$LOAD_PATH << File.join(__dir__, '../lib')
+$LOAD_PATH << File.join(__dir__, '../ext')
 
 require 'optparse'
-require 'yajl'
+# require 'yajl'
 require 'perf'
 require 'json'
 require 'json/ext'
 require 'oj'
 
-$verbose = false
-$indent = 0
-$iter = 50000
-$with_bignum = false
-$size = 0
+@verbose = false
+@indent = 0
+@iter = 50_000
+@with_bignum = false
+@size = 1
 
 opts = OptionParser.new
-opts.on("-v", "verbose")                                  { $verbose = true }
-opts.on("-c", "--count [Int]", Integer, "iterations")     { |i| $iter = i }
-opts.on("-i", "--indent [Int]", Integer, "indentation")   { |i| $indent = i }
-opts.on("-s", "--size [Int]", Integer, "size (~Kbytes)")    { |i| $size = i }
-opts.on("-b", "with bignum")                                { $with_bignum = true }
-opts.on("-h", "--help", "Show this display")              { puts opts; Process.exit!(0) }
-files = opts.parse(ARGV)
-
-$obj = {
+opts.on('-v', 'verbose')                                  { @verbose = true }
+opts.on('-c', '--count [Int]', Integer, 'iterations')     { |i| @iter = i }
+opts.on('-i', '--indent [Int]', Integer, 'indentation')   { |i| @indent = i }
+opts.on('-s', '--size [Int]', Integer, 'size (~Kbytes)')  { |i| @size = i }
+opts.on('-b', 'with bignum')                              { @with_bignum = true }
+opts.on('-h', '--help', 'Show this display')              { puts opts; Process.exit!(0) }
+opts.parse(ARGV)
+
+@obj = {
   'a' => 'Alpha', # string
   'b' => true,    # boolean
-  'c' => 12345,   # number
-  'd' => [ true, [false, [-123456789, nil], 3.9676, ['Something else.', false], nil]], # mix it up array
+  'c' => 12_345,   # number
+  'd' => [ true, [false, [-123_456_789, nil], 3.9676, ['Something else.', false], nil]], # mix it up array
   'e' => { 'zero' => nil, 'one' => 1, 'two' => 2, 'three' => [3], 'four' => [0, 1, 2, 3, 4] }, # hash
   'f' => nil,     # nil
   'h' => { 'a' => { 'b' => { 'c' => { 'd' => {'e' => { 'f' => { 'g' => nil }}}}}}}, # deep hash, not that deep
   'i' => [[[[[[[nil]]]]]]]  # deep array, again, not that deep
 }
-$obj['g'] = 12345678901234567890123456789 if $with_bignum
+@obj['g'] = 12_345_678_901_234_567_890_123_456_789 if @with_bignum
 
-if 0 < $size
-  o = $obj
-  $obj = []
-  (4 * $size).times do
-    $obj << o
+if 0 < @size
+  ob = @obj
+  @obj = []
+  (4 * @size).times do
+    @obj << ob
   end
 end
 
-Oj.default_options = { :indent => $indent, :mode => :compat }
+Oj.default_options = { :indent => @indent, :mode => :strict, cache_keys: true, cache_str: 5 }
 
-$json = Oj.dump($obj)
-$failed = {} # key is same as String used in tests later
+@json = Oj.dump(@obj)
+@failed = {} # key is same as String used in tests later
 
 class AllSaj < Oj::Saj
-  def initialize()
-  end
-
   def hash_start(key)
   end
 
@@ -73,31 +70,24 @@ class AllSaj < Oj::Saj
 end # AllSaj
 
 class NoSaj < Oj::Saj
-  def initialize()
-  end
 end # NoSaj
 
 class NoHandler < Oj::ScHandler
-  def initialize()
-  end
 end # NoHandler
 
 class AllHandler < Oj::ScHandler
-  def initialize()
+  def hash_start
+    nil
   end
 
-  def hash_start()
-    return nil
+  def hash_end
   end
 
-  def hash_end()
+  def array_start
+    nil
   end
 
-  def array_start()
-    return nil
-  end
-
-  def array_end()
+  def array_end
   end
 
   def add_value(value)
@@ -105,7 +95,7 @@ class AllHandler < Oj::ScHandler
 
   def hash_set(h, key, value)
   end
-  
+
   def array_append(a, value)
   end
 
@@ -118,34 +108,33 @@ sc_handler = AllHandler.new()
 no_handler = NoHandler.new()
 
 def capture_error(tag, orig, load_key, dump_key, &blk)
-  begin
-    obj = blk.call(orig)
-    raise "#{tag} #{dump_key} and #{load_key} did not return the same object as the original." unless orig == obj
-  rescue Exception => e
-    $failed[tag] = "#{e.class}: #{e.message}"
-  end
+  obj = blk.call(orig)
+  raise "#{tag} #{dump_key} and #{load_key} did not return the same object as the original." unless orig == obj
+rescue Exception => e
+  @failed[tag] = "#{e.class}: #{e.message}"
 end
 
 # Verify that all packages dump and load correctly and return the same Object as the original.
-capture_error('Yajl', $obj, 'encode', 'parse') { |o| Yajl::Parser.parse(Yajl::Encoder.encode(o)) }
-capture_error('JSON::Ext', $obj, 'generate', 'parse') { |o| JSON.generator = JSON::Ext::Generator; JSON::Ext::Parser.new(JSON.generate(o)).parse }
+# capture_error('Yajl', @obj, 'encode', 'parse') { |o| Yajl::Parser.parse(Yajl::Encoder.encode(o)) }
+capture_error('JSON::Ext', @obj, 'generate', 'parse') { |o| JSON.generator = JSON::Ext::Generator; JSON::Ext::Parser.new(JSON.generate(o)).parse }
 
-if $verbose
-  puts "json:\n#{$json}\n"
+if @verbose
+  puts "json:\n#{@json}\n"
 end
 
 puts '-' * 80
-puts "Parse Performance"
+puts 'Parse Performance'
 perf = Perf.new()
-perf.add('Oj::Saj', 'all') { Oj.saj_parse(saj_handler, $json) }
-perf.add('Oj::Saj', 'none') { Oj.saj_parse(no_saj, $json) }
-perf.add('Oj::Scp', 'all') { Oj.sc_parse(sc_handler, $json) }
-perf.add('Oj::Scp', 'none') { Oj.sc_parse(no_handler, $json) }
-perf.add('Yajl', 'parse') { Yajl::Parser.parse($json) } unless $failed.has_key?('Yajl')
-perf.add('JSON::Ext', 'parse') { JSON::Ext::Parser.new($json).parse } unless $failed.has_key?('JSON::Ext')
-perf.run($iter)
-
-unless $failed.empty?
-  puts "The following packages were not included for the reason listed"
-  $failed.each { |tag,msg| puts "***** #{tag}: #{msg}" }
+perf.add('Oj::Saj.all', 'all') { Oj.saj_parse(saj_handler, @json) }
+perf.add('Oj::Saj.none', 'none') { Oj.saj_parse(no_saj, @json) }
+perf.add('Oj::Scp.all', 'all') { Oj.sc_parse(sc_handler, @json) }
+perf.add('Oj::Scp.none', 'none') { Oj.sc_parse(no_handler, @json) }
+perf.add('Oj::load', 'none') { Oj.wab_load(@json) }
+# perf.add('Yajl', 'parse') { Yajl::Parser.parse(@json) } unless @failed.has_key?('Yajl')
+perf.add('JSON::Ext', 'parse') { JSON::Ext::Parser.new(@json).parse } unless @failed.key?('JSON::Ext')
+perf.run(@iter)
+
+unless @failed.empty?
+  puts 'The following packages were not included for the reason listed'
+  @failed.each { |tag, msg| puts "***** #{tag}: #{msg}" }
 end

data/test/perf_simple.rb

@@ -1,8 +1,8 @@
 #!/usr/bin/env ruby -wW1
-# encoding: UTF-8
+# frozen_string_literal: true
 
-$: << File.join(File.dirname(__FILE__), "../lib")
-$: << File.join(File.dirname(__FILE__), "../ext")
+$LOAD_PATH << File.join(__dir__, '../lib')
+$LOAD_PATH << File.join(__dir__, '../ext')
 
 require 'optparse'
 require 'yajl'
@@ -14,14 +14,15 @@ require 'oj'
 require 'ox'
 
 class Jazz
-  def initialize()
+  def initialize
     @boolean = true
     @number = 58
-    @string = "A string"
+    @string = 'A string'
     @array = [true, false, nil]
     @hash = { 'one' => 1, 'two' => 2 }
   end
-  def to_json()
+
+  def to_json(*_args)
     %{
     { "boolean":#{@boolean},
       "number":#{@number},
@@ -31,7 +32,8 @@ class Jazz
     }
 }
   end
-  def to_hash()
+
+  def to_hash
     { 'boolean' => @boolean,
       'number' => @number,
       'string' => @string,
@@ -39,37 +41,38 @@ class Jazz
       'hash' => @hash,
     }
   end
+
   def to_msgpack(out='')
     to_hash().to_msgpack(out)
   end
 end
 
 $indent = 2
-$iter = 10000
+$iter = 10_000
 $with_object = true
 $with_bignum = true
 $with_nums = true
 
 opts = OptionParser.new
-opts.on("-c", "--count [Int]", Integer, "iterations")       { |i| $iter = i }
-opts.on("-i", "--indent [Int]", Integer, "indentation")     { |i| $indent = i }
-opts.on("-o", "without objects")                            { $with_object = false }
-opts.on("-b", "without bignum")                             { $with_bignum = false }
-opts.on("-n", "without numbers")                            { $with_nums = false }
-opts.on("-h", "--help", "Show this display")                { puts opts; Process.exit!(0) }
-files = opts.parse(ARGV)
+opts.on('-c', '--count [Int]', Integer, 'iterations')       { |i| $iter = i }
+opts.on('-i', '--indent [Int]', Integer, 'indentation')     { |i| $indent = i }
+opts.on('-o', 'without objects')                            { $with_object = false }
+opts.on('-b', 'without bignum')                             { $with_bignum = false }
+opts.on('-n', 'without numbers')                            { $with_nums = false }
+opts.on('-h', '--help', 'Show this display')                { puts opts; Process.exit!(0) }
+opts.parse(ARGV)
 
 if $with_nums
   obj = {
     'a' => 'Alpha',
     'b' => true,
-    'c' => 12345,
-    'd' => [ true, [false, [12345, nil], 3.967, ['something', false], nil]],
+    'c' => 12_345,
+    'd' => [ true, [false, [12_345, nil], 3.967, ['something', false], nil]],
     'e' => { 'one' => 1, 'two' => 2 },
     'f' => nil,
   }
   obj['g'] = Jazz.new() if $with_object
-  obj['h'] = 12345678901234567890123456789 if $with_bignum
+  obj['h'] = 12_345_678_901_234_567_890_123_456_789 if $with_bignum
 else
   obj = {
     'a' => 'Alpha',
@@ -102,7 +105,7 @@ end
 dt = Time.now - start
 base_dt = dt
 parse_results[:oj] = dt
-puts "%d Oj.load()s in %0.3f seconds or %0.1f loads/msec" % [$iter, dt, $iter/dt/1000.0]
+puts '%d Oj.load()s in %0.3f seconds or %0.1f loads/msec' % [$iter, dt, $iter/dt/1000.0]
 
 start = Time.now
 $iter.times do
@@ -114,7 +117,7 @@ if base_dt < dt
   base_name = 'Yajl'
 end
 parse_results[:yajl] = dt
-puts "%d Yajl::Parser.parse()s in %0.3f seconds or %0.1f parses/msec" % [$iter, dt, $iter/dt/1000.0]
+puts '%d Yajl::Parser.parse()s in %0.3f seconds or %0.1f parses/msec' % [$iter, dt, $iter/dt/1000.0]
 
 begin
   JSON.parser = JSON::Ext::Parser
@@ -128,7 +131,7 @@ begin
     base_name = 'JSON::Ext'
   end
   parse_results[:ext] = dt
-  puts "%d JSON::Ext::Parser parse()s in %0.3f seconds or %0.1f parses/msec" % [$iter, dt, $iter/dt/1000.0]
+  puts '%d JSON::Ext::Parser parse()s in %0.3f seconds or %0.1f parses/msec' % [$iter, dt, $iter/dt/1000.0]
 rescue Exception => e
   puts "JSON::Ext failed: #{e.class}: #{e.message}"
 end
@@ -145,7 +148,7 @@ begin
     base_name = 'JSON::Pure'
   end
   parse_results[:pure] = dt
-  puts "%d JSON::Pure::Parser parse()s in %0.3f seconds or %0.1f parses/msec" % [$iter, dt, $iter/dt/1000.0]
+  puts '%d JSON::Pure::Parser parse()s in %0.3f seconds or %0.1f parses/msec' % [$iter, dt, $iter/dt/1000.0]
 rescue Exception => e
   puts "JSON::Pure failed: #{e.class}: #{e.message}"
 end
@@ -162,7 +165,7 @@ begin
     base_name = 'MessagePack'
   end
   parse_results[:msgpack] = dt
-  puts "%d MessagePack.unpack()s in %0.3f seconds or %0.1f packs/msec" % [$iter, dt, $iter/dt/1000.0]
+  puts '%d MessagePack.unpack()s in %0.3f seconds or %0.1f packs/msec' % [$iter, dt, $iter/dt/1000.0]
 rescue Exception => e
   puts "MessagePack failed: #{e.class}: #{e.message}"
 end
@@ -173,23 +176,22 @@ $iter.times do
 end
 dt = Time.now - start
 parse_results[:ox] = dt
-puts "%d Ox.load()s in %0.3f seconds or %0.1f loads/msec" % [$iter, dt, $iter/dt/1000.0]
+puts '%d Ox.load()s in %0.3f seconds or %0.1f loads/msec' % [$iter, dt, $iter/dt/1000.0]
 
-puts "Parser results:"
+puts 'Parser results:'
 puts "gem       seconds  parses/msec  X faster than #{base_name} (higher is better)"
-parse_results.each do |name,dt|
-  if 0.0 == dt
+parse_results.each do |name, dt2|
+  if dt2 <= 0.0
     puts "#{name} failed to generate JSON"
     next
   end
-  puts "%-7s  %6.3f    %5.1f        %4.1f" % [name, dt, $iter/dt/1000.0, base_dt/dt]
+  puts '%-7s  %6.3f    %5.1f        %4.1f' % [name, dt2, $iter/dt/1000.0, base_dt/dt2]
 end
 
 puts
 
 # Back to object mode for best performance when dumping.
 Oj.default_options = { :indent => $indent, :mode => :object }
-dump_results = { :oj => 0.0, :yajl => 0.0, :msgpack => 0.0, :pure => 0.0, :ext => 0.0, :ox => 0.0 }
 
 start = Time.now
 $iter.times do
@@ -199,7 +201,7 @@ dt = Time.now - start
 base_dt = dt
 base_name = 'Oj'
 parse_results[:oj] = dt
-puts "%d Oj.dump()s in %0.3f seconds or %0.1f dumps/msec" % [$iter, dt, $iter/dt/1000.0]
+puts '%d Oj.dump()s in %0.3f seconds or %0.1f dumps/msec' % [$iter, dt, $iter/dt/1000.0]
 
 start = Time.now
 $iter.times do
@@ -211,7 +213,7 @@ if base_dt < dt
   base_name = 'Yajl'
 end
 parse_results[:yajl] = dt
-puts "%d Yajl::Encoder.encode()s in %0.3f seconds or %0.1f encodes/msec" % [$iter, dt, $iter/dt/1000.0]
+puts '%d Yajl::Encoder.encode()s in %0.3f seconds or %0.1f encodes/msec' % [$iter, dt, $iter/dt/1000.0]
 
 begin
   JSON.parser = JSON::Ext::Parser
@@ -225,7 +227,7 @@ begin
     base_name = 'JSON::Ext'
   end
   parse_results[:pure] = dt
-  puts "%d JSON::Ext generate()s in %0.3f seconds or %0.1f generates/msec" % [$iter, dt, $iter/dt/1000.0]
+  puts '%d JSON::Ext generate()s in %0.3f seconds or %0.1f generates/msec' % [$iter, dt, $iter/dt/1000.0]
 rescue Exception => e
   parse_results[:ext] = 0.0
   puts "JSON::Ext failed: #{e.class}: #{e.message}"
@@ -243,7 +245,7 @@ begin
     base_name = 'JSON::Pure'
   end
   parse_results[:pure] = dt
-  puts "%d JSON::Pure generate()s in %0.3f seconds or %0.1f generates/msec" % [$iter, dt, $iter/dt/1000.0]
+  puts '%d JSON::Pure generate()s in %0.3f seconds or %0.1f generates/msec' % [$iter, dt, $iter/dt/1000.0]
 rescue Exception => e
   parse_results[:pure] = 0.0
   puts "JSON::Pure failed: #{e.class}: #{e.message}"
@@ -260,7 +262,7 @@ begin
     base_name = 'MessagePack'
   end
   parse_results[:msgpack] = dt
-  puts "%d Msgpack()s in %0.3f seconds or %0.1f unpacks/msec" % [$iter, dt, $iter/dt/1000.0]
+  puts '%d Msgpack()s in %0.3f seconds or %0.1f unpacks/msec' % [$iter, dt, $iter/dt/1000.0]
 rescue Exception => e
   parse_results[:msgpack] = 0.0
   puts "MessagePack failed: #{e.class}: #{e.message}"
@@ -272,16 +274,16 @@ $iter.times do
 end
 dt = Time.now - start
 parse_results[:ox] = dt
-puts "%d Ox.dump()s in %0.3f seconds or %0.1f dumps/msec" % [$iter, dt, $iter/dt/1000.0]
+puts '%d Ox.dump()s in %0.3f seconds or %0.1f dumps/msec' % [$iter, dt, $iter/dt/1000.0]
 
-puts "Parser results:"
+puts 'Parser results:'
 puts "gem       seconds  dumps/msec  X faster than #{base_name} (higher is better)"
-parse_results.each do |name,dt|
-  if 0.0 == dt
+parse_results.each do |name, dt2|
+  if dt2 <= 0.0
     puts "#{name} failed to generate JSON"
     next
   end
-  puts "%-7s  %6.3f    %5.1f       %4.1f" % [name, dt, $iter/dt/1000.0, base_dt/dt]
+  puts '%-7s  %6.3f    %5.1f       %4.1f' % [name, dt2, $iter/dt/1000.0, base_dt/dt2]
 end
 
 puts