Merge pull request #52 from ruby/revert-49

Revert "Allow use of DelegateClass in ractors"

Author: nobu

lib/delegate.rb

@@ -345,16 +345,10 @@ def __setobj__(obj)
 end
 
 def Delegator.delegating_block(mid) # :nodoc:
-  prok = lambda do |*args, &block|
+  lambda do |*args, &block|
     target = self.__getobj__
     target.__send__(mid, *args, &block)
-  end
-  prok.ruby2_keywords
-  if defined?(Ractor.shareable_proc)
-    Ractor.shareable_proc(&prok)
-  else
-    prok
-  end
+  end.ruby2_keywords
 end
 
 #

test/test_delegate.rb

@@ -403,20 +403,4 @@ def test(a, k:) [a, k]; end
     a = DelegateClass(k).new(k.new)
     assert_equal([1, 0], a.test(1, k: 0))
   end
-
-  def test_delegate_class_can_be_used_in_ractors
-    omit "no Ractor#value" unless defined?(Ractor) && Ractor.method_defined?(:value)
-    require_path = File.expand_path(File.join(__dir__, "..", "lib", "delegate.rb"))
-    raise "file doesn't exist: #{require_path}" unless File.exist?(require_path)
-    assert_ractor <<-RUBY
-      require "#{require_path}"
-      class MyClass < DelegateClass(Array);end
-      values = 2.times.map do
-        Ractor.new do
-          MyClass.new([1,2,3]).at(0)
-        end
-      end.map(&:value)
-      assert_equal [1,1], values
-    RUBY
-  end
 end