cow_proxy 0.1.1 → 0.1.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: ce226b279191db83c5d94fdcd974f9ff078f9aed
-  data.tar.gz: 209d901098c7203cd6c08194fe5565a432885c71
+  metadata.gz: d055ad28e6ab0d7c88bb1543ab122b1b9dd310ba
+  data.tar.gz: 18b291113a0cd6b3b9662b4640e7087bd126297a
 SHA512:
-  metadata.gz: 02695f8b475ab7c813d169a4eb44ae91c2a6b00ec7a0ad41f690667a2bca29b3cedb228c72e1d572a917bbfafd54ddf8080badb0770e229a956e37bb00e52789
-  data.tar.gz: 64156f469d1b130068bfdaeced1988c8daad0528ce28ced26ebc132dcb74785602faa36457632ba79bc00d7bfb6c3fef2c08007543e5909d5a31115970b88a75
+  metadata.gz: 607ea8e8a4a7d0bf713c65aeb55993e778f0fab06ebe9050de0a98ed7bd3e46c5c3c70cbe6388a1d03579baf45d756677954a6672a3b90ffaf95a2bf5bf280d8
+  data.tar.gz: 7a167b8d6f5df69eea9579e5ba161a0e780c1338411fb43ee3197dc9a98ae298289d26d6d7cfb64945f75c8d7449d82659ae4066fb459b47d53f78f1982fdc49

data/lib/cow_proxy.rb

@@ -14,7 +14,7 @@
 module CowProxy
   class << self
     # @!visibility private
-    @@wrapper_classes = {}
+    @@wrapper_classes = {Integer: nil, Float: nil, Symbol: nil, TrueClass: nil, FalseClass: nil, NilClass: nil}
 
     # Create new proxy class for klass, with copy on write enabled.
     #
@@ -35,20 +35,26 @@ module CowProxy
     # Register proxy to be used when wrapping an object of klass.
     #
     # It's called automatically when inheriting from class returned by WrapClass
+    # Can be called with nil proxy_klass to disable wrapping objects of klass, for
+    # example Integer is registered with nil because there is no point in wrapping
+    # immutable classes.
     #
     # @return proxy_klass
     def register_proxy(klass, proxy_klass)
-      debug "register proxy for #{klass} with #{proxy_klass} < #{proxy_klass.superclass}" unless @@wrapper_classes[klass]
+      debug { "register proxy for #{klass} with #{proxy_klass} < #{proxy_klass.superclass}" } unless @@wrapper_classes[klass]
       @@wrapper_classes[klass] ||= proxy_klass
     end
 
     # Returns a proxy wrapping obj, using registered class for obj's class.
     # If no class is registered for obj's class, it uses default proxy, without
-    # copy on write
+    # copy on write.
+    #
+    # If class is registered with nil Proxy, return obj.
     #
     # @return wrapped obj with CowProxy class
     def wrap(obj)
-      wrapper_class(obj).new(obj)
+      klass = wrapper_class(obj)
+      klass ? klass.new(obj) : obj
     end
 
     # Returns proxy wrapper class for obj.
@@ -59,20 +65,29 @@ module CowProxy
     #   if none is registered
     def wrapper_class(obj)
       # only classes with defined wrapper and Structs has COW enabled by default
-      @@wrapper_classes[obj.class] || _WrapClass(obj.class, obj.class < Struct, true)
+      if @@wrapper_classes.has_key?(obj.class)
+        @@wrapper_classes[obj.class]
+      else
+        _WrapClass(obj.class, obj.class < Struct, true)
+      end
     end
 
     # Print debug line if debug is enabled (ENV['DEBUG'] true)
+    # Accepts a block instead of line, so interpolation is skipped
+    # when debug is disabled
+    #
     # @param [String] line debug line to print
     # @return nil
-    def debug(line)
-      Kernel.puts line if ENV['DEBUG']
+    def debug(line = nil)
+      return unless ENV['DEBUG']
+      line ||= yield if block_given?
+      Kernel.puts line
     end
 
     private
     def _WrapClass(klass, cow = true, register = false)
       proxy_superclass = get_proxy_klass_for(klass.superclass) || Base
-      debug "create new proxy class for #{klass}#{" from #{proxy_superclass}" if proxy_superclass}"
+      debug { "create new proxy class for #{klass}#{" from #{proxy_superclass}" if proxy_superclass}" }
       proxy_klass = Class.new(proxy_superclass) do |k|
         k.wrapped_class = klass
       end

data/lib/cow_proxy/base.rb

@@ -31,7 +31,7 @@ module CowProxy
           inst_var = "@#{method}" if method.to_s =~ /^\w+$/
           return _instance_variable_get(inst_var) if inst_var && _instance_variable_defined?(inst_var)
           if method.to_s =~ /^(\w+)=$/ && _instance_variable_defined?("@#{$1}")
-            CowProxy.debug "remove #{$1}"
+            CowProxy.debug { "remove #{$1}" }
             _remove_instance_variable "@#{$1}"
           end
           __wrapped_method__(inst_var, cow_enabled, method, *args, &block)
@@ -61,7 +61,7 @@ module CowProxy
     #   another CowProxy.
     # @return duplicated wrapped object
     def __copy_on_write__(parent = true)
-      CowProxy.debug "copy on write on #{__getobj__.class.name}"
+      CowProxy.debug { "copy on write on #{__getobj__.class.name}" }
       return @delegate_dc_obj if @dc_obj_duplicated
       @delegate_dc_obj = @delegate_dc_obj.dup.tap do |new_target|
         @dc_obj_duplicated = true
@@ -83,7 +83,7 @@ module CowProxy
 
     def __wrap__(value, inst_var = nil)
       if value.frozen?
-        CowProxy.debug "wrap #{value.class.name} with parent #{self.class.name}"
+        CowProxy.debug { "wrap #{value.class.name} with parent #{__getobj__.class.name}" }
         wrap_value = CowProxy.wrapper_class(value).new(value, self, inst_var)
         _instance_variable_set(inst_var, wrap_value) if inst_var
         wrap_value
@@ -91,7 +91,7 @@ module CowProxy
     end
 
     def __wrapped_value__(inst_var, method, *args, &block)
-      CowProxy.debug "run on #{__getobj__.class.name} (#{__getobj__.object_id}) #{method} #{args.inspect unless args.empty?}"
+      CowProxy.debug { "run on #{__getobj__.class.name} (#{__getobj__.object_id}) #{method} #{args.inspect unless args.empty?}" }
       value = __getobj__.__send__(method, *args, &block)
       wrap_value = __wrap__(value, inst_var) if inst_var && args.empty? && block.nil?
       wrap_value || value
@@ -101,9 +101,9 @@ module CowProxy
       __wrapped_value__(inst_var, method, *args, &block)
     rescue => e
       raise unless cow && e.message =~ /^can't modify frozen/
-      CowProxy.debug "copy on write to run #{method} #{args.inspect unless args.empty?} (#{e.message})"
+      CowProxy.debug { "copy on write to run #{method} #{args.inspect unless args.empty?} (#{e.message})" }
       __copy_on_write__
-      CowProxy.debug "new target #{__getobj__.class.name} (#{__getobj__.object_id})"
+      CowProxy.debug { "new target #{__getobj__.class.name} (#{__getobj__.object_id})" }
       __wrapped_value__(inst_var, method, *args, &block)
     end
 

data/lib/cow_proxy/string.rb

@@ -1,4 +1,7 @@
 module CowProxy
   class String < WrapClass(::String)
+    def to_str
+      __getobj__.to_str
+    end
   end
 end

data/lib/cow_proxy/version.rb

@@ -2,7 +2,7 @@ module CowProxy
   module Version
     MAJOR = 0
     MINOR = 1
-    PATCH = 1
+    PATCH = 2
 
     STRING = [MAJOR, MINOR, PATCH].compact.join('.')
   end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: cow_proxy
 version: !ruby/object:Gem::Version
-  version: 0.1.1
+  version: 0.1.2
 platform: ruby
 authors:
 - Sergio Cambra
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-03-21 00:00:00.000000000 Z
+date: 2017-03-23 00:00:00.000000000 Z
 dependencies: []
 description: Make a COW proxy for a frozen object (or deep frozen), it will delegate
   every read method to proxied object, wrap value in COW proxy if frozen. Trying to