thread_local_accessor 0.1.0 → 0.1.1

data/.travis.yml

@@ -3,5 +3,7 @@ rvm:
   - 1.8.7
   - 1.9.2
   - 1.9.3
+  - jruby-18mode
+  - jruby-19mode
 script:
   - bundle exec rake test

data/CHANGELOG.md

@@ -1,5 +1,9 @@
 # Changelog
 
+## Thread Local Accessor 0.1.1 (Juni 13, 2012)
+
+* Improve implementation (thanks @brainopia for an idea)
+
 ## Thread Local Accessor 0.1.0 (April 18, 2012)
 
 * First public release

data/README.md

@@ -1,6 +1,6 @@
 # ThreadLocalAccessor
 
-Thread local accessor. An idea from [here](http://coderrr.wordpress.com/2008/04/10/lets-stop-polluting-the-threadcurrent-hash/).
+Thread local accessor. An idea and big part of code from [here](http://coderrr.wordpress.com/2008/04/10/lets-stop-polluting-the-threadcurrent-hash/).
 
 ## Installation
 
@@ -18,7 +18,24 @@ Or install it yourself as:
 
 ## Usage
 
-TODO: Write usage instructions here
+It implement thread-safe class accessor implemented via Thread.current:
+
+``` ruby
+class ThreadedLib
+  thread_local_accessor :some_setting, :default => :default
+end
+
+instance = ThreadedLib.new
+ThreadedLib.some_setting = 5
+
+ThreadedLib.some_setting # => 5
+instance.some_setting    # => 5
+
+Thread.new {
+  ThreadedLib.some_setting # => :default
+  instance.some_setting    # => :default
+end
+```
 
 ## Status
 

data/lib/thread_local_accessor.rb

@@ -3,22 +3,21 @@ require "thread_local_accessor/version"
 # An idea from http://coderrr.wordpress.com/2008/04/10/lets-stop-polluting-the-threadcurrent-hash/
 class Class
   def thread_local_accessor name, options = {}
-    m = Module.new
-    m.module_eval %{
-      def #{name}
-        k = ((Class === self ? self : self.class).object_id.to_s + '_#{name}').to_sym
+    m = Module.new do
+      define_method "#{name}=" do |value|
+        k = (Class === self ? self : self.class).object_id.to_s + "_" + name.to_s
+        Thread.current[k] = value
+      end
+
+      define_method name do
+        k = (Class === self ? self : self.class).object_id.to_s + "_" + name.to_s
         if Thread.current.key?(k)
           Thread.current[k]
         else
-          #{options[:default].inspect}
+          options[:default]
         end
       end
- 
-      def #{name}=(val)
-        k = ((Class === self ? self : self.class).object_id.to_s + '_#{name}').to_sym
-        Thread.current[k] = val
-      end
-    }
+    end
  
     class_eval do
       include m

data/lib/thread_local_accessor/version.rb

@@ -1,3 +1,3 @@
 module ThreadLocalAccessor
-  VERSION = "0.1.0"
+  VERSION = "0.1.1"
 end

data/test/thread_local_accessor_test.rb

@@ -4,6 +4,7 @@ require 'thread_local_accessor'
 # http://coderrr.wordpress.com/2008/04/10/lets-stop-polluting-the-threadcurrent-hash/
 class ThreadedLib
   thread_local_accessor :some_setting, :default => :default
+  thread_local_accessor :some_proc, :default => Proc.new{|x| x*x}
 end
  
 class TestThreadedClassAttrAccessor < Test::Unit::TestCase
@@ -26,4 +27,26 @@ class TestThreadedClassAttrAccessor < Test::Unit::TestCase
  
     assert_equal 5, ThreadedLib.some_setting
   end
+
+  def test_with_procs
+    instance = ThreadedLib.new
+ 
+    ThreadedLib.some_proc = Proc.new{|x| x - 1}
+ 
+    assert_equal 9, ThreadedLib.some_proc.call(10)
+    assert_equal 9, instance.some_proc.call(10)
+ 
+    Thread.new {
+      instance.some_proc = Proc.new{|x| x + 1}
+ 
+      assert_equal 11, ThreadedLib.some_proc.call(10)
+      assert_equal 11, instance.some_proc.call(10)
+    }.join
+ 
+    Thread.new { 
+      assert_equal 100, ThreadedLib.some_proc.call(10)
+    }.join
+ 
+    assert_equal 9, ThreadedLib.some_proc.call(10)
+  end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: thread_local_accessor
 version: !ruby/object:Gem::Version
-  version: 0.1.0
+  version: 0.1.1
   prerelease: 
 platform: ruby
 authors:
@@ -9,11 +9,11 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-04-18 00:00:00.000000000 Z
+date: 2012-06-13 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rake
-  requirement: &79427970 !ruby/object:Gem::Requirement
+  requirement: !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -21,7 +21,12 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *79427970
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
 description: Thread Local Accessor
 email:
 - vakhov@gmail.com
@@ -54,7 +59,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: -587324531
+      hash: 272763085
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements:
@@ -63,10 +68,10 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: -587324531
+      hash: 272763085
 requirements: []
 rubyforge_project: 
-rubygems_version: 1.8.17
+rubygems_version: 1.8.24
 signing_key: 
 specification_version: 3
 summary: Thread Safe Local Accessor