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

support suppressing default Clone derivation #325

Closed
mathstuf opened this issue Jun 27, 2024 · 1 comment
Closed

support suppressing default Clone derivation #325

mathstuf opened this issue Jun 27, 2024 · 1 comment

Comments

@mathstuf
Copy link
Contributor

I'd like to instead use perfect_derive(Clone) than derive(Clone). I can do the former with builder_struct_addr, but this conflicts with the derive(Clone) added by default. A direct dependency on perfect_derive seems excessive, but an escape hatch to do Clone in a different way would be nice.
mathstuf added a commit to mathstuf/rust-derive-builder that referenced this issue Jun 27, 2024
@mathstuf
Add suppress_derive_clone struct attribute
08c76e4 
This attribute suppresses the default-generated `derive(Clone)`. This
may be necessary where `derive(Clone)` copies constraints in an
unsatisfiable way.

See: https://smallcultfollowing.com/babysteps//blog/2022/04/12/implied-bounds-and-perfect-derive/
See: rust-lang/rust#26925
Fixes: colin-kiegel#325
mathstuf added a commit to mathstuf/rust-derive-builder that referenced this issue Jun 27, 2024
@mathstuf
Add suppress_derive_clone struct attribute
04c3644 
This attribute suppresses the default-generated `derive(Clone)`. This
may be necessary where `derive(Clone)` copies constraints in an
unsatisfiable way.

See: https://smallcultfollowing.com/babysteps//blog/2022/04/12/implied-bounds-and-perfect-derive/
See: rust-lang/rust#26925
Fixes: colin-kiegel#325
@mathstuf mathstuf mentioned this issue Jun 27, 2024
Closed
mathstuf added a commit to mathstuf/rust-derive-builder that referenced this issue Jun 27, 2024
@mathstuf
Add suppress_derive_clone struct attribute
6b6aac6 
This attribute suppresses the default-generated `derive(Clone)`. This
may be necessary where `derive(Clone)` copies constraints in an
unsatisfiable way.

See: https://smallcultfollowing.com/babysteps//blog/2022/04/12/implied-bounds-and-perfect-derive/
See: rust-lang/rust#26925
Fixes: colin-kiegel#325
mathstuf added a commit to mathstuf/rust-derive-builder that referenced this issue Jun 27, 2024
@mathstuf
Add suppress_derive_clone struct attribute
d30884a 
This attribute suppresses the default-generated `derive(Clone)`. This
may be necessary where `derive(Clone)` copies constraints in an
unsatisfiable way.

See: https://smallcultfollowing.com/babysteps//blog/2022/04/12/implied-bounds-and-perfect-derive/
See: rust-lang/rust#26925
Fixes: colin-kiegel#325
mathstuf added a commit to mathstuf/rust-derive-builder that referenced this issue Jun 27, 2024
@mathstuf
Add suppress_derive_clone struct attribute
3b55bb5 
This attribute suppresses the default-generated `derive(Clone)`. This
may be necessary where `derive(Clone)` copies constraints in an
unsatisfiable way.

See: https://smallcultfollowing.com/babysteps//blog/2022/04/12/implied-bounds-and-perfect-derive/
See: rust-lang/rust#26925
Fixes: colin-kiegel#325
mathstuf added a commit to mathstuf/rust-derive-builder that referenced this issue Jun 27, 2024
@mathstuf
Add suppress_derive_clone struct attribute
1dd03d5 
This attribute suppresses the default-generated `derive(Clone)`. This
may be necessary where `derive(Clone)` copies constraints in an
unsatisfiable way.

See: https://smallcultfollowing.com/babysteps//blog/2022/04/12/implied-bounds-and-perfect-derive/
See: rust-lang/rust#26925
Fixes: colin-kiegel#325
@mathstuf
Copy link
Contributor Author

Suppression is more reliable with pattern = "owned" instead.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add suppress_derive_clone struct attribute mathstuf/rust-derive-builder
1 participant
@mathstuf