Skip to content

Conversation

y21
Copy link
Member

@y21 y21 commented May 21, 2023

Fixes #10741.
Prior to this PR, the lint only checked array repeat expressions (ie. [T; n]). Now it also checks array initializer expressions.

changelog: [large_stack_arrays]: check array initializer expressions

@rustbot
Copy link
Collaborator

rustbot commented May 21, 2023

r? @giraffate

(rustbot has picked a reviewer for you, use r? to override)

@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties label May 21, 2023
@giraffate
Copy link
Contributor

@bors r+

Thanks!

@bors
Copy link
Contributor

bors commented May 23, 2023

📌 Commit e9a98d9 has been approved by giraffate

It is now in the queue for this repository.

@bors
Copy link
Contributor

bors commented May 23, 2023

⌛ Testing commit e9a98d9 with merge ec2f2d5...

@bors
Copy link
Contributor

bors commented May 23, 2023

☀️ Test successful - checks-action_dev_test, checks-action_remark_test, checks-action_test
Approved by: giraffate
Pushing ec2f2d5 to master...

@bors bors merged commit ec2f2d5 into rust-lang:master May 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-review Status: Awaiting review from the assignee but also interested parties
Projects
None yet
Development

Successfully merging this pull request may close these issues.

clippy::large_stack_arrays doesn't detect large arrays defined by comma-separated list
4 participants