github/rails
1
1
Fork 0
mirror of https://github.com/github/rails.git synced 2026-04-26 03:00:59 -04:00
Code Issues Packages Projects Releases Wiki Activity

remove_possible_method: test if method exists

Browse Source 
This speeds up remove_possible_method substantially since it doesn't
have to rescue a NameError in the common case.

Closes #2346.
This commit is contained in:
Brad Ediger
2011-07-31 07:38:38 -05:00
parent 4c85c4ed95
commit 7f88539087
1 changed files with 7 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
activesupport/lib/active_support/core_ext/module/remove_method.rb
View File

@@ -1,11 +1,16 @@
class Module
def remove_possible_method(method)
remove_method(method)
if method_defined?(method) || private_method_defined?(method)
remove_method(method)
end
rescue NameError
# If the requested method is defined on a superclass or included module,
# method_defined? returns true but remove_method throws a NameError.
# Ignore this.
end
def redefine_method(method, &block)
remove_possible_method(method)
define_method(method, &block)
end
end
end