Skip to content

cleanup of redundant patterns#6546

Open
jogru0 wants to merge 2 commits into
rust-lang:mainfrom
jogru0:redundant_pattern
Open

cleanup of redundant patterns#6546
jogru0 wants to merge 2 commits into
rust-lang:mainfrom
jogru0:redundant_pattern

Conversation

@jogru0
Copy link
Copy Markdown

@jogru0 jogru0 commented Apr 21, 2025

Just some minor code cleanups.

Comment thread src/types.rs Outdated
Comment thread src/types.rs
// and any of the internal arrays contains more than one item;
let retry_with_force_newline = match context.config.style_edition() {
style_edition @ _ if style_edition <= StyleEdition::Edition2021 => {
Edition2015 | Edition2018 | Edition2021 => {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

We don't need to import StyleEdition::*. Should still fit on one line even if you use the fully qualified variant names.

Suggested change
Edition2015 | Edition2018 | Edition2021 => {
StyleEdition::Edition2015 | StyleEdition::Edition2018 | StyleEdition::Edition2021 => {

@jieyouxu jieyouxu added S-waiting-on-review Status: awaiting review from the assignee but also interested parties. and removed pr-not-reviewed labels Feb 21, 2026
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.

5 participants