From 436086d7664268f18ee0c4951fe39fca5dadec9c Mon Sep 17 00:00:00 2001 From: matz Date: Fri, 16 Oct 2009 11:04:15 +0000 Subject: * lib/delegate.rb (Delegator#respond_to_missing): warn if optional include_private argument is not false. Delegator does (and should) not forward private methods. [ruby-core:26080] * lib/delegate.rb (Delegator#respond_to_missing): instead of redefining #respond_to?, use #respond_to_missing?. git-svn-id: http://svn.ruby-lang.org/repos/ruby/trunk@25372 b2dd03c8-39d4-4d8f-98ff-823fe69b080e --- lib/delegate.rb | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) (limited to 'lib') diff --git a/lib/delegate.rb b/lib/delegate.rb index 57c479c61..215ebf5e4 100644 --- a/lib/delegate.rb +++ b/lib/delegate.rb @@ -146,9 +146,11 @@ class Delegator # Checks for a method provided by this the delegate object by fowarding the # call through \_\_getobj\_\_. # - def respond_to?(m, include_private = false) - return true if super - return self.__getobj__.respond_to?(m, include_private) + def respond_to_missing?(m, include_private = false) + if include_private + warn "#{caller(3)[0]}: delegator does not forward private methods" + end + self.__getobj__.respond_to?(m) end # -- cgit