Fix to_s methods

ruby-concurrency · Feb 21, 2018 · 688d765 · 688d765
1 parent 3265766
commit 688d765
Show file tree

Hide file tree

Showing 7 changed files with 9 additions and 9 deletions.
diff --git a/lib/concurrent/atomic/atomic_boolean.rb b/lib/concurrent/atomic/atomic_boolean.rb
@@ -114,7 +114,7 @@ module Concurrent
   class AtomicBoolean < AtomicBooleanImplementation
     # @return [String] Short string representation.
     def to_s
-      format '<#%s:0x%x value:%s>', self.class, object_id << 1, value
+      format '%s value:%s>', super[0..-2], value
     end
 
     alias_method :inspect, :to_s

diff --git a/lib/concurrent/atomic/atomic_fixnum.rb b/lib/concurrent/atomic/atomic_fixnum.rb
@@ -131,7 +131,7 @@ module Concurrent
   class AtomicFixnum < AtomicFixnumImplementation
     # @return [String] Short string representation.
     def to_s
-      format '<#%s:0x%x value:%s>', self.class, object_id << 1, value
+      format '%s value:%s>', super[0..-2], value
     end
 
     alias_method :inspect, :to_s

diff --git a/lib/concurrent/atomic/atomic_reference.rb b/lib/concurrent/atomic/atomic_reference.rb
@@ -44,7 +44,7 @@ class Concurrent::AtomicReference < Concurrent::MutexAtomicReference
 class Concurrent::AtomicReference
   # @return [String] Short string representation.
   def to_s
-    format '<#%s:0x%x value:%s>', self.class, object_id << 1, get
+    format '%s value:%s>', super[0..-2], get
   end
 
   alias_method :inspect, :to_s

diff --git a/lib/concurrent/edge/cancellation.rb b/lib/concurrent/edge/cancellation.rb
@@ -54,7 +54,7 @@ def canceled?
     # Short string representation.
     # @return [String]
     def to_s
-      format '<#%s:0x%x canceled:%s>', self.class, object_id << 1, canceled?
+      format '%s canceled:%s>', super[0..-2], canceled?
     end
 
     alias_method :inspect, :to_s
@@ -120,7 +120,7 @@ def join(*tokens, &block)
       # Short string representation.
       # @return [String]
       def to_s
-        format '<#%s:0x%x canceled:%s>', self.class, object_id << 1, canceled?
+        format '%s canceled:%s>', super[0..-2], canceled?
       end
 
       alias_method :inspect, :to_s

diff --git a/lib/concurrent/edge/lock_free_stack.rb b/lib/concurrent/edge/lock_free_stack.rb
@@ -118,7 +118,7 @@ def clear_each(&block)
 
     # @return [String] Short string representation.
     def to_s
-      format '<#%s:0x%x %s>', self.class, object_id << 1, to_a.to_s
+      format '%s %s>', super[0..-2], to_a.to_s
     end
 
     alias_method :inspect, :to_s

diff --git a/lib/concurrent/edge/promises.rb b/lib/concurrent/edge/promises.rb
@@ -626,7 +626,7 @@ def chain_on(executor, *args, &task)
 
       # @return [String] Short string representation.
       def to_s
-        format '<#%s:0x%x %s>', self.class, object_id << 1, state
+        format '%s %s>', super[0..-2], state
       end
 
       alias_method :inspect, :to_s
@@ -2067,7 +2067,7 @@ def pop_for_select(probe = Concurrent::Promises.resolvable_future)
 
       # @return [String] Short string representation.
       def to_s
-        format '<#%s:0x%x size:%s>', self.class, object_id << 1, @Size
+        format '%s size:%s>', super[0..-2], @Size
       end
 
       alias_method :inspect, :to_s

diff --git a/lib/concurrent/edge/throttle.rb b/lib/concurrent/edge/throttle.rb
@@ -117,7 +117,7 @@ def throttled_block(&block)
 
     # @return [String] Short string representation.
     def to_s
-      format '<#%s:0x%x limit:%s can_run:%d>', self.class, object_id << 1, @Limit, can_run
+      format '%s limit:%s can_run:%d>', super[0..-2], @Limit, can_run
     end
 
     alias_method :inspect, :to_s