diff options
author | 卜部昌平 <shyouhei@ruby-lang.org> | 2020-06-17 09:58:35 +0900 |
---|---|---|
committer | 卜部昌平 <shyouhei@ruby-lang.org> | 2020-06-17 10:06:01 +0900 |
commit | af6e63a9df5cd21d57a7a431cf5b45234d1f1b20 (patch) | |
tree | 97e8d65c3296d09f49ac9f2e4c7fe48cf3631667 /proc.c | |
parent | ccb7a4b9f22c6ef8d51517322f82031fadf68769 (diff) |
rb_method_name_error: delete unused code
If you look at the code flow (break -> goto), this assignment never
makes any sense. Should just remove.
I _guess_ this behaviour is unintended. Original code at commit
4dc1a2180946ab793adee5eb235fc4ee8fa4cefe did something. It might be
the code flow that is buggy. However rubyspec already includes this
particular edge case at ruby/core/module/undef_method_spec.rb. I don't
think we can change the way it is any longer.
Diffstat (limited to 'proc.c')
-rw-r--r-- | proc.c | 1 |
1 files changed, 0 insertions, 1 deletions
@@ -1817,7 +1817,6 @@ rb_method_name_error(VALUE klass, VALUE str) case T_MODULE: case T_CLASS: c = obj; - s = MSG(""); break; default: break; |