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.

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

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

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

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 moved this to To do in 2024June May 28, 2024
@mulkieran mulkieran removed this from 2024June Jul 3, 2024
@mulkieran mulkieran moved this to Todo in 2024July Jul 3, 2024
@mulkieran mulkieran removed this from 2024July Aug 2, 2024
@mulkieran mulkieran moved this from Todo to In Progress (long term) in 2024August Aug 2, 2024
@mulkieran mulkieran removed this from 2024August Sep 5, 2024
@mulkieran mulkieran moved this from Todo to In Progress (long term) in 2024September Sep 5, 2024
@mulkieran mulkieran removed this from 2024September Oct 7, 2024
@mulkieran mulkieran moved this from Todo to In Progress (long term) in 2024October Oct 7, 2024
@mulkieran mulkieran removed this from 2024October Nov 4, 2024
@mulkieran mulkieran moved this from Todo to In Progress (long term) in 2024November Nov 4, 2024
@mulkieran mulkieran removed this from 2024November Dec 3, 2024
@mulkieran mulkieran moved this from Todo to In Progress (long term) in 2024December Dec 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: In Progress (long term)
Development

No branches or pull requests

1 participant