Update Rust crate clap to 4.5.3 #438
clippy
17 warnings
Details
Results
Message level | Amount |
---|---|
Internal compiler error | 0 |
Error | 0 |
Warning | 17 |
Note | 0 |
Help | 0 |
Versions
- rustc 1.76.0 (07dca489a 2024-02-04)
- cargo 1.76.0 (c84b36747 2024-01-18)
- clippy 0.1.76 (07dca48 2024-02-04)
Annotations
Check warning on line 16 in src/main.rs
github-actions / clippy
the `Err`-variant returned from this function is very large
warning: the `Err`-variant returned from this function is very large
--> src/main.rs:16:17
|
16 | fn execute() -> Result<()> {
| ^^^^^^^^^^ the `Err`-variant is at least 128 bytes
|
= help: try reducing the size of `dsc::error::Error`, for example by boxing large elements or replacing it with `Box<dsc::error::Error>`
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#result_large_err
= note: `#[warn(clippy::result_large_err)]` on by default
Check warning on line 22 in src/util/pass.rs
github-actions / clippy
this expression borrows a value the compiler would automatically borrow
warning: this expression borrows a value the compiler would automatically borrow
--> src/util/pass.rs:22:8
|
22 | Ok((&line[len..]).trim().into())
| ^^^^^^^^^^^^^^ help: change this to: `line[len..]`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrow
Check warning on line 211 in src/http/session.rs
github-actions / clippy
the borrowed expression implements the required traits
warning: the borrowed expression implements the required traits
--> src/http/session.rs:211:34
|
211 | std::fs::write(path, &cnt).context(StoreSessionFileSnafu { path })
| ^^^^ help: change this to: `cnt`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrows_for_generic_args
Check warning on line 197 in src/http/session.rs
github-actions / clippy
the borrowed expression implements the required traits
warning: the borrowed expression implements the required traits
--> src/http/session.rs:197:43
|
197 | let cnt = std::fs::read_to_string(&path).context(ReadSessionFileSnafu { path })?;
| ^^^^^ help: change this to: `path`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrows_for_generic_args
Check warning on line 210 in src/config.rs
github-actions / clippy
the borrowed expression implements the required traits
warning: the borrowed expression implements the required traits
--> src/config.rs:210:31
|
210 | std::fs::write(&file, &c).map_err(|e| ConfigError::CreateDefault {
| ^^ help: change this to: `c`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrows_for_generic_args
Check warning on line 210 in src/config.rs
github-actions / clippy
the borrowed expression implements the required traits
warning: the borrowed expression implements the required traits
--> src/config.rs:210:24
|
210 | std::fs::write(&file, &c).map_err(|e| ConfigError::CreateDefault {
| ^^^^^ help: change this to: `file`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrows_for_generic_args
Check warning on line 342 in src/cli/opts.rs
github-actions / clippy
this expression creates a reference which is immediately dereferenced by the compiler
warning: this expression creates a reference which is immediately dereferenced by the compiler
--> src/cli/opts.rs:342:46
|
342 | let np = Self::read_name_val(&basic_file)?;
| ^^^^^^^^^^^ help: change this to: `basic_file`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrow
Check warning on line 337 in src/cli/opts.rs
github-actions / clippy
this expression creates a reference which is immediately dereferenced by the compiler
warning: this expression creates a reference which is immediately dereferenced by the compiler
--> src/cli/opts.rs:337:46
|
337 | let np = Self::read_name_val(&header_file)?;
| ^^^^^^^^^^^^ help: change this to: `header_file`
|
= 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
Check warning on line 299 in src/cli/opts.rs
github-actions / clippy
single-character string constant used as pattern
warning: single-character string constant used as pattern
--> src/cli/opts.rs:299:40
|
299 | .filter(|s| !s.starts_with("#"))
| ^^^ help: try using a `char` instead: `'#'`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#single_char_pattern
= note: `#[warn(clippy::single_char_pattern)]` on by default
Check warning on line 52 in src/cli/cmd.rs
github-actions / clippy
the following explicit lifetimes could be elided: 'a
warning: the following explicit lifetimes could be elided: 'a
--> src/cli/cmd.rs:52:13
|
52 | fn exec<'a>(&self, args: &'a Context) -> Result<(), Self::CmdError>;
| ^^ ^^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_lifetimes
= note: `#[warn(clippy::needless_lifetimes)]` on by default
help: elide the lifetimes
|
52 - fn exec<'a>(&self, args: &'a Context) -> Result<(), Self::CmdError>;
52 + fn exec(&self, args: &Context) -> Result<(), Self::CmdError>;
|
Check warning on line 102 in src/cli/cmd/view.rs
github-actions / clippy
accessing first element with `ctx.cfg.pdf_viewer.get(0)`
warning: accessing first element with `ctx.cfg.pdf_viewer.get(0)`
--> src/cli/cmd/view.rs:102:25
|
102 | let tool = &ctx.cfg.pdf_viewer.get(0).ok_or(Error::NoPdfViewer)?;
| ^^^^^^^^^^^^^^^^^^^^^^^^^ help: try: `ctx.cfg.pdf_viewer.first()`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#get_first
= note: `#[warn(clippy::get_first)]` on by default
Check warning on line 286 in src/cli/cmd/export.rs
github-actions / clippy
the borrowed expression implements the required traits
warning: the borrowed expression implements the required traits
--> src/cli/cmd/export.rs:286:62
|
286 | let rel_link_target = pathdiff::diff_paths(&link_target, &link_name_path).unwrap();
| ^^^^^^^^^^^^^^^ help: change this to: `link_name_path`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrows_for_generic_args
Check warning on line 286 in src/cli/cmd/export.rs
github-actions / clippy
the borrowed expression implements the required traits
warning: the borrowed expression implements the required traits
--> src/cli/cmd/export.rs:286:48
|
286 | let rel_link_target = pathdiff::diff_paths(&link_target, &link_name_path).unwrap();
| ^^^^^^^^^^^^ help: change this to: `link_target`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrows_for_generic_args
Check warning on line 279 in src/cli/cmd/export.rs
github-actions / clippy
the borrowed expression implements the required traits
warning: the borrowed expression implements the required traits
--> src/cli/cmd/export.rs:279:33
|
279 | std::fs::create_dir_all(&link_name_path).context(CreateFileSnafu)?;
| ^^^^^^^^^^^^^^^ help: change this to: `link_name_path`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrows_for_generic_args
Check warning on line 230 in src/cli/cmd/export.rs
github-actions / clippy
the borrowed expression implements the required traits
warning: the borrowed expression implements the required traits
--> src/cli/cmd/export.rs:230:33
|
230 | std::fs::create_dir_all(&item_dir).context(CreateFileSnafu)?;
| ^^^^^^^^^ help: change this to: `item_dir`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrows_for_generic_args
Check warning on line 24 in src/cli/cmd/export.rs
github-actions / clippy
this `impl` can be derived
warning: this `impl` can be derived
--> src/cli/cmd/export.rs:20:1
|
20 | / impl Default for LinkNaming {
21 | | fn default() -> Self {
22 | | LinkNaming::Id
23 | | }
24 | | }
| |_^
|
= 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...
|
14 + #[derive(Default)]
15 | pub enum LinkNaming {
|
help: ...and mark the default variant
|
16 ~ #[default]
17 ~ Id,
|
Check warning on line 142 in src/cli/cmd/download.rs
github-actions / clippy
the borrowed expression implements the required traits
warning: the borrowed expression implements the required traits
--> src/cli/cmd/download.rs:142:53
|
142 | ... std::fs::create_dir_all(&parent).context(CreateFileSnafu)?;
| ^^^^^^^ help: change this to: `parent`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrows_for_generic_args
= note: `#[warn(clippy::needless_borrows_for_generic_args)]` on by default