Skip to content

No numbered parameters in Binding#local_variables#12746

Merged
mame merged 3 commits intoruby:masterfrom
mame:no-numparams-in-local_variables
Feb 18, 2025
Merged

No numbered parameters in Binding#local_variables#12746
mame merged 3 commits intoruby:masterfrom
mame:no-numparams-in-local_variables

Conversation

@mame
Copy link
Copy Markdown
Member

@mame mame commented Feb 14, 2025

@launchable-app
Copy link
Copy Markdown

launchable-app bot commented Feb 14, 2025

All Tests passed!

✖️no tests failed ✔️61616 tests passed(3 flakes)

@mame mame force-pushed the no-numparams-in-local_variables branch from 4d9e898 to d780f74 Compare February 14, 2025 05:23
@mame mame requested a review from nobu February 14, 2025 07:55
@mame mame marked this pull request as ready for review February 18, 2025 01:29
... instead of "rb_env_t **" because no one uses the updated env.
@mame mame force-pushed the no-numparams-in-local_variables branch from d780f74 to 9df39ba Compare February 18, 2025 01:29
mame added 2 commits February 18, 2025 11:10
Also, Binding#local_variable_get and #local_variable_set rejects an
access to numbered parameters.

[Bug #20965] [Bug #21049]
@mame mame force-pushed the no-numparams-in-local_variables branch from 9df39ba to d857602 Compare February 18, 2025 02:10
@mame mame merged commit 3a0d00d into ruby:master Feb 18, 2025
@mame mame deleted the no-numparams-in-local_variables branch February 18, 2025 07:23
@eregon
Copy link
Copy Markdown
Member

eregon commented Feb 18, 2025

@eregon
Copy link
Copy Markdown
Member

eregon commented Feb 18, 2025

BTW doesn't ruby/ruby CI run test-all with -DRUBY_DEBUG=1? It should have caught that and it seems to fail fairly reliably in that linked build. It'd be great to add such a CI job.

headius added a commit to headius/jruby that referenced this pull request Nov 18, 2025
headius added a commit to headius/jruby that referenced this pull request Jan 28, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants