multi_json 1.7.3 → 1.7.4

data/CHANGELOG.md

@@ -1,3 +1,7 @@
+1.7.4
+-----
+* [Cache options for better performance](https://github.com/intridea/multi_json/commit/8a26ee93140c4bed36194ed9fb887a1b6919257b)
+
 1.7.3
 -----
 * [Require json/ext to ensure extension version gets loaded for json_gem](https://github.com/intridea/multi_json/commit/942686f7e8597418c6f90ee69e1d45242fac07b1)

data/lib/multi_json.rb

@@ -27,6 +27,14 @@ module MultiJson
     self.load_options = self.dump_options = value
   end
 
+  # cache busting
+  %w(load_options= dump_options=).each do |method|
+    define_method method do |*args|
+      use current_adapter
+      super *args
+    end
+  end
+
   ALIASES = {
     'jrjackson' => :jr_jackson
   }
@@ -51,7 +59,7 @@ module MultiJson
     return :gson if defined?(::Gson)
     return :jr_jackson if defined?(::JrJackson)
 
-    REQUIREMENT_MAP.each do |(library, adapter)|
+    REQUIREMENT_MAP.each do |library, adapter|
       begin
         require library
         return adapter

data/lib/multi_json/adapter.rb

@@ -15,27 +15,33 @@ module MultiJson
         end
       end
 
+      def activate!
+        @load_options_cache = {}
+        @dump_options_cache = {}
+        instance.activate if instance.respond_to?(:activate)
+      end
+
       def load(string, options={})
-        instance.load(string, collect_load_options(string, options))
+        instance.load(string, collect_load_options(string, options).clone)
       end
 
       def dump(object, options={})
-        instance.dump(object, collect_dump_options(object, options))
+        instance.dump(object, collect_dump_options(object, options).clone)
       end
 
     protected
 
       def collect_load_options(string, options)
-        collect_options :load_options, options, [ string, options ]
+        @load_options_cache[options] ||= collect_options(:load_options, options).merge(options)
       end
 
       def collect_dump_options(object, options)
-        collect_options :dump_options, options, [ object, options ]
+        @dump_options_cache[options] ||= collect_options(:dump_options, options).merge(options)
       end
 
-      def collect_options(method, overrides, args)
+      def collect_options(method, *args)
         global, local = *[MultiJson, self].map{ |r| r.send(method, *args) }
-        local.merge(global).merge(overrides)
+        local.merge(global)
       end
 
     end

data/lib/multi_json/adapters/json_common.rb

@@ -7,7 +7,7 @@ module MultiJson
 
       GEM_VERSION = '1.7.7'
 
-      def self.activate!
+      def activate
         if JSON::VERSION < GEM_VERSION
           Kernel.warn "You are using an old or stdlib version of #{gem_name} gem\n" +
             "Please upgrade to the recent version by adding this to your Gemfile:\n\n" +

data/lib/multi_json/adapters/json_gem.rb

@@ -7,7 +7,7 @@ module MultiJson
     class JsonGem < JsonCommon
       ParseError = ::JSON::ParserError
 
-      def self.gem_name
+      def gem_name
         'json'
       end
     end

data/lib/multi_json/adapters/json_pure.rb

@@ -7,7 +7,7 @@ module MultiJson
     class JsonPure < JsonCommon
       ParseError = ::JSON::ParserError
 
-      def self.gem_name
+      def gem_name
         'json_pure'
       end
     end

data/lib/multi_json/version.rb

@@ -1,3 +1,3 @@
 module MultiJson
-  VERSION = '1.7.3' unless defined?(MultiJson::VERSION)
+  VERSION = '1.7.4' unless defined?(MultiJson::VERSION)
 end

data/spec/multi_json_spec.rb

@@ -50,6 +50,18 @@ describe 'MultiJson' do
       end
     end
 
+    context 'caching' do
+      before{ MultiJson.use :json_gem }
+      let(:json_string){ '{"abc":"def"}' }
+
+      it 'busts options caches on change' do
+        MultiJson.load_options = { :symbolize_keys => true }
+        expect(MultiJson.load(json_string)).to eq(:abc => 'def')
+        MultiJson.load_options = nil
+        expect(MultiJson.load(json_string)).to eq('abc' => 'def')
+      end
+    end
+
     context 'with stdlib version' do
       around do |example|
         version = JSON::VERSION
@@ -63,7 +75,7 @@ describe 'MultiJson' do
         MultiJson.use :json_gem
       end
 
-      it 'should warb about json/pure' do
+      it 'should warn about json/pure' do
         Kernel.should_receive(:warn).with(/'json_pure', '~> 1.7.7'/)
         MultiJson.use :json_pure
       end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: multi_json
 version: !ruby/object:Gem::Version
-  version: 1.7.3
+  version: 1.7.4
   prerelease: 
 platform: ruby
 authors:
@@ -39,7 +39,7 @@ cert_chain:
   U0xxV3ZRUnNCbHlwSGZoczZKSnVMbHlaUEdoVTNSL3YKU2YzbFZLcEJDV2dS
   cEdUdnk0NVhWcEIrNTl5MzNQSm1FdVExUFRFT1l2UXlhbzlVS01BQWFBTi83
   cVdRdGpsMApobHc9Ci0tLS0tRU5EIENFUlRJRklDQVRFLS0tLS0K
-date: 2013-05-05 00:00:00.000000000 Z
+date: 2013-05-26 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler