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

fix: pretty print with suffix xclip or yclip #63

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

fix: pretty print with suffix xclip or yclip

be9eac7
Select commit
Loading
Failed to load commit list.
Sign in for the full log view
Open

fix: pretty print with suffix xclip or yclip #63

fix: pretty print with suffix xclip or yclip
be9eac7
Select commit
Loading
Failed to load commit list.
GitHub Actions / clippy succeeded Jul 14, 2023 in 0s

clippy

5 warnings

Details

Results

Message level Amount
Internal compiler error 0
Error 0
Warning 5
Note 0
Help 0

Versions

  • rustc 1.71.0 (8ede3aae2 2023-07-12)
  • cargo 1.71.0 (cfd3bbd8f 2023-06-08)
  • clippy 0.1.71 (8ede3aa 2023-07-12)

Annotations

Check warning on line 296 in src/strand.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

dereferencing a tuple pattern where every element takes a reference

warning: dereferencing a tuple pattern where every element takes a reference
   --> src/strand.rs:296:36
    |
296 |             (&Option::Some(ref x), &Option::Some(ref x1)) => x.same(x1),
    |                                    ^^^^^^^^^^^^^^^^^^^^^
    |
    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrowed_reference
help: try removing the `&` and `ref` parts
    |
296 -             (&Option::Some(ref x), &Option::Some(ref x1)) => x.same(x1),
296 +             (&Option::Some(ref x), Option::Some(x1)) => x.same(x1),
    |

Check warning on line 296 in src/strand.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

dereferencing a tuple pattern where every element takes a reference

warning: dereferencing a tuple pattern where every element takes a reference
   --> src/strand.rs:296:14
    |
296 |             (&Option::Some(ref x), &Option::Some(ref x1)) => x.same(x1),
    |              ^^^^^^^^^^^^^^^^^^^^
    |
    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrowed_reference
    = note: `#[warn(clippy::needless_borrowed_reference)]` on by default
help: try removing the `&` and `ref` parts
    |
296 -             (&Option::Some(ref x), &Option::Some(ref x1)) => x.same(x1),
296 +             (Option::Some(x), &Option::Some(ref x1)) => x.same(x1),
    |

Check warning on line 65 in src/sequence.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

this `impl` can be derived

warning: this `impl` can be derived
  --> src/sequence.rs:61:1
   |
61 | / impl Default for SequenceReadPairOrientation {
62 | |     fn default() -> Self {
63 | |         None
64 | |     }
65 | | }
   | |_^
   |
   = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#derivable_impls
   = help: remove the manual implementation...
help: ...and instead derive it...
   |
49 + #[derive(Default)]
50 | pub enum SequenceReadPairOrientation {
   |
help: ...and mark the default variant
   |
58 ~     #[default]
59 ~     None,
   |

Check warning on line 70 in src/annot/refids.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

this pattern creates a reference to a reference

warning: this pattern creates a reference to a reference
  --> src/annot/refids.rs:70:25
   |
70 |             if let Some(ref r) = self.refids.get(id) {
   |                         ^^^^^
   |
   = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrow
   = note: `#[warn(clippy::needless_borrow)]` on by default
help: try this
   |
70 ~             if let Some(r) = self.refids.get(id) {
71 ~                 r.clone()
   |

Check warning on line 56 in src/alignment.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

this `impl` can be derived

warning: this `impl` can be derived
  --> src/alignment.rs:52:1
   |
52 | / impl Default for AlignmentMode {
53 | |     fn default() -> Self {
54 | |         AlignmentMode::Global
55 | |     }
56 | | }
   | |_^
   |
   = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#derivable_impls
   = note: `#[warn(clippy::derivable_impls)]` on by default
   = help: remove the manual implementation...
help: ...and instead derive it...
   |
45 + #[derive(Default)]
46 | pub enum AlignmentMode {
   |
help: ...and mark the default variant
   |
48 ~     #[default]
49 ~     Global,
   |