diff options
| author | matz <matz@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2009-10-16 11:04:15 +0000 |
|---|---|---|
| committer | matz <matz@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2009-10-16 11:04:15 +0000 |
| commit | 436086d7664268f18ee0c4951fe39fca5dadec9c (patch) | |
| tree | 749609486af2a6813e3f84d18cc91d3b2b7501db /lib | |
| parent | b320370f8f67309b0259e72f10b1cc5ed82d4213 (diff) | |
| download | ruby-436086d7664268f18ee0c4951fe39fca5dadec9c.tar.gz ruby-436086d7664268f18ee0c4951fe39fca5dadec9c.tar.xz ruby-436086d7664268f18ee0c4951fe39fca5dadec9c.zip | |
* 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
Diffstat (limited to 'lib')
| -rw-r--r-- | lib/delegate.rb | 8 |
1 files changed, 5 insertions, 3 deletions
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 # |