Skip to content

Commit df5ebec

Browse files
committed
Merge pull request #418 from amatsuda/warnings
Ruby warnings
2 parents 83824b7 + 968936f commit df5ebec

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

lib/concurrent/atom.rb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -61,7 +61,7 @@ class Atom < Synchronization::Object
6161
include Concern::Observable
6262

6363
safe_initialization!
64-
private *attr_volatile_with_cas(:value)
64+
private(*attr_volatile_with_cas(:value))
6565
public :value
6666

6767
# Create a new atom with the given initial value.

lib/concurrent/synchronization/object.rb

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -79,7 +79,7 @@ def self.new(*)
7979
end
8080

8181
def self.safe_initialization?
82-
@safe_initialization || (superclass.respond_to?(:safe_initialization?) && superclass.safe_initialization?)
82+
(defined?(@safe_initialization) && @safe_initialization) || (superclass.respond_to?(:safe_initialization?) && superclass.safe_initialization?)
8383
end
8484

8585
# For testing purposes, quite slow.
@@ -122,7 +122,7 @@ def compare_and_set_#{name}(expected, value)
122122
end
123123
124124
def update_#{name}(&block)
125-
#{ivar}.update &block
125+
#{ivar}.update(&block)
126126
end
127127
RUBY
128128
end

0 commit comments

Comments
 (0)