Binding#local_variable_defined? raises a NameError for numbered params.

mame · mame · commit 1794cfe12fe6 · 2025-12-12T15:21:29.000+09:00
[Bug #21776]
diff --git a/proc.c b/proc.c
@@ -641,7 +641,8 @@ bind_local_variable_defined_p(VALUE bindval, VALUE sym)
 
     if (!lid) return Qfalse;
     if (rb_numparam_id_p(lid)) {
-        return Qfalse;
+        rb_name_err_raise("numbered parameter '%1$s' is not a local variable",
+                          bindval, ID2SYM(lid));
     }
 
     GetBindingPtr(bindval, bind);
diff --git a/test/ruby/test_proc.rb b/test/ruby/test_proc.rb
@@ -1659,35 +1659,35 @@ def test_numparam_is_not_local_variables
       assert_equal([], binding.local_variables)
       assert_raise(NameError) { binding.local_variable_get(:_9) }
       assert_raise(NameError) { binding.local_variable_set(:_9, 1) }
-      assert_equal(false, binding.local_variable_defined?(:_9))
+      assert_raise(NameError) { binding.local_variable_defined?(:_9) }
       "bar".tap do
         assert_equal([], binding.local_variables)
         assert_raise(NameError) { binding.local_variable_get(:_9) }
         assert_raise(NameError) { binding.local_variable_set(:_9, 1) }
-        assert_equal(false, binding.local_variable_defined?(:_9))
+        assert_raise(NameError) { binding.local_variable_defined?(:_9) }
       end
       assert_equal([], binding.local_variables)
       assert_raise(NameError) { binding.local_variable_get(:_9) }
       assert_raise(NameError) { binding.local_variable_set(:_9, 1) }
-      assert_equal(false, binding.local_variable_defined?(:_9))
+      assert_raise(NameError) { binding.local_variable_defined?(:_9) }
     end
 
     "foo".tap do
       assert_equal([], binding.local_variables)
       assert_raise(NameError) { binding.local_variable_get(:_9) }
       assert_raise(NameError) { binding.local_variable_set(:_9, 1) }
-      assert_equal(false, binding.local_variable_defined?(:_9))
+      assert_raise(NameError) { binding.local_variable_defined?(:_9) }
       "bar".tap do
         _9 and flunk
         assert_equal([], binding.local_variables)
         assert_raise(NameError) { binding.local_variable_get(:_9) }
         assert_raise(NameError) { binding.local_variable_set(:_9, 1) }
-        assert_equal(false, binding.local_variable_defined?(:_9))
+        assert_raise(NameError) { binding.local_variable_defined?(:_9) }
       end
       assert_equal([], binding.local_variables)
       assert_raise(NameError) { binding.local_variable_get(:_9) }
       assert_raise(NameError) { binding.local_variable_set(:_9, 1) }
-      assert_equal(false, binding.local_variable_defined?(:_9))
+      assert_raise(NameError) { binding.local_variable_defined?(:_9) }
     end
   end
 

Original file line number	Diff line number	Diff line change
`@@ -641,7 +641,8 @@ bind_local_variable_defined_p(VALUE bindval, VALUE sym)`
`641`	`641`
`642`	`642`	`if (!lid) return Qfalse;`
`643`	`643`	`if (rb_numparam_id_p(lid)) {`
`644`		`- return Qfalse;`
	`644`	`+ rb_name_err_raise("numbered parameter '%1$s' is not a local variable",`
	`645`	`+ bindval, ID2SYM(lid));`
`645`	`646`	`}`
`646`	`647`
`647`	`648`	`GetBindingPtr(bindval, bind);`