[ruby-core:122761] [Ruby Bug#21356] Error when accessing local variable named "default" with Binding#local_variable_get

Issue #21356 has been updated by k0kubun (Takashi Kokubun).

Backport changed from 3.2: REQUIRED, 3.3: DONTNEED, 3.4: REQUIRED to 3.2: R=
EQUIRED, 3.3: DONTNEED, 3.4: DONTNEED

Same here; the behavior was added in Ruby 3.5 and we don't even have the mo=
dified function in Ruby 3.4. I mark it "DONTNEED" for ruby_3_4 as well.

----------------------------------------
Bug #21356: Error when accessing local variable named "default" with Bindin=
g#local_variable_get
https://bugs.ruby-lang.org/issues/21356#change-114040

* Author: richardboehme (Richard B=F6hme)
* Status: Closed
* ruby -v: ruby 3.5.0dev (2025-05-21T06:12:18Z master f6cbf499bc) +PRISM [x=
86_64-linux]
* Backport: 3.2: REQUIRED, 3.3: DONTNEED, 3.4: DONTNEED
----------------------------------------
On Ruby master when using `Binding#local_variable_get` with the name `:defa=
ult` it raises the following error:

```
'Binding#local_variable_get': numbered parameter 'default' is not a local v=
ariable (NameError)
```

I saw that this error was introduced in #21049 but it seems like the check =
for whether the passed variable name is a numbered parameter does not work =
for the `:default` symbol.=20

I think this might be an off-by-one error in the `rb_numparam_id_p` functio=
n in proc.c, where instead of adding 9 (the maximum number of numbered para=
meters) to tNUMPARAM_1 we currently add 10 which then includes the id of de=
fault as well? Compiling with this change (see attached diff) fixed the iss=
ue but I'm not sure if it might break other things.

---Files--------------------------------
patch.diff (384 Bytes)


--=20
https://bugs.ruby-lang.org/
______________________________________________
 ruby-core mailing list -- [e-mail]
 To unsubscribe send an email to [e-mail]
 ruby-core info -- https://ml.ruby-lang.org/mailman3/lists/ruby-core.ml.rub=
y-lang.org/