Skip to content

Turn off syn/clone-impls feature#273

Merged
dtolnay merged 2 commits intomasterfrom
cloneimpls
Jul 7, 2024
Merged

Turn off syn/clone-impls feature#273
dtolnay merged 2 commits intomasterfrom
cloneimpls

Conversation

@dtolnay
Copy link
Copy Markdown
Owner

@dtolnay dtolnay commented Jul 7, 2024

No description provided.

dtolnay added 2 commits July 7, 2024 13:16
When syn/clone-impls isn't enabled, this type is not Copy.

    warning: this argument (4 byte) is passed by reference, but would be more efficient if passed by value (limit: 8 byte)
      --> src/lifetime.rs:22:49
       |
    22 |     fn visit_opt_lifetime(&mut self, reference: &Token![&], lifetime: &mut Option<Lifetime>) {
       |                                                 ^^^^^^^^^^ help: consider passing by value instead: `$crate::token::And`
       |
       = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#trivially_copy_pass_by_ref
       = note: `-W clippy::trivially-copy-pass-by-ref` implied by `-W clippy::pedantic`
       = help: to override `-W clippy::pedantic` add `#[allow(clippy::trivially_copy_pass_by_ref)]`
@dtolnay dtolnay merged commit 4ec740e into master Jul 7, 2024
@dtolnay dtolnay deleted the cloneimpls branch July 7, 2024 20:22
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.

1 participant