Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Address large_enum_variant clippy allow on DeviceInfo in identify.rs #3575

Open
mulkieran opened this issue Mar 22, 2024 · 0 comments
Open

Address large_enum_variant clippy allow on DeviceInfo in identify.rs #3575

mulkieran opened this issue Mar 22, 2024 · 0 comments
Assignees
@mulkieran
Projects
2024June

Comments

@mulkieran
Copy link
Member

One of the constructors contains a BDA, and is therefore a bit larger than the other. We could box something up, perhaps the BDA. But it would affect other lines of the code, maybe quite a few.
@mulkieran mulkieran self-assigned this Mar 22, 2024
@mulkieran mulkieran added this to To do in 2024March via automation Mar 22, 2024
@mulkieran mulkieran removed this from To do in 2024March Apr 2, 2024
@mulkieran mulkieran added this to To do in 2024April via automation Apr 2, 2024
@mulkieran mulkieran removed this from To do in 2024April Apr 29, 2024
@mulkieran mulkieran added this to To do in 2024May via automation Apr 29, 2024
@mulkieran mulkieran removed this from To do in 2024May May 9, 2024
@mulkieran mulkieran added this to To do in 2024June via automation May 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mulkieran mulkieran

Labels
None yet
Projects
2024June
Status: To do
2024June
To do
Milestone
No milestone
Development

No branches or pull requests
1 participant
@mulkieran