[INFO] updating cached repository peap/git-global [INFO] running `"git" "fetch" "--all"` [INFO] [stdout] Fetching origin [INFO] [stderr] From git://github.com/peap/git-global [INFO] [stderr] * branch HEAD -> FETCH_HEAD [INFO] running `"git" "clone" "work/cache/sources/gh/peap/git-global" "work/ex/clippy-test-run/sources/stable/gh/peap/git-global"` [INFO] [stderr] Cloning into 'work/ex/clippy-test-run/sources/stable/gh/peap/git-global'... [INFO] [stderr] done. [INFO] running `"git" "clone" "work/cache/sources/gh/peap/git-global" "work/ex/clippy-test-run/sources/stable+rustflags=-Dclippy%3A%3Ainto_iter_on_array/gh/peap/git-global"` [INFO] [stderr] Cloning into 'work/ex/clippy-test-run/sources/stable+rustflags=-Dclippy%3A%3Ainto_iter_on_array/gh/peap/git-global'... [INFO] [stderr] done. [INFO] running `"git" "rev-parse" "HEAD"` [INFO] [stdout] d3aaabc30c24fa08ec4a89d9e144375d52b04534 [INFO] sha for GitHub repo peap/git-global: d3aaabc30c24fa08ec4a89d9e144375d52b04534 [INFO] validating manifest of peap/git-global on toolchain stable [INFO] running `"/mnt/big/crater/work/local/cargo-home/bin/cargo" "+stable" "read-manifest" "--manifest-path" "Cargo.toml"` [INFO] validating manifest of peap/git-global on toolchain stable+rustflags=-Dclippy::into_iter_on_array [INFO] running `"/mnt/big/crater/work/local/cargo-home/bin/cargo" "+stable" "read-manifest" "--manifest-path" "Cargo.toml"` [INFO] started frobbing peap/git-global [INFO] finished frobbing peap/git-global [INFO] frobbed toml for peap/git-global written to work/ex/clippy-test-run/sources/stable/gh/peap/git-global/Cargo.toml [INFO] started frobbing peap/git-global [INFO] finished frobbing peap/git-global [INFO] frobbed toml for peap/git-global written to work/ex/clippy-test-run/sources/stable+rustflags=-Dclippy%3A%3Ainto_iter_on_array/gh/peap/git-global/Cargo.toml [INFO] crate peap/git-global has a lockfile. skipping [INFO] running `"/mnt/big/crater/work/local/cargo-home/bin/cargo" "+stable" "fetch" "--locked" "--manifest-path" "Cargo.toml"` [INFO] running `"/mnt/big/crater/work/local/cargo-home/bin/cargo" "+stable" "fetch" "--locked" "--manifest-path" "Cargo.toml"` [INFO] linting peap/git-global against stable for clippy-test-run [INFO] running `"docker" "create" "-v" "/mnt/big/crater/work/local/target-dirs/clippy-test-run/worker-5/stable:/opt/crater/target:rw,Z" "-v" "/mnt/big/crater/work/ex/clippy-test-run/sources/stable/gh/peap/git-global:/opt/crater/workdir:ro,Z" "-v" "/mnt/big/crater/work/local/cargo-home:/opt/crater/cargo-home:ro,Z" "-v" "/mnt/big/crater/work/local/rustup-home:/opt/crater/rustup-home:ro,Z" "-e" "USER_ID=1000" "-e" "SOURCE_DIR=/opt/crater/workdir" "-e" "MAP_USER_ID=1000" "-e" "CARGO_TARGET_DIR=/opt/crater/target" "-e" "CARGO_INCREMENTAL=0" "-e" "RUST_BACKTRACE=full" "-e" "RUSTFLAGS=--cap-lints=forbid" "-e" "CARGO_HOME=/opt/crater/cargo-home" "-e" "RUSTUP_HOME=/opt/crater/rustup-home" "-w" "/opt/crater/workdir" "-m" "1536M" "--network" "none" "rustops/crates-build-env" "/opt/crater/cargo-home/bin/cargo" "+stable" "clippy" "--frozen" "--all" "--all-targets"` [INFO] [stdout] 6508710c40aafd16a44806643441f22ec6da033adc6645daf17d308f74bc36da [INFO] running `"docker" "start" "-a" "6508710c40aafd16a44806643441f22ec6da033adc6645daf17d308f74bc36da"` [INFO] [stderr] Checking bitflags v0.8.0 [INFO] [stderr] Checking vec_map v0.7.0 [INFO] [stderr] Checking json v0.11.6 [INFO] [stderr] Compiling cmake v0.1.21 [INFO] [stderr] Compiling libz-sys v1.0.13 [INFO] [stderr] Checking atty v0.2.2 [INFO] [stderr] Checking term_size v0.2.3 [INFO] [stderr] Checking app_dirs v1.1.1 [INFO] [stderr] Checking idna v0.1.0 [INFO] [stderr] Checking tempdir v0.3.5 [INFO] [stderr] Checking clap v2.21.1 [INFO] [stderr] Compiling libgit2-sys v0.6.7 [INFO] [stderr] Checking url v1.4.0 [INFO] [stderr] Checking git2 v0.6.4 [INFO] [stderr] Checking git-global v0.1.1 (/opt/crater/workdir) [INFO] [stderr] warning: redundant field names in struct initialization [INFO] [stderr] --> src/core.rs:31:13 [INFO] [stderr] | [INFO] [stderr] 31 | path: path, [INFO] [stderr] | ^^^^^^^^^^ help: replace it with: `path` [INFO] [stderr] | [INFO] [stderr] = note: #[warn(clippy::redundant_field_names)] on by default [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_field_names [INFO] [stderr] [INFO] [stderr] warning: redundant field names in struct initialization [INFO] [stderr] --> src/core.rs:71:13 [INFO] [stderr] | [INFO] [stderr] 71 | repo_messages: repo_messages, [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: replace it with: `repo_messages` [INFO] [stderr] | [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_field_names [INFO] [stderr] [INFO] [stderr] warning: redundant field names in struct initialization [INFO] [stderr] --> src/core.rs:175:13 [INFO] [stderr] | [INFO] [stderr] 175 | basedir: basedir, [INFO] [stderr] | ^^^^^^^^^^^^^^^^ help: replace it with: `basedir` [INFO] [stderr] | [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_field_names [INFO] [stderr] [INFO] [stderr] warning: redundant field names in struct initialization [INFO] [stderr] --> src/core.rs:177:13 [INFO] [stderr] | [INFO] [stderr] 177 | cache_file: cache_file, [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^ help: replace it with: `cache_file` [INFO] [stderr] | [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_field_names [INFO] [stderr] [INFO] [stderr] warning: redundant field names in struct initialization [INFO] [stderr] --> src/core.rs:31:13 [INFO] [stderr] | [INFO] [stderr] 31 | path: path, [INFO] [stderr] | ^^^^^^^^^^ help: replace it with: `path` [INFO] [stderr] | [INFO] [stderr] = note: #[warn(clippy::redundant_field_names)] on by default [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_field_names [INFO] [stderr] [INFO] [stderr] warning: redundant field names in struct initialization [INFO] [stderr] --> src/core.rs:71:13 [INFO] [stderr] | [INFO] [stderr] 71 | repo_messages: repo_messages, [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: replace it with: `repo_messages` [INFO] [stderr] | [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_field_names [INFO] [stderr] [INFO] [stderr] warning: redundant field names in struct initialization [INFO] [stderr] --> src/core.rs:175:13 [INFO] [stderr] | [INFO] [stderr] 175 | basedir: basedir, [INFO] [stderr] | ^^^^^^^^^^^^^^^^ help: replace it with: `basedir` [INFO] [stderr] | [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_field_names [INFO] [stderr] [INFO] [stderr] warning: redundant field names in struct initialization [INFO] [stderr] --> src/core.rs:177:13 [INFO] [stderr] | [INFO] [stderr] 177 | cache_file: cache_file, [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^ help: replace it with: `cache_file` [INFO] [stderr] | [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_field_names [INFO] [stderr] [INFO] [stderr] warning: Constants have by default a `'static` lifetime [INFO] [stderr] --> src/core.rs:18:20 [INFO] [stderr] | [INFO] [stderr] 18 | const CACHE_FILE: &'static str = "repos.txt"; [INFO] [stderr] | -^^^^^^^---- help: consider removing `'static`: `&str` [INFO] [stderr] | [INFO] [stderr] = note: #[warn(clippy::const_static_lifetime)] on by default [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#const_static_lifetime [INFO] [stderr] [INFO] [stderr] warning: Constants have by default a `'static` lifetime [INFO] [stderr] --> src/core.rs:19:25 [INFO] [stderr] | [INFO] [stderr] 19 | const SETTING_BASEDIR: &'static str = "global.basedir"; [INFO] [stderr] | -^^^^^^^---- help: consider removing `'static`: `&str` [INFO] [stderr] | [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#const_static_lifetime [INFO] [stderr] [INFO] [stderr] warning: Constants have by default a `'static` lifetime [INFO] [stderr] --> src/core.rs:20:25 [INFO] [stderr] | [INFO] [stderr] 20 | const SETTING_IGNORED: &'static str = "global.ignore"; [INFO] [stderr] | -^^^^^^^---- help: consider removing `'static`: `&str` [INFO] [stderr] | [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#const_static_lifetime [INFO] [stderr] [INFO] [stderr] warning: Constants have by default a `'static` lifetime [INFO] [stderr] --> src/core.rs:18:20 [INFO] [stderr] | [INFO] [stderr] 18 | const CACHE_FILE: &'static str = "repos.txt"; [INFO] [stderr] | -^^^^^^^---- help: consider removing `'static`: `&str` [INFO] [stderr] | [INFO] [stderr] = note: #[warn(clippy::const_static_lifetime)] on by default [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#const_static_lifetime [INFO] [stderr] [INFO] [stderr] warning: Constants have by default a `'static` lifetime [INFO] [stderr] --> src/core.rs:19:25 [INFO] [stderr] | [INFO] [stderr] 19 | const SETTING_BASEDIR: &'static str = "global.basedir"; [INFO] [stderr] | -^^^^^^^---- help: consider removing `'static`: `&str` [INFO] [stderr] | [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#const_static_lifetime [INFO] [stderr] [INFO] [stderr] warning: Constants have by default a `'static` lifetime [INFO] [stderr] --> src/core.rs:20:25 [INFO] [stderr] | [INFO] [stderr] 20 | const SETTING_IGNORED: &'static str = "global.ignore"; [INFO] [stderr] | -^^^^^^^---- help: consider removing `'static`: `&str` [INFO] [stderr] | [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#const_static_lifetime [INFO] [stderr] [INFO] [stderr] warning: use of deprecated item 'std::env::home_dir': This function's behavior is unexpected and probably not what you want. Consider using the home_dir function from https://crates.io/crates/dirs instead. [INFO] [stderr] --> src/core.rs:151:24 [INFO] [stderr] | [INFO] [stderr] 151 | let home_dir = env::home_dir() [INFO] [stderr] | ^^^^^^^^^^^^^ [INFO] [stderr] | [INFO] [stderr] = note: #[warn(deprecated)] on by default [INFO] [stderr] [INFO] [stderr] warning: use of deprecated item 'std::env::home_dir': This function's behavior is unexpected and probably not what you want. Consider using the home_dir function from https://crates.io/crates/dirs instead. [INFO] [stderr] --> src/core.rs:151:24 [INFO] [stderr] | [INFO] [stderr] 151 | let home_dir = env::home_dir() [INFO] [stderr] | ^^^^^^^^^^^^^ [INFO] [stderr] | [INFO] [stderr] = note: #[warn(deprecated)] on by default [INFO] [stderr] [INFO] [stderr] warning: writing `&Vec<_>` instead of `&[_]` involves one more reference and cannot be used with non-Vec-based slices. [INFO] [stderr] --> src/core.rs:63:23 [INFO] [stderr] | [INFO] [stderr] 63 | pub fn new(repos: &Vec) -> GitGlobalResult { [INFO] [stderr] | ^^^^^^^^^^ [INFO] [stderr] | [INFO] [stderr] = note: #[warn(clippy::ptr_arg)] on by default [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#ptr_arg [INFO] [stderr] help: change this to [INFO] [stderr] | [INFO] [stderr] 63 | pub fn new(repos: &[Repo]) -> GitGlobalResult { [INFO] [stderr] | ^^^^^^^ [INFO] [stderr] help: change `repos.clone()` to [INFO] [stderr] | [INFO] [stderr] 70 | repos: repos.to_owned(), [INFO] [stderr] | ^^^^^^^^^^^^^^^^ [INFO] [stderr] [INFO] [stderr] warning: you seem to be trying to use match for destructuring a single pattern. Consider using `if let` [INFO] [stderr] --> src/core.rs:91:9 [INFO] [stderr] | [INFO] [stderr] 91 | / match self.repo_messages.get_mut(&repo) { [INFO] [stderr] 92 | | Some(item) => item.push(data_line), [INFO] [stderr] 93 | | None => (), [INFO] [stderr] 94 | | } [INFO] [stderr] | |_________^ help: try this: `if let Some(item) = self.repo_messages.get_mut(&repo) { item.push(data_line) }` [INFO] [stderr] | [INFO] [stderr] = note: #[warn(clippy::single_match)] on by default [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#single_match [INFO] [stderr] [INFO] [stderr] warning: called `.get().unwrap()` on a HashMap. Using `[]` is more clear and more concise [INFO] [stderr] --> src/core.rs:103:28 [INFO] [stderr] | [INFO] [stderr] 103 | let messages = self.repo_messages.get(&repo).unwrap(); [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: try this: `&self.repo_messages[&repo]` [INFO] [stderr] | [INFO] [stderr] = note: #[warn(clippy::get_unwrap)] on by default [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#get_unwrap [INFO] [stderr] [INFO] [stderr] warning: length comparison to zero [INFO] [stderr] --> src/core.rs:104:16 [INFO] [stderr] | [INFO] [stderr] 104 | if messages.len() > 0 { [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^ help: using `is_empty` is clearer and more explicit: `!messages.is_empty()` [INFO] [stderr] | [INFO] [stderr] = note: #[warn(clippy::len_zero)] on by default [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#len_zero [INFO] [stderr] [INFO] [stderr] warning: length comparison to zero [INFO] [stderr] --> src/core.rs:130:16 [INFO] [stderr] | [INFO] [stderr] 130 | if messages.len() > 0 { [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^ help: using `is_empty` is clearer and more explicit: `!messages.is_empty()` [INFO] [stderr] | [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#len_zero [INFO] [stderr] [INFO] [stderr] warning: you should consider deriving a `Default` implementation for `core::GitGlobalConfig` [INFO] [stderr] --> src/core.rs:150:5 [INFO] [stderr] | [INFO] [stderr] 150 | / pub fn new() -> GitGlobalConfig { [INFO] [stderr] 151 | | let home_dir = env::home_dir() [INFO] [stderr] 152 | | .expect("Could not determine home directory.") [INFO] [stderr] 153 | | .to_str() [INFO] [stderr] ... | [INFO] [stderr] 178 | | } [INFO] [stderr] 179 | | } [INFO] [stderr] | |_____^ [INFO] [stderr] | [INFO] [stderr] = note: #[warn(clippy::new_without_default_derive)] on by default [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#new_without_default_derive [INFO] [stderr] help: try this [INFO] [stderr] | [INFO] [stderr] 143 | #[derive(Default)] [INFO] [stderr] | [INFO] [stderr] [INFO] [stderr] warning: single-character string constant used as pattern [INFO] [stderr] --> src/core.rs:161:29 [INFO] [stderr] | [INFO] [stderr] 161 | .split(",") [INFO] [stderr] | ^^^ help: try using a char instead: `','` [INFO] [stderr] | [INFO] [stderr] = note: #[warn(clippy::single_char_pattern)] on by default [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#single_char_pattern [INFO] [stderr] [INFO] [stderr] warning: use of `unwrap_or` followed by a call to `new` [INFO] [stderr] --> src/core.rs:159:18 [INFO] [stderr] | [INFO] [stderr] 159 | / config.get_string(SETTING_IGNORED) [INFO] [stderr] 160 | | .unwrap_or(String::new()) [INFO] [stderr] | |______________________________________________^ help: try this: `config.get_string(SETTING_IGNORED).unwrap_or_default()` [INFO] [stderr] | [INFO] [stderr] = note: #[warn(clippy::or_fun_call)] on by default [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#or_fun_call [INFO] [stderr] [INFO] [stderr] warning: Err(_) will match all errors, maybe not a good idea [INFO] [stderr] --> src/core.rs:172:13 [INFO] [stderr] | [INFO] [stderr] 172 | Err(_) => panic!("TODO: work without XDG"), [INFO] [stderr] | ^^^^^^ [INFO] [stderr] | [INFO] [stderr] = note: #[warn(clippy::match_wild_err_arm)] on by default [INFO] [stderr] = note: to remove this warning, match each error separately or use unreachable macro [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#match_wild_err_arm [INFO] [stderr] [INFO] [stderr] warning: this `.fold` can be written more succinctly using another method [INFO] [stderr] --> src/core.rs:185:20 [INFO] [stderr] | [INFO] [stderr] 185 | .iter() [INFO] [stderr] | ____________________^ [INFO] [stderr] 186 | | .fold(true, |acc, pattern| acc && !entry_path.contains(pattern)) [INFO] [stderr] | |____________________________________________________________________________^ help: try: `.all(|pattern| !entry_path.contains(pattern))` [INFO] [stderr] | [INFO] [stderr] = note: #[warn(clippy::unnecessary_fold)] on by default [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#unnecessary_fold [INFO] [stderr] [INFO] [stderr] warning: writing `&Vec<_>` instead of `&[_]` involves one more reference and cannot be used with non-Vec-based slices. [INFO] [stderr] --> src/core.rs:195:34 [INFO] [stderr] | [INFO] [stderr] 195 | fn cache_repos(&self, repos: &Vec) { [INFO] [stderr] | ^^^^^^^^^^ help: change this to: `&[Repo]` [INFO] [stderr] | [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#ptr_arg [INFO] [stderr] [INFO] [stderr] warning: you seem to be trying to use match for destructuring a single pattern. Consider using `if let` [INFO] [stderr] --> src/core.rs:220:17 [INFO] [stderr] | [INFO] [stderr] 220 | / match line { [INFO] [stderr] 221 | | Ok(repo_path) => repos.push(Repo::new(repo_path)), [INFO] [stderr] 222 | | Err(_) => (), // TODO: handle errors [INFO] [stderr] 223 | | } [INFO] [stderr] | |_________________^ help: try this: `if let Ok(repo_path) = line { repos.push(Repo::new(repo_path)) }` [INFO] [stderr] | [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#single_match [INFO] [stderr] [INFO] [stderr] warning: you seem to be trying to use match for destructuring a single pattern. Consider using `if let` [INFO] [stderr] --> src/core.rs:239:9 [INFO] [stderr] | [INFO] [stderr] 239 | / match entry { [INFO] [stderr] 240 | | Ok(entry) => { [INFO] [stderr] 241 | | if entry.file_type().is_dir() && entry.file_name() == ".git" { [INFO] [stderr] 242 | | let parent_path = entry.path().parent().expect("Could not determine parent."); [INFO] [stderr] ... | [INFO] [stderr] 251 | | Err(_) => (), [INFO] [stderr] 252 | | } [INFO] [stderr] | |_________^ [INFO] [stderr] | [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#single_match [INFO] [stderr] help: try this [INFO] [stderr] | [INFO] [stderr] 239 | if let Ok(entry) = entry { [INFO] [stderr] 240 | if entry.file_type().is_dir() && entry.file_name() == ".git" { [INFO] [stderr] 241 | let parent_path = entry.path().parent().expect("Could not determine parent."); [INFO] [stderr] 242 | match parent_path.to_str() { [INFO] [stderr] 243 | Some(path) => { [INFO] [stderr] 244 | repos.push(Repo::new(path.to_string())); [INFO] [stderr] ... [INFO] [stderr] [INFO] [stderr] warning: you seem to be trying to use match for destructuring a single pattern. Consider using `if let` [INFO] [stderr] --> src/core.rs:243:21 [INFO] [stderr] | [INFO] [stderr] 243 | / match parent_path.to_str() { [INFO] [stderr] 244 | | Some(path) => { [INFO] [stderr] 245 | | repos.push(Repo::new(path.to_string())); [INFO] [stderr] 246 | | } [INFO] [stderr] 247 | | None => (), [INFO] [stderr] 248 | | } [INFO] [stderr] | |_____________________^ [INFO] [stderr] | [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#single_match [INFO] [stderr] help: try this [INFO] [stderr] | [INFO] [stderr] 243 | if let Some(path) = parent_path.to_str() { [INFO] [stderr] 244 | repos.push(Repo::new(path.to_string())); [INFO] [stderr] 245 | } [INFO] [stderr] | [INFO] [stderr] [INFO] [stderr] warning: writing `&Vec<_>` instead of `&[_]` involves one more reference and cannot be used with non-Vec-based slices. [INFO] [stderr] --> src/core.rs:259:27 [INFO] [stderr] | [INFO] [stderr] 259 | pub fn cache_repos(repos: &Vec) { [INFO] [stderr] | ^^^^^^^^^^ help: change this to: `&[Repo]` [INFO] [stderr] | [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#ptr_arg [INFO] [stderr] [INFO] [stderr] warning: useless use of `format!` [INFO] [stderr] --> src/subcommands/info.rs:32:19 [INFO] [stderr] | [INFO] [stderr] 32 | let version = format!("{}", crate_version!()); [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: consider using .to_string(): `env ! ( "CARGO_PKG_VERSION" ).to_string()` [INFO] [stderr] | [INFO] [stderr] = note: #[warn(clippy::useless_format)] on by default [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#useless_format [INFO] [stderr] = note: this error originates in a macro outside of the current crate (in Nightly builds, run with -Z external-macro-backtrace for more info) [INFO] [stderr] [INFO] [stderr] warning: useless use of `format!` [INFO] [stderr] --> src/subcommands/info.rs:34:25 [INFO] [stderr] | [INFO] [stderr] 34 | let mut underline = format!("==========="); [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^ help: consider using .to_string(): `"===========".to_string()` [INFO] [stderr] | [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#useless_format [INFO] [stderr] [INFO] [stderr] warning: useless use of `format!` [INFO] [stderr] --> src/subcommands/info.rs:46:24 [INFO] [stderr] | [INFO] [stderr] 46 | result.add_message(format!("Ignored patterns:")); [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: consider using .to_string(): `"Ignored patterns:".to_string()` [INFO] [stderr] | [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#useless_format [INFO] [stderr] [INFO] [stderr] warning: useless use of `format!` [INFO] [stderr] --> src/subcommands/list.rs:14:39 [INFO] [stderr] | [INFO] [stderr] 14 | result.add_repo_message(repo, format!("")); [INFO] [stderr] | ^^^^^^^^^^^ help: consider using .to_string(): `"".to_string()` [INFO] [stderr] | [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#useless_format [INFO] [stderr] [INFO] [stderr] warning: `if _ { .. } else { .. }` is an expression [INFO] [stderr] --> src/subcommands/status.rs:24:5 [INFO] [stderr] | [INFO] [stderr] 24 | / let mut wstatus = match status { [INFO] [stderr] 25 | | s if s.contains(git2::STATUS_WT_NEW) => { [INFO] [stderr] 26 | | if istatus == ' ' { [INFO] [stderr] 27 | | istatus = '?'; [INFO] [stderr] ... | [INFO] [stderr] 39 | | wstatus = '!'; [INFO] [stderr] 40 | | } [INFO] [stderr] | |_____^ [INFO] [stderr] | [INFO] [stderr] = note: #[warn(clippy::useless_let_if_seq)] on by default [INFO] [stderr] = note: you might not need `mut` at all [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#useless_let_if_seq [INFO] [stderr] help: it is more idiomatic to write [INFO] [stderr] | [INFO] [stderr] 24 | let wstatus = if status.contains(git2::STATUS_IGNORED) { ..; '!' } else { match status { [INFO] [stderr] 25 | s if s.contains(git2::STATUS_WT_NEW) => { [INFO] [stderr] 26 | if istatus == ' ' { [INFO] [stderr] 27 | istatus = '?'; [INFO] [stderr] 28 | } [INFO] [stderr] 29 | '?' [INFO] [stderr] ... [INFO] [stderr] [INFO] [stderr] warning: use of `expect` followed by a function call [INFO] [stderr] --> src/subcommands/status.rs:62:10 [INFO] [stderr] | [INFO] [stderr] 62 | .expect(&format!("Could not get statuses for {}.", repo)); [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: try this: `unwrap_or_else(|_| panic!("Could not get statuses for {}.", repo))` [INFO] [stderr] | [INFO] [stderr] = note: #[warn(clippy::expect_fun_call)] on by default [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#expect_fun_call [INFO] [stderr] [INFO] [stderr] warning: useless use of `format!` [INFO] [stderr] --> src/subcommands/status.rs:69:13 [INFO] [stderr] | [INFO] [stderr] 69 | format!("{}", status_for_path) [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: consider using .to_string(): `status_for_path.to_string()` [INFO] [stderr] | [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#useless_format [INFO] [stderr] = note: this error originates in a macro outside of the current crate (in Nightly builds, run with -Z external-macro-backtrace for more info) [INFO] [stderr] [INFO] [stderr] warning: writing `&Vec<_>` instead of `&[_]` involves one more reference and cannot be used with non-Vec-based slices. [INFO] [stderr] --> src/core.rs:63:23 [INFO] [stderr] | [INFO] [stderr] 63 | pub fn new(repos: &Vec) -> GitGlobalResult { [INFO] [stderr] | ^^^^^^^^^^ [INFO] [stderr] | [INFO] [stderr] = note: #[warn(clippy::ptr_arg)] on by default [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#ptr_arg [INFO] [stderr] help: change this to [INFO] [stderr] | [INFO] [stderr] 63 | pub fn new(repos: &[Repo]) -> GitGlobalResult { [INFO] [stderr] | ^^^^^^^ [INFO] [stderr] help: change `repos.clone()` to [INFO] [stderr] | [INFO] [stderr] 70 | repos: repos.to_owned(), [INFO] [stderr] | ^^^^^^^^^^^^^^^^ [INFO] [stderr] [INFO] [stderr] warning: you seem to be trying to use match for destructuring a single pattern. Consider using `if let` [INFO] [stderr] --> src/core.rs:91:9 [INFO] [stderr] | [INFO] [stderr] 91 | / match self.repo_messages.get_mut(&repo) { [INFO] [stderr] 92 | | Some(item) => item.push(data_line), [INFO] [stderr] 93 | | None => (), [INFO] [stderr] 94 | | } [INFO] [stderr] | |_________^ help: try this: `if let Some(item) = self.repo_messages.get_mut(&repo) { item.push(data_line) }` [INFO] [stderr] | [INFO] [stderr] = note: #[warn(clippy::single_match)] on by default [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#single_match [INFO] [stderr] [INFO] [stderr] warning: called `.get().unwrap()` on a HashMap. Using `[]` is more clear and more concise [INFO] [stderr] --> src/core.rs:103:28 [INFO] [stderr] | [INFO] [stderr] 103 | let messages = self.repo_messages.get(&repo).unwrap(); [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: try this: `&self.repo_messages[&repo]` [INFO] [stderr] | [INFO] [stderr] = note: #[warn(clippy::get_unwrap)] on by default [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#get_unwrap [INFO] [stderr] [INFO] [stderr] warning: length comparison to zero [INFO] [stderr] --> src/core.rs:104:16 [INFO] [stderr] | [INFO] [stderr] 104 | if messages.len() > 0 { [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^ help: using `is_empty` is clearer and more explicit: `!messages.is_empty()` [INFO] [stderr] | [INFO] [stderr] = note: #[warn(clippy::len_zero)] on by default [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#len_zero [INFO] [stderr] [INFO] [stderr] warning: length comparison to zero [INFO] [stderr] --> src/core.rs:130:16 [INFO] [stderr] | [INFO] [stderr] 130 | if messages.len() > 0 { [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^ help: using `is_empty` is clearer and more explicit: `!messages.is_empty()` [INFO] [stderr] | [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#len_zero [INFO] [stderr] [INFO] [stderr] warning: you should consider deriving a `Default` implementation for `core::GitGlobalConfig` [INFO] [stderr] --> src/core.rs:150:5 [INFO] [stderr] | [INFO] [stderr] 150 | / pub fn new() -> GitGlobalConfig { [INFO] [stderr] 151 | | let home_dir = env::home_dir() [INFO] [stderr] 152 | | .expect("Could not determine home directory.") [INFO] [stderr] 153 | | .to_str() [INFO] [stderr] ... | [INFO] [stderr] 178 | | } [INFO] [stderr] 179 | | } [INFO] [stderr] | |_____^ [INFO] [stderr] | [INFO] [stderr] = note: #[warn(clippy::new_without_default_derive)] on by default [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#new_without_default_derive [INFO] [stderr] help: try this [INFO] [stderr] | [INFO] [stderr] 143 | #[derive(Default)] [INFO] [stderr] | [INFO] [stderr] [INFO] [stderr] warning: single-character string constant used as pattern [INFO] [stderr] --> src/core.rs:161:29 [INFO] [stderr] | [INFO] [stderr] 161 | .split(",") [INFO] [stderr] | ^^^ help: try using a char instead: `','` [INFO] [stderr] | [INFO] [stderr] = note: #[warn(clippy::single_char_pattern)] on by default [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#single_char_pattern [INFO] [stderr] [INFO] [stderr] warning: use of `unwrap_or` followed by a call to `new` [INFO] [stderr] --> src/core.rs:159:18 [INFO] [stderr] | [INFO] [stderr] 159 | / config.get_string(SETTING_IGNORED) [INFO] [stderr] 160 | | .unwrap_or(String::new()) [INFO] [stderr] | |______________________________________________^ help: try this: `config.get_string(SETTING_IGNORED).unwrap_or_default()` [INFO] [stderr] | [INFO] [stderr] = note: #[warn(clippy::or_fun_call)] on by default [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#or_fun_call [INFO] [stderr] [INFO] [stderr] warning: Err(_) will match all errors, maybe not a good idea [INFO] [stderr] --> src/core.rs:172:13 [INFO] [stderr] | [INFO] [stderr] 172 | Err(_) => panic!("TODO: work without XDG"), [INFO] [stderr] | ^^^^^^ [INFO] [stderr] | [INFO] [stderr] = note: #[warn(clippy::match_wild_err_arm)] on by default [INFO] [stderr] = note: to remove this warning, match each error separately or use unreachable macro [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#match_wild_err_arm [INFO] [stderr] [INFO] [stderr] warning: this `.fold` can be written more succinctly using another method [INFO] [stderr] --> src/core.rs:185:20 [INFO] [stderr] | [INFO] [stderr] 185 | .iter() [INFO] [stderr] | ____________________^ [INFO] [stderr] 186 | | .fold(true, |acc, pattern| acc && !entry_path.contains(pattern)) [INFO] [stderr] | |____________________________________________________________________________^ help: try: `.all(|pattern| !entry_path.contains(pattern))` [INFO] [stderr] | [INFO] [stderr] = note: #[warn(clippy::unnecessary_fold)] on by default [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#unnecessary_fold [INFO] [stderr] [INFO] [stderr] warning: writing `&Vec<_>` instead of `&[_]` involves one more reference and cannot be used with non-Vec-based slices. [INFO] [stderr] --> src/core.rs:195:34 [INFO] [stderr] | [INFO] [stderr] 195 | fn cache_repos(&self, repos: &Vec) { [INFO] [stderr] | ^^^^^^^^^^ help: change this to: `&[Repo]` [INFO] [stderr] | [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#ptr_arg [INFO] [stderr] [INFO] [stderr] warning: you seem to be trying to use match for destructuring a single pattern. Consider using `if let` [INFO] [stderr] --> src/core.rs:220:17 [INFO] [stderr] | [INFO] [stderr] 220 | / match line { [INFO] [stderr] 221 | | Ok(repo_path) => repos.push(Repo::new(repo_path)), [INFO] [stderr] 222 | | Err(_) => (), // TODO: handle errors [INFO] [stderr] 223 | | } [INFO] [stderr] | |_________________^ help: try this: `if let Ok(repo_path) = line { repos.push(Repo::new(repo_path)) }` [INFO] [stderr] | [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#single_match [INFO] [stderr] [INFO] [stderr] warning: you seem to be trying to use match for destructuring a single pattern. Consider using `if let` [INFO] [stderr] --> src/core.rs:239:9 [INFO] [stderr] | [INFO] [stderr] 239 | / match entry { [INFO] [stderr] 240 | | Ok(entry) => { [INFO] [stderr] 241 | | if entry.file_type().is_dir() && entry.file_name() == ".git" { [INFO] [stderr] 242 | | let parent_path = entry.path().parent().expect("Could not determine parent."); [INFO] [stderr] ... | [INFO] [stderr] 251 | | Err(_) => (), [INFO] [stderr] 252 | | } [INFO] [stderr] | |_________^ [INFO] [stderr] | [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#single_match [INFO] [stderr] help: try this [INFO] [stderr] | [INFO] [stderr] 239 | if let Ok(entry) = entry { [INFO] [stderr] 240 | if entry.file_type().is_dir() && entry.file_name() == ".git" { [INFO] [stderr] 241 | let parent_path = entry.path().parent().expect("Could not determine parent."); [INFO] [stderr] 242 | match parent_path.to_str() { [INFO] [stderr] 243 | Some(path) => { [INFO] [stderr] 244 | repos.push(Repo::new(path.to_string())); [INFO] [stderr] ... [INFO] [stderr] [INFO] [stderr] warning: you seem to be trying to use match for destructuring a single pattern. Consider using `if let` [INFO] [stderr] --> src/core.rs:243:21 [INFO] [stderr] | [INFO] [stderr] 243 | / match parent_path.to_str() { [INFO] [stderr] 244 | | Some(path) => { [INFO] [stderr] 245 | | repos.push(Repo::new(path.to_string())); [INFO] [stderr] 246 | | } [INFO] [stderr] 247 | | None => (), [INFO] [stderr] 248 | | } [INFO] [stderr] | |_____________________^ [INFO] [stderr] | [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#single_match [INFO] [stderr] help: try this [INFO] [stderr] | [INFO] [stderr] 243 | if let Some(path) = parent_path.to_str() { [INFO] [stderr] 244 | repos.push(Repo::new(path.to_string())); [INFO] [stderr] 245 | } [INFO] [stderr] | [INFO] [stderr] [INFO] [stderr] warning: writing `&Vec<_>` instead of `&[_]` involves one more reference and cannot be used with non-Vec-based slices. [INFO] [stderr] --> src/core.rs:259:27 [INFO] [stderr] | [INFO] [stderr] 259 | pub fn cache_repos(repos: &Vec) { [INFO] [stderr] | ^^^^^^^^^^ help: change this to: `&[Repo]` [INFO] [stderr] | [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#ptr_arg [INFO] [stderr] [INFO] [stderr] warning: useless use of `format!` [INFO] [stderr] --> src/subcommands/info.rs:32:19 [INFO] [stderr] | [INFO] [stderr] 32 | let version = format!("{}", crate_version!()); [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: consider using .to_string(): `env ! ( "CARGO_PKG_VERSION" ).to_string()` [INFO] [stderr] | [INFO] [stderr] = note: #[warn(clippy::useless_format)] on by default [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#useless_format [INFO] [stderr] = note: this error originates in a macro outside of the current crate (in Nightly builds, run with -Z external-macro-backtrace for more info) [INFO] [stderr] [INFO] [stderr] warning: useless use of `format!` [INFO] [stderr] --> src/subcommands/info.rs:34:25 [INFO] [stderr] | [INFO] [stderr] 34 | let mut underline = format!("==========="); [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^ help: consider using .to_string(): `"===========".to_string()` [INFO] [stderr] | [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#useless_format [INFO] [stderr] [INFO] [stderr] warning: useless use of `format!` [INFO] [stderr] --> src/subcommands/info.rs:46:24 [INFO] [stderr] | [INFO] [stderr] 46 | result.add_message(format!("Ignored patterns:")); [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: consider using .to_string(): `"Ignored patterns:".to_string()` [INFO] [stderr] | [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#useless_format [INFO] [stderr] [INFO] [stderr] warning: useless use of `format!` [INFO] [stderr] --> src/subcommands/list.rs:14:39 [INFO] [stderr] | [INFO] [stderr] 14 | result.add_repo_message(repo, format!("")); [INFO] [stderr] | ^^^^^^^^^^^ help: consider using .to_string(): `"".to_string()` [INFO] [stderr] | [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#useless_format [INFO] [stderr] [INFO] [stderr] warning: `if _ { .. } else { .. }` is an expression [INFO] [stderr] --> src/subcommands/status.rs:24:5 [INFO] [stderr] | [INFO] [stderr] 24 | / let mut wstatus = match status { [INFO] [stderr] 25 | | s if s.contains(git2::STATUS_WT_NEW) => { [INFO] [stderr] 26 | | if istatus == ' ' { [INFO] [stderr] 27 | | istatus = '?'; [INFO] [stderr] ... | [INFO] [stderr] 39 | | wstatus = '!'; [INFO] [stderr] 40 | | } [INFO] [stderr] | |_____^ [INFO] [stderr] | [INFO] [stderr] = note: #[warn(clippy::useless_let_if_seq)] on by default [INFO] [stderr] = note: you might not need `mut` at all [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#useless_let_if_seq [INFO] [stderr] help: it is more idiomatic to write [INFO] [stderr] | [INFO] [stderr] 24 | let wstatus = if status.contains(git2::STATUS_IGNORED) { ..; '!' } else { match status { [INFO] [stderr] 25 | s if s.contains(git2::STATUS_WT_NEW) => { [INFO] [stderr] 26 | if istatus == ' ' { [INFO] [stderr] 27 | istatus = '?'; [INFO] [stderr] 28 | } [INFO] [stderr] 29 | '?' [INFO] [stderr] ... [INFO] [stderr] [INFO] [stderr] warning: use of `expect` followed by a function call [INFO] [stderr] --> src/subcommands/status.rs:62:10 [INFO] [stderr] | [INFO] [stderr] 62 | .expect(&format!("Could not get statuses for {}.", repo)); [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: try this: `unwrap_or_else(|_| panic!("Could not get statuses for {}.", repo))` [INFO] [stderr] | [INFO] [stderr] = note: #[warn(clippy::expect_fun_call)] on by default [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#expect_fun_call [INFO] [stderr] [INFO] [stderr] warning: useless use of `format!` [INFO] [stderr] --> src/subcommands/status.rs:69:13 [INFO] [stderr] | [INFO] [stderr] 69 | format!("{}", status_for_path) [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: consider using .to_string(): `status_for_path.to_string()` [INFO] [stderr] | [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#useless_format [INFO] [stderr] = note: this error originates in a macro outside of the current crate (in Nightly builds, run with -Z external-macro-backtrace for more info) [INFO] [stderr] [INFO] [stderr] warning: unneeded unit return type [INFO] [stderr] --> tests/utils/mod.rs:10:35 [INFO] [stderr] | [INFO] [stderr] 10 | pub fn with_temp_repo(test: T) -> () [INFO] [stderr] | ^^^^^ help: remove the `-> ()` [INFO] [stderr] | [INFO] [stderr] = note: #[warn(clippy::unused_unit)] on by default [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#unused_unit [INFO] [stderr] [INFO] [stderr] warning: unneeded unit return type [INFO] [stderr] --> tests/utils/mod.rs:23:49 [INFO] [stderr] | [INFO] [stderr] 23 | pub fn with_base_dir_of_three_repos(test: T) -> () [INFO] [stderr] | ^^^^^ help: remove the `-> ()` [INFO] [stderr] | [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#unused_unit [INFO] [stderr] [INFO] [stderr] warning: unused variable: `path` [INFO] [stderr] --> tests/subcommands.rs:28:46 [INFO] [stderr] | [INFO] [stderr] 28 | utils::with_base_dir_of_three_repos(|ref path| { [INFO] [stderr] | ^^^^ help: consider using `_path` instead [INFO] [stderr] | [INFO] [stderr] = note: #[warn(unused_variables)] on by default [INFO] [stderr] [INFO] [stderr] warning: unneeded unit return type [INFO] [stderr] --> tests/utils/mod.rs:10:35 [INFO] [stderr] | [INFO] [stderr] 10 | pub fn with_temp_repo(test: T) -> () [INFO] [stderr] | ^^^^^ help: remove the `-> ()` [INFO] [stderr] | [INFO] [stderr] = note: #[warn(clippy::unused_unit)] on by default [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#unused_unit [INFO] [stderr] [INFO] [stderr] warning: unneeded unit return type [INFO] [stderr] --> tests/utils/mod.rs:23:49 [INFO] [stderr] | [INFO] [stderr] 23 | pub fn with_base_dir_of_three_repos(test: T) -> () [INFO] [stderr] | ^^^^^ help: remove the `-> ()` [INFO] [stderr] | [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#unused_unit [INFO] [stderr] [INFO] [stderr] Finished dev [unoptimized + debuginfo] target(s) in 34.63s [INFO] running `"docker" "inspect" "6508710c40aafd16a44806643441f22ec6da033adc6645daf17d308f74bc36da"` [INFO] running `"docker" "rm" "-f" "6508710c40aafd16a44806643441f22ec6da033adc6645daf17d308f74bc36da"` [INFO] [stdout] 6508710c40aafd16a44806643441f22ec6da033adc6645daf17d308f74bc36da