Skip to content

default_constructed_unit_structs -> is_unit_struct#16873

Closed
nyurik wants to merge 1 commit intorust-lang:masterfrom
nyurik:default_constructed_unit_structs
Closed

default_constructed_unit_structs -> is_unit_struct#16873
nyurik wants to merge 1 commit intorust-lang:masterfrom
nyurik:default_constructed_unit_structs

Conversation

@nyurik
Copy link
Copy Markdown
Contributor

@nyurik nyurik commented Apr 17, 2026

Move is_unit_struct into clippy_utils to use it in the new lint default_mismatches_new

Per @samueltardieu in #16531 (review)

r? @samueltardieu


changelog: none

Move is_unit_struct into clippy_utils to use it in
the new lint default_mismatches_new
@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties label Apr 17, 2026
@samueltardieu
Copy link
Copy Markdown
Member

Please don't create PRs in isolation. For example, this PR has no justification by itself. This should be a separate commit in a PR where this refactoring makes sense.

@nyurik
Copy link
Copy Markdown
Contributor Author

nyurik commented Apr 17, 2026

@samueltardieu same here - this PR is also a commit in #16531, so i'll delete it here

@nyurik nyurik closed this Apr 17, 2026
@rustbot rustbot removed the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties label Apr 17, 2026
@nyurik nyurik deleted the default_constructed_unit_structs branch April 17, 2026 14:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants