sin_lru_redux 2.1.0 → 2.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 10d8a07bc1dcf5957a24804163d1ac62086dac85b3cefce3ff305757231a3696
-  data.tar.gz: 4820d035651c937d4486800a4ca3b1fda48ccfb1118d0bcd420bf49b88c5583e
+  metadata.gz: 3a4f3922e84907ba0de45efa46ee990b5947f28a7cd0c636de9ed268bc8a2335
+  data.tar.gz: 4e0da38bf6e25d814751eb2304a409dde4da530db49dc17b9c7fbd44ddb16e1b
 SHA512:
-  metadata.gz: 3cf6042d5c6cb2203e2505a4db581263e4bbe99475b1928558ee4a541a73d60b16c3d162199a11b103240e02f9ec09ece7c93f0b05fed10660c5fd84cec32907
-  data.tar.gz: 22d4e5addfb2efd94536334fddbb97f6cdd99237ca5215f6132b683ecce46bdae00d8e002239cfcc40ec26d0a37ff7b6fff08cb2a001518bb47a1a7bfaafc0f7
+  metadata.gz: ce0f773d075691a31efb284a2ce23f860cf225d38450020ca32a280e22fb39a1941e79f0c7adb775fbf07c5bf1ecd93241b55b468b500abbc64f2698bfa6e60e
+  data.tar.gz: 58b1096ed7841aa618080bd85fbe00d42f056e8784cf9ce0ee03ec5b82792bfcd497abd9c90026e1374b5b513804f78c517ec5f648bef80ed23b94db4b650bd4

data/CHANGELOG.md

@@ -1,5 +1,15 @@
 # Changelog
 
+## [2.2.0] - 2024-12-31
+
+- New: Raise an error when trying to assign nil to max_size, ttl, or ignore_nil
+- Fix: Fix a bug where ignore_nil was not updated when ignore_nil was specified as false
+- Fix: Fix a bug where items with nil value were deleted when ignore_nil was changed
+- Refactor: Improve implementation
+- Test: Improve tests
+- Other: Improve CI
+- Other: Improve documents
+
 ## [2.1.0] - 2024-12-30
 
 - New: Set default max_size to 1000

data/README.md

@@ -93,7 +93,7 @@ cache.to_a
 # => [[:b, '2'], [:a, '1']]
 
 # Now we advance time 5 min 30 sec into the future.
-Timecop.freeze(Time.now + 330)
+Timecop.freeze(Time.now + ((5 * 60) + 30))
 
 # And we see that the expired values have been evicted.
 cache.to_a
@@ -104,7 +104,7 @@ cache.to_a
 cache[:a] = '1'
 cache[:b] = '2'
 
-Timecop.freeze(Time.now + 330)
+Timecop.freeze(Time.now + ((5 * 60) + 30))
 
 cache.ttl = 10 * 60
 
@@ -115,7 +115,7 @@ cache.to_a
 # => [[:b, '2'], [:a, '1']]
 
 # TTL eviction can be triggered manually with the #expire method.
-Timecop.freeze(Time.now + 330)
+Timecop.freeze(Time.now + ((5 * 60) + 30))
 
 cache.expire
 cache.to_a
@@ -162,7 +162,7 @@ cache = LruRedux::TTL::ThreadSafeCache.new(100, 5 * 60)
 
 ## Contributing
 
-Bug reports and pull requests are welcome on GitHub at https://github.com/[USERNAME]/abcd. This project is intended to be a safe, welcoming space for collaboration, and contributors are expected to adhere to the [code of conduct](https://github.com/cadenza-tech/sin_lru_redux/blob/main/CODE_OF_CONDUCT.md).
+Bug reports and pull requests are welcome on GitHub at https://github.com/cadenza-tech/sin_lru_redux. This project is intended to be a safe, welcoming space for collaboration, and contributors are expected to adhere to the [code of conduct](https://github.com/cadenza-tech/sin_lru_redux/blob/main/CODE_OF_CONDUCT.md).
 
 ## License
 

data/lib/lru_redux/cache.rb

@@ -18,8 +18,6 @@ class LruRedux::Cache
   end
 
   def max_size=(new_max_size)
-    new_max_size ||= @max_size
-
     validate_max_size!(new_max_size)
 
     @max_size = new_max_size
@@ -31,8 +29,6 @@ class LruRedux::Cache
   end
 
   def ignore_nil=(new_ignore_nil)
-    new_ignore_nil ||= @ignore_nil
-
     validate_ignore_nil!(new_ignore_nil)
 
     @ignore_nil = new_ignore_nil
@@ -144,10 +140,14 @@ class LruRedux::Cache
 
   if RUBY_VERSION >= '2.6.0'
     def evict_nil
+      return unless @ignore_nil
+
       @data.compact!
     end
   else
     def evict_nil
+      return unless @ignore_nil
+
       @data.reject! { |_key, value| value.nil? }
     end
   end

data/lib/lru_redux/ttl/cache.rb

@@ -24,17 +24,14 @@ module LruRedux
       end
 
       def max_size=(new_max_size)
-        new_max_size ||= @max_size
-
         validate_max_size!(new_max_size)
 
         @max_size = new_max_size
-        resize
+        evict_expired
+        evict_excess
       end
 
       def ttl=(new_ttl)
-        new_ttl ||= @ttl
-
         validate_ttl!(new_ttl)
 
         @ttl = new_ttl
@@ -42,8 +39,6 @@ module LruRedux
       end
 
       def ignore_nil=(new_ignore_nil)
-        new_ignore_nil ||= @ignore_nil
-
         validate_ignore_nil!(new_ignore_nil)
 
         @ignore_nil = new_ignore_nil
@@ -192,11 +187,6 @@ module LruRedux
         raise ArgumentError.new("Invalid ignore_nil: #{ignore_nil.inspect}")
       end
 
-      def resize
-        evict_expired
-        evict_if_exceeded
-      end
-
       def evict_expired
         return if @ttl == :none
 
@@ -209,7 +199,7 @@ module LruRedux
         end
       end
 
-      def evict_if_exceeded
+      def evict_excess
         evict_least_recently_used while @data_lru.size > @max_size
       end
 
@@ -241,7 +231,7 @@ module LruRedux
           @data_lru[key] = value
           @data_ttl[key] = Time.now.to_f
         end
-        evict_if_exceeded
+        evict_excess
         value
       end
     end

data/lib/lru_redux/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module LruRedux
-  VERSION = '2.1.0'
+  VERSION = '2.2.0'
 end

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification
 name: sin_lru_redux
 version: !ruby/object:Gem::Version
-  version: 2.1.0
+  version: 2.2.0
 platform: ruby
 authors:
 - Masahiro
 bindir: exe
 cert_chain: []
-date: 2024-12-30 00:00:00.000000000 Z
+date: 2024-12-31 00:00:00.000000000 Z
 dependencies: []
 description: |
   Efficient and thread-safe LRU cache.
@@ -33,15 +33,15 @@ files:
 - lib/lru_redux/util.rb
 - lib/lru_redux/util/safe_sync.rb
 - lib/lru_redux/version.rb
-homepage: https://github.com/cadenza-tech/sin_lru_redux/tree/v2.1.0
+homepage: https://github.com/cadenza-tech/sin_lru_redux/tree/v2.2.0
 licenses:
 - MIT
 metadata:
-  homepage_uri: https://github.com/cadenza-tech/sin_lru_redux/tree/v2.1.0
-  source_code_uri: https://github.com/cadenza-tech/sin_lru_redux/tree/v2.1.0
-  changelog_uri: https://github.com/cadenza-tech/sin_lru_redux/blob/v2.1.0/CHANGELOG.md
+  homepage_uri: https://github.com/cadenza-tech/sin_lru_redux/tree/v2.2.0
+  source_code_uri: https://github.com/cadenza-tech/sin_lru_redux/tree/v2.2.0
+  changelog_uri: https://github.com/cadenza-tech/sin_lru_redux/blob/v2.2.0/CHANGELOG.md
   bug_tracker_uri: https://github.com/cadenza-tech/sin_lru_redux/issues
-  documentation_uri: https://rubydoc.info/gems/sin_lru_redux/2.1.0
+  documentation_uri: https://rubydoc.info/gems/sin_lru_redux/2.2.0
   rubygems_mfa_required: 'true'
 rdoc_options: []
 require_paths: