Skip to content

Conversation

@hkBst
Copy link
Member

@hkBst hkBst commented Jan 23, 2025

No description provided.

@rustbot
Copy link
Collaborator

rustbot commented Jan 23, 2025

r? @wesleywiser

rustbot has assigned @wesleywiser.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

@rustbot rustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. labels Jan 23, 2025
@rust-log-analyzer

This comment has been minimized.

@compiler-errors
Copy link
Member

This seems inconsistent at best, and slight code churn at worst. Thanks for the contribution! but I don't think it provides much value, so I gonna go ahead and close it :)

@hkBst
Copy link
Member Author

hkBst commented Jan 26, 2025

@compiler-errors At the time, I thought I could use this change to test whether this enum was used outside this crate. I've since realised that does not work because the enum itself is also exported. However it does show that the variant names aren't used without qualification by the enum name. The inconsistency can be fixed by changing the other enums to also not publically export their variants. I don't think you would disagree that there is a small code smell here. I should probably have made this PR a draft, but maybe you closed this too quickly?

@hkBst hkBst deleted the patch-18 branch February 2, 2025 11:09
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. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants