Use safe_initialization! instead of ensure_ivar_visibility!

pitr-ch · pitr-ch · commit f6b29a768efc · 2015-10-30T20:53:04.000+01:00
diff --git a/lib/concurrent/async.rb b/lib/concurrent/async.rb
@@ -299,6 +299,7 @@ def new(*args, &block)
     #
     # @!visibility private
     class AsyncDelegator < Synchronization::LockableObject
+      safe_initialization!
 
       # Create a new delegator object wrapping the given delegate.
       #
@@ -308,7 +309,6 @@ def initialize(delegate)
         @delegate = delegate
         @queue = []
         @executor = Concurrent.global_io_executor
-        ensure_ivar_visibility!
       end
 
       # Delegates method calls to the wrapped object.

Original file line number	Diff line number	Diff line change
`@@ -299,6 +299,7 @@ def new(*args, &block)`
`299`	`299`	`#`
`300`	`300`	`# @!visibility private`
`301`	`301`	`class AsyncDelegator < Synchronization::LockableObject`
	`302`	`+ safe_initialization!`
`302`	`303`
`303`	`304`	`# Create a new delegator object wrapping the given delegate.`
`304`	`305`	`#`
`@@ -308,7 +309,6 @@ def initialize(delegate)`
`308`	`309`	`@delegate = delegate`
`309`	`310`	`@queue = []`
`310`	`311`	`@executor = Concurrent.global_io_executor`
`311`		`- ensure_ivar_visibility!`
`312`	`312`	`end`
`313`	`313`
`314`	`314`	`# Delegates method calls to the wrapped object.`