diff --git a/ChangeLog b/ChangeLog index 3b9b4c2a8b..5f28208dbd 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,7 @@ +Thu Jun 7 19:02:48 2007 Tanaka Akira + + * lib/pp.rb: call original "method" method instead of redefined one. + Thu Jun 7 17:20:57 2007 Koichi Sasada * iseq.c (prepare_iseq_build): freeze filename and name string. diff --git a/lib/pp.rb b/lib/pp.rb index 7d2d7ebe05..179f5013fb 100644 --- a/lib/pp.rb +++ b/lib/pp.rb @@ -283,9 +283,9 @@ class PP < PrettyPrint # This module provides predefined #pretty_print methods for some of # the most commonly used built-in classes for convenience. def pretty_print(q) - if /\(Kernel\)#/ !~ method(:inspect).inspect + if /\(Kernel\)#/ !~ Object.instance_method(:method).bind(self).call(:inspect).inspect q.text self.inspect - elsif /\(Kernel\)#/ !~ method(:to_s).inspect && instance_variables.empty? + elsif /\(Kernel\)#/ !~ Object.instance_method(:method).bind(self).call(:to_s).inspect && instance_variables.empty? q.text self.to_s else q.pp_object(self) @@ -317,7 +317,7 @@ class PP < PrettyPrint # However, doing this requires that every class that #inspect is called on # implement #pretty_print, or a RuntimeError will be raised. def pretty_print_inspect - if /\(PP::ObjectMixin\)#/ =~ method(:pretty_print).inspect + if /\(PP::ObjectMixin\)#/ =~ Object.instance_method(:method).bind(self).call(:pretty_print).inspect raise "pretty_print is not overridden for #{self.class}" end PP.singleline_pp(self, '') @@ -516,6 +516,13 @@ if __FILE__ == $0 a = OverriddenStruct.new(1,2) assert_equal("#\n", PP.pp(a, '')) end + + def test_redefined_method + o = "" + def o.method + end + assert_equal(%(""\n), PP.pp(o, "")) + end end class HasInspect