Merging options donesn't have to be synchronized

2019-08-31 15:29:22 +09:00 · 2019-08-31 15:29:22 +09:00 · 2c528dd446
parent dd1aa73889
commit 2c528dd446
1 changed files with 1 additions and 1 deletions
--- a/activesupport/lib/active_support/cache/memory_store.rb
+++ b/activesupport/lib/active_support/cache/memory_store.rb
@ -159,8 +159,8 @@ module ActiveSupport
        end

        def modify_value(name, amount, options)
+          options = merged_options(options)
          synchronize do
-            options = merged_options(options)
            if num = read(name, options)
              num = num.to_i + amount
              write(name, num, options)