summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
author卜部昌平 <shyouhei@ruby-lang.org>2020-06-17 09:58:35 +0900
committer卜部昌平 <shyouhei@ruby-lang.org>2020-06-17 10:06:01 +0900
commitaf6e63a9df5cd21d57a7a431cf5b45234d1f1b20 (patch)
tree97e8d65c3296d09f49ac9f2e4c7fe48cf3631667
parentccb7a4b9f22c6ef8d51517322f82031fadf68769 (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.
-rw-r--r--proc.c1
1 files changed, 0 insertions, 1 deletions
diff --git a/proc.c b/proc.c
index 94722dd8e2..4af42c04bd 100644
--- a/proc.c
+++ b/proc.c
@@ -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;