Bug #918
closedsuper doesn't work correctly when used with define_method
Description
=begin
The following prints B,A in 1.8 but B,B,A in 1.9.*.
It seems that the receiver's class (C) is incorrectly used to determine the base class. The class where the target method (B#m) is defined (i.e. B) should be used so that A#m is invoked immediately.
class A
def m
puts 'A'
end
end
class B < A
define_method(:m) do
puts 'B'
super()
end
end
class C < B
end
C.new.m
=end
Updated by antares (Michael Klishin) almost 16 years ago
=begin
Tomas,
I think the same problem has been reported in #730. Could you check if they are duplicates? Thanks.
=end
Updated by ko1 (Koichi Sasada) almost 16 years ago
=begin
Tomas Matousek wrote::
The following prints B,A in 1.8 but B,B,A in 1.9.*.
It seems that the receiver's class (C) is incorrectly used to determine the base class. The class where the target method (B#m) is defined (i.e. B) should be used so that A#m is invoked immediately.
On my environment, following patch fixes this issue. Could you try it?
http://www.atdot.net/sp/readonly/ildack
--
// SASADA Koichi at atdot dot net
=end
Updated by tmat (Tomas Matousek) almost 16 years ago
=begin
It might be related yet there might also be an issue specific to define_method. It depends on implementation details in Ruby.
=end
Updated by tmat (Tomas Matousek) almost 16 years ago
=begin
Thanks for fixing this so quickly. Unfortunately I can't work with Ruby source code. I would need Win32 binaries to test it.
=end
Updated by antares (Michael Klishin) almost 16 years ago
=begin
ko1,
Your patch fixes the problem, though does not help with #730.
I ran test-all as well as test-rubyspec against patched 1.9.1 from today.
=end
Updated by ko1 (Koichi Sasada) almost 16 years ago
- Status changed from Open to Closed
- % Done changed from 0 to 100
=begin
Applied in changeset r20967.
=end