Skip to content

fix: [#304] remove clippy large_stack_arrays workaround (false positive resolved in nightly 1.96.0)#425

Merged
josecelano merged 1 commit intomainfrom
304-fix-remove-clippy-large-stack-arrays-workaround
Apr 8, 2026
Merged

fix: [#304] remove clippy large_stack_arrays workaround (false positive resolved in nightly 1.96.0)#425
josecelano merged 1 commit intomainfrom
304-fix-remove-clippy-large-stack-arrays-workaround

Conversation

@josecelano
Copy link
Copy Markdown
Member

Summary

Removes the crate-level #![allow(clippy::large_stack_arrays)] workaround from src/lib.rs that was added to suppress a clippy false positive.

Background

Issue #304 tracked a false positive where clippy incorrectly flagged vec![] macro usage (heap-allocated) as a large stack array. The upstream bug was reported at rust-lang/rust-clippy#12586.

Verification

Tested on nightly 1.96.0 (April 7, 2026):

  1. Temporarily removed the #![allow(clippy::large_stack_arrays)] attribute
  2. Ran cargo clippy --all-targets -- -D clippy::large-stack-arrays → no errors
  3. Ran cargo clippy --all-targets -- -D clippy::pedantic → no errors

The false positive no longer occurs. The workaround is no longer needed.

Changes

  • src/lib.rs: removed 6 lines (the #![allow(clippy::large_stack_arrays)] attribute and its 4 comment lines)

Related

@josecelano josecelano self-assigned this Apr 8, 2026
@josecelano
Copy link
Copy Markdown
Member Author

ACK 00d8146

@josecelano josecelano merged commit 58c1049 into main Apr 8, 2026
33 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.

Clippy large_stack_arrays false positive with vec![] macro

1 participant