Bug #124

Module#remove_method broken in 1.8.6-p166

Added by Arthur Schreiber over 3 years ago. Updated 9 months ago.

[ruby-core:17154]
Status:Closed Start date:06/09/2008
Priority:Normal Due date:
Assignee:- % Done:

100%

Category:-
Target version:-
ruby -v:

Description

Module#remove Method should raise a NameError when trying to remove non-existant methods.

On Ruby 1.8.6-p111, Ruby 1.8.6-p114, Ruby 1.8.7, Ruby 1.8.7-p7 and Ruby 1.8.7-p17:

irb(main):001:0> class A; remove_method :blabla; end
NameError: method `blabla' not defined in A
	from (irb):1:in `remove_method'
	from (irb):1

But on Ruby 1.8.6-p166:

irb(main):001:0> class A; remove_method :blabla; end
=> A

History

Updated by Vladimir Sizikov over 3 years ago

Looks like r16816:
"merge revision(s) 13586:
  * eval.c (remove_method): should not remove undef place holder. [ruby-dev:31817]"

is the reason for this regression.

Updated by Anonymous over 3 years ago

Issue #124 has been updated by Vladimir Sizikov.


Looks like r16816:
"merge revision(s) 13586:
  * eval.c (remove_method): should not remove undef place holder. [ruby-dev:31817]"

is the reason for this regression.
----------------------------------------
Bug #124: Module#remove_method broken in 1.8.6-p166
http://redmine.ruby-lang.org/issues/show/124

Author: Arthur Schreiber
Status: Open
Priority: Normal
Assigned to: 
Category: 
Target version: 


Module#remove Method should raise a NameError when trying to remove non-existant methods.

On Ruby 1.8.6-p111, Ruby 1.8.6-p114, Ruby 1.8.7, Ruby 1.8.7-p7 and Ruby 1.8.7-p17:

irb(main):001:0> class A; remove_method :blabla; end
NameError: method `blabla' not defined in A
	from (irb):1:in `remove_method'
	from (irb):1

But on Ruby 1.8.6-p166:

irb(main):001:0> class A; remove_method :blabla; end
=> A


----------------------------------------
You have received this notification because you have either subscribed to it, or are involved in it.
To change your notification preferences, please click here: http://redmine.ruby-lang.org/my/account

Updated by Nobuyoshi Nakada over 3 years ago

  • Status changed from Open to Closed
  • % Done changed from 0 to 100
Merged to 1.8.6 branch in the repository, at r17646.

Also available in: Atom PDF