Skip to content

Spec: no verbose warning for inner block params#1352

Merged
eregon merged 1 commit intoruby:masterfrom
sampokuokkanen:destructured-param-unused-warning
May 3, 2026
Merged

Spec: no verbose warning for inner block params#1352
eregon merged 1 commit intoruby:masterfrom
sampokuokkanen:destructured-param-unused-warning

Conversation

@sampokuokkanen
Copy link
Copy Markdown
Contributor

Add a spec to ensure block-parameter destructuring does not emit unused variable warnings in verbose mode. The test evals a proc with parameters |key, (val1, val2)| and asserts it should_not complain(verbose: true), preventing false-positive warnings for inner destructured names.

This came up while working on jruby/jruby#9400. There is an MRI test for the method-definition case, but no test for destructured block arguments.

Add a spec to ensure block-parameter destructuring does not emit unused-name warnings in verbose mode. The test evals a proc with parameters |key, (val1, val2)| and asserts it should_not complain(verbose: true), preventing false-positive warnings for inner destructured names.
Copy link
Copy Markdown
Member

@eregon eregon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@eregon eregon merged commit 990cdde into ruby:master May 3, 2026
14 checks passed
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.

2 participants