lru_redux 0.0.6 → 0.8.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: b501fa162a36c12d13ff683ccbaa0e85ddda003d
-  data.tar.gz: 2888a33f2c15f078bec636d3782b8de0cb61f519
+  metadata.gz: 3ea015033987bd1895311ed7621db04aaff3ab6b
+  data.tar.gz: 0a9ee0f191f866e5070e381c958b0d12bc1a00bb
 SHA512:
-  metadata.gz: 5290cdf6e1b0a5014154891464790005625451db37c7959217da68b6f1f6a3ad41d46370bdcb734ed098cda4cb3945a12ea805284898c213c581a9f9b913d83c
-  data.tar.gz: 7b1cf05f55aa7f314ef9b4b346dca6d7ca20bf5458e893c4a32d4ae1fcb225a3bc6f2992cd1aee54e944f5e03e77cfeacecde69e46b711b272ebb1fe2648c0cd
+  metadata.gz: 615ebf6f43f5b111b63dd136272d84019d5100e889951c458a00b50d901bf07e927614659dbd9e9095967c95b14e4da29d9b840a8a57b7004e4b8b3155f29a4d
+  data.tar.gz: 9be61a373c5c8ff1a212999996b690b14c77a996e469b896362b2164dba4b52f061a69a62c30e94380cf4c6e2887c2fa7fcbe8dc8107cc7f06f71ba012bd0f43

data/CHANGELOG

@@ -0,0 +1,5 @@
+7-Sep-2013
+
+* fix #each implementation
+* fix deadlocks with ThreadSafeCache
+* version 0.8.1 ( its been used in prd for quite a while now )

data/lib/lru_redux/cache19.rb

@@ -59,7 +59,8 @@ class LruRedux::Cache
   end
 
   def each
-    @data.reverse.each do |pair|
+    array = @data.to_a
+    array.reverse!.each do |pair|
       yield pair
     end
   end
@@ -68,7 +69,8 @@ class LruRedux::Cache
   alias_method :each_unsafe, :each
 
   def to_a
-    @data.to_a.reverse
+    array = @data.to_a
+    array.reverse!
   end
 
   def delete(k)

data/lib/lru_redux/thread_safe_cache.rb

@@ -1,14 +1,16 @@
 require 'thread'
+require 'monitor'
+
 class LruRedux::ThreadSafeCache < LruRedux::Cache
+  include MonitorMixin
   def initialize(size)
-    @lock = Mutex.new
     super(size)
   end
 
   def self.synchronize(*methods)
     methods.each do |method|
       define_method method do |*args, &blk|
-      @lock.synchronize do
+        synchronize do
           super(*args,&blk)
         end
       end

data/lib/lru_redux/version.rb

@@ -1,3 +1,3 @@
 module LruRedux
-  VERSION = "0.0.6"
+  VERSION = "0.8.1"
 end

data/test/cache_test.rb

@@ -96,4 +96,19 @@ class CacheTest < MiniTest::Unit::TestCase
     @c.max_size = 1
     assert_equal [[:c,3]], @c.to_a
   end
+
+  def test_each
+    @c.max_size = 2
+    @c[:a] = 1
+    @c[:b] = 2
+    @c[:c] = 3
+
+    pairs = []
+    @c.each do |pair|
+      pairs << pair
+    end
+
+    assert_equal [[:c,3],[:b, 2]], pairs
+
+  end
 end

data/test/thread_safe_cache_test.rb

@@ -1,22 +1,17 @@
-require 'lru_redux'
-require 'minitest/autorun'
-require 'minitest/pride'
+require 'cache_test'
 
-class ThreadSafeCacheTest < MiniTest::Unit::TestCase
-  def test_additions
-    cache = LruRedux::ThreadSafeCache.new(1000)
-    threads = []
-    4.times do |t|
-      threads << Thread.new do
-        250.times do |i|
-          cache[i] = "#{t} #{i}"
-        end
-      end
-    end
+class ThreadSafeCacheTest < CacheTest
+  def setup
+    @c = LruRedux::ThreadSafeCache.new(3)
+  end
 
-    threads.each{|t| t.join}
-    assert_equal cache.count,250
-    assert true, cache.valid?
+  def test_recursion
+      @c[:a] = 1
+      @c[:b] = 2
 
+      # should not blow up
+      @c.each do |k,v|
+        @c[k]
+      end
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: lru_redux
 version: !ruby/object:Gem::Version
-  version: 0.0.6
+  version: 0.8.1
 platform: ruby
 authors:
 - Sam Saffron
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-04-23 00:00:00.000000000 Z
+date: 2013-09-06 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -102,6 +102,7 @@ extensions: []
 extra_rdoc_files: []
 files:
 - .gitignore
+- CHANGELOG
 - Gemfile
 - Guardfile
 - LICENSE.txt
@@ -136,7 +137,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.0.0.rc.2
+rubygems_version: 2.0.3
 signing_key: 
 specification_version: 4
 summary: An efficient implementation of an lru cache