You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Right now the unused struct warning will not trigger if the struct has some impls:
structFoo {} // No warning hereimplFoo {}
Happy Case
The above code warns that Foo is unused.
Checking what Rust does, it says that Foo is never constructed. So I'm think that in the case of structs, we don't need to check if they are referenced: we need to check if they are constructed. If not, then they are unused.
Workaround
None
Workaround Description
No response
Additional Context
No response
Project Impact
None
Blocker Context
No response
Would you like to submit a PR for this Issue?
None
Support Needs
No response
The text was updated successfully, but these errors were encountered:
# Description
## Problem
Fixes#6055
## Summary
This is similar to how it works in Rust.
## Additional Context
## Documentation
Check one:
- [ ] No documentation needed.
- [x] Documentation included in this PR.
- [ ] **[For Experimental Features]** Documentation to be submitted in a
separate PR.
# PR Checklist
- [x] I have tested the changes locally.
- [x] I have formatted the changes with [Prettier](https://prettier.io/)
and/or `cargo fmt` on default settings.
Problem
Right now the unused struct warning will not trigger if the struct has some impls:
Happy Case
The above code warns that
Foo
is unused.Checking what Rust does, it says that
Foo
is never constructed. So I'm think that in the case of structs, we don't need to check if they are referenced: we need to check if they are constructed. If not, then they are unused.Workaround
None
Workaround Description
No response
Additional Context
No response
Project Impact
None
Blocker Context
No response
Would you like to submit a PR for this Issue?
None
Support Needs
No response
The text was updated successfully, but these errors were encountered: