Don't use Object#inspect in Kernel#method

Yorick Peterse · Yorick Peterse · commit 76701eba77a5 · 2015-04-21T22:07:13.000+02:00
As discussed already in #3373 some people might overwrite this in a way that's not compatible with the default behaviour. While we can't (and are not going to) guard ourselves against every possible case where somebody might've overwritten a core Ruby method, we _can_ at least in this particular case remove the dependency on Object#inspect. As an aside this now ensures that our error messages are the same as MRI's, although one could argue that this is a bit odd: class Foo end Foo.method(:bar) # => NameError: undefined method `barr' for class `Class' Instead of: Foo.method(:bar) # => NameError: undefined method `barr' for class `Foo'
diff --git a/kernel/common/kernel.rb b/kernel/common/kernel.rb
@@ -524,7 +524,7 @@ def method(name)
     elsif respond_to_missing?(name, true)
       Method.new(self, self.class, Rubinius::MissingMethod.new(self,  name), name)
     else
-      raise NameError, "undefined method `#{name}' for #{self.inspect}"
+      raise NameError, "undefined method `#{name}' for class #{self.class}"
     end
   end
 
