[INFO] crate indexmap 1.0.2 is already in cache [INFO] extracting crate indexmap 1.0.2 into work/ex/clippy-test-run/sources/stable/reg/indexmap/1.0.2 [INFO] extracting crate indexmap 1.0.2 into work/ex/clippy-test-run/sources/stable+rustflags=-Dclippy%3A%3Ainto_iter_on_array/reg/indexmap/1.0.2 [INFO] validating manifest of indexmap-1.0.2 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 indexmap-1.0.2 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 indexmap-1.0.2 [INFO] finished frobbing indexmap-1.0.2 [INFO] frobbed toml for indexmap-1.0.2 written to work/ex/clippy-test-run/sources/stable/reg/indexmap/1.0.2/Cargo.toml [INFO] started frobbing indexmap-1.0.2 [INFO] finished frobbing indexmap-1.0.2 [INFO] frobbed toml for indexmap-1.0.2 written to work/ex/clippy-test-run/sources/stable+rustflags=-Dclippy%3A%3Ainto_iter_on_array/reg/indexmap/1.0.2/Cargo.toml [INFO] running `"/mnt/big/crater/work/local/cargo-home/bin/cargo" "+stable" "generate-lockfile" "--manifest-path" "Cargo.toml" "-Zno-index-update"` [INFO] running `"/mnt/big/crater/work/local/cargo-home/bin/cargo" "+stable" "generate-lockfile" "--manifest-path" "Cargo.toml" "-Zno-index-update"` [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 indexmap-1.0.2 against stable for clippy-test-run [INFO] running `"docker" "create" "-v" "/mnt/big/crater/work/local/target-dirs/clippy-test-run/worker-0/stable:/opt/crater/target:rw,Z" "-v" "/mnt/big/crater/work/ex/clippy-test-run/sources/stable/reg/indexmap/1.0.2:/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] 9a02bdba6ab37d99fe8c75b737dfe43de38f5611f65938b8cff8306e96032f04 [INFO] running `"docker" "start" "-a" "9a02bdba6ab37d99fe8c75b737dfe43de38f5611f65938b8cff8306e96032f04"` [INFO] [stderr] Checking indexmap v1.0.2 (/opt/crater/workdir) [INFO] [stderr] Checking quickcheck v0.6.2 [INFO] [stderr] warning: redundant field names in struct initialization [INFO] [stderr] --> src/set.rs:202:13 [INFO] [stderr] | [INFO] [stderr] 202 | other: other, [INFO] [stderr] | ^^^^^^^^^^^^ help: replace it with: `other` [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/set.rs:228:13 [INFO] [stderr] | [INFO] [stderr] 228 | other: other, [INFO] [stderr] | ^^^^^^^^^^^^ help: replace it with: `other` [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/map.rs:385:17 [INFO] [stderr] | [INFO] [stderr] 385 | hash_builder: hash_builder, [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^^^^^ help: replace it with: `hash_builder` [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/map.rs:396:17 [INFO] [stderr] | [INFO] [stderr] 396 | hash_builder: hash_builder, [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^^^^^ help: replace it with: `hash_builder` [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/map.rs:653:72 [INFO] [stderr] | [INFO] [stderr] 653 | self.map.entries.push(Bucket { hash: self.hash, key: self.key, value: value }); [INFO] [stderr] | ^^^^^^^^^^^^ help: replace it with: `value` [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/map.rs:1245:36 [INFO] [stderr] | [INFO] [stderr] 1245 | self.entries.push(Bucket { hash: hash, key: key, value: value }); [INFO] [stderr] | ^^^^^^^^^^ help: replace it with: `hash` [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/map.rs:1245:48 [INFO] [stderr] | [INFO] [stderr] 1245 | self.entries.push(Bucket { hash: hash, key: key, value: value }); [INFO] [stderr] | ^^^^^^^^ help: replace it with: `key` [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/map.rs:1245:58 [INFO] [stderr] | [INFO] [stderr] 1245 | self.entries.push(Bucket { hash: hash, key: key, value: value }); [INFO] [stderr] | ^^^^^^^^^^^^ help: replace it with: `value` [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: this `else { if .. }` block can be collapsed [INFO] [stderr] --> src/map.rs:166:16 [INFO] [stderr] | [INFO] [stderr] 166 | } else { [INFO] [stderr] | ________________^ [INFO] [stderr] 167 | | if !self.is_none() { [INFO] [stderr] 168 | | let (i, hash) = split_lo_hi(self.index); [INFO] [stderr] 169 | | Some((i as usize, ShortHashProxy::new(hash as usize))) [INFO] [stderr] ... | [INFO] [stderr] 172 | | } [INFO] [stderr] 173 | | } [INFO] [stderr] | |_________^ [INFO] [stderr] | [INFO] [stderr] = note: #[warn(clippy::collapsible_if)] on by default [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#collapsible_if [INFO] [stderr] help: try [INFO] [stderr] | [INFO] [stderr] 166 | } else if !self.is_none() { [INFO] [stderr] 167 | let (i, hash) = split_lo_hi(self.index); [INFO] [stderr] 168 | Some((i as usize, ShortHashProxy::new(hash as usize))) [INFO] [stderr] 169 | } else { [INFO] [stderr] 170 | None [INFO] [stderr] 171 | } [INFO] [stderr] | [INFO] [stderr] [INFO] [stderr] warning: You are using an explicit closure for cloning elements [INFO] [stderr] --> src/set.rs:536:20 [INFO] [stderr] | [INFO] [stderr] 536 | let iter = iterable.into_iter().map(|&x| x); [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: Consider calling the dedicated `cloned` method: `iterable.into_iter().cloned()` [INFO] [stderr] | [INFO] [stderr] = note: #[warn(clippy::map_clone)] on by default [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#map_clone [INFO] [stderr] [INFO] [stderr] warning: this argument is passed by reference, but would be more efficient if passed by value [INFO] [stderr] --> src/map.rs:116:16 [INFO] [stderr] | [INFO] [stderr] 116 | fn is_none(&self) -> bool { self.index == !0 } [INFO] [stderr] | ^^^^^ help: consider passing by value instead: `self` [INFO] [stderr] | [INFO] [stderr] = note: #[warn(clippy::trivially_copy_pass_by_ref)] on by default [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#trivially_copy_pass_by_ref [INFO] [stderr] [INFO] [stderr] warning: this argument is passed by reference, but would be more efficient if passed by value [INFO] [stderr] --> src/map.rs:121:12 [INFO] [stderr] | [INFO] [stderr] 121 | fn pos(&self) -> Option { [INFO] [stderr] | ^^^^^ help: consider passing by value instead: `self` [INFO] [stderr] | [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#trivially_copy_pass_by_ref [INFO] [stderr] [INFO] [stderr] warning: this argument is passed by reference, but would be more efficient if passed by value [INFO] [stderr] --> src/map.rs:157:20 [INFO] [stderr] | [INFO] [stderr] 157 | fn resolve(&self) -> Option<(usize, ShortHashProxy)> [INFO] [stderr] | ^^^^^ help: consider passing by value instead: `self` [INFO] [stderr] | [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#trivially_copy_pass_by_ref [INFO] [stderr] [INFO] [stderr] warning: this argument is passed by reference, but would be more efficient if passed by value [INFO] [stderr] --> src/map.rs:178:35 [INFO] [stderr] | [INFO] [stderr] 178 | fn resolve_existing_index(&self) -> usize [INFO] [stderr] | ^^^^^ help: consider passing by value instead: `self` [INFO] [stderr] | [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#trivially_copy_pass_by_ref [INFO] [stderr] [INFO] [stderr] warning: length comparison to zero [INFO] [stderr] --> src/map.rs:881:12 [INFO] [stderr] | [INFO] [stderr] 881 | if self.len() == 0 { return None; } [INFO] [stderr] | ^^^^^^^^^^^^^^^ help: using `is_empty` is clearer and more explicit: `self.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: redundant pattern matching, consider using `is_some()` [INFO] [stderr] --> src/map.rs:1140:24 [INFO] [stderr] | [INFO] [stderr] 1140 | if let Some(_) = self.indices[probe].resolve::() { [INFO] [stderr] | _________________- ^^^^^^^ [INFO] [stderr] 1141 | | /* nothing */ [INFO] [stderr] 1142 | | } else { [INFO] [stderr] 1143 | | // empty bucket, insert here [INFO] [stderr] 1144 | | self.indices[probe] = Pos::with_hash::(i, entry_hash); [INFO] [stderr] 1145 | | return; [INFO] [stderr] 1146 | | } [INFO] [stderr] | |_________________- help: try this: `if self.indices[probe].resolve::().is_some()` [INFO] [stderr] | [INFO] [stderr] = note: #[warn(clippy::redundant_pattern_matching)] on by default [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_pattern_matching [INFO] [stderr] [INFO] [stderr] warning: redundant field names in struct initialization [INFO] [stderr] --> src/set.rs:202:13 [INFO] [stderr] | [INFO] [stderr] 202 | other: other, [INFO] [stderr] | ^^^^^^^^^^^^ help: replace it with: `other` [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/set.rs:228:13 [INFO] [stderr] | [INFO] [stderr] 228 | other: other, [INFO] [stderr] | ^^^^^^^^^^^^ help: replace it with: `other` [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/map.rs:385:17 [INFO] [stderr] | [INFO] [stderr] 385 | hash_builder: hash_builder, [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^^^^^ help: replace it with: `hash_builder` [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/map.rs:396:17 [INFO] [stderr] | [INFO] [stderr] 396 | hash_builder: hash_builder, [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^^^^^ help: replace it with: `hash_builder` [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/map.rs:653:72 [INFO] [stderr] | [INFO] [stderr] 653 | self.map.entries.push(Bucket { hash: self.hash, key: self.key, value: value }); [INFO] [stderr] | ^^^^^^^^^^^^ help: replace it with: `value` [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/map.rs:1245:36 [INFO] [stderr] | [INFO] [stderr] 1245 | self.entries.push(Bucket { hash: hash, key: key, value: value }); [INFO] [stderr] | ^^^^^^^^^^ help: replace it with: `hash` [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/map.rs:1245:48 [INFO] [stderr] | [INFO] [stderr] 1245 | self.entries.push(Bucket { hash: hash, key: key, value: value }); [INFO] [stderr] | ^^^^^^^^ help: replace it with: `key` [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/map.rs:1245:58 [INFO] [stderr] | [INFO] [stderr] 1245 | self.entries.push(Bucket { hash: hash, key: key, value: value }); [INFO] [stderr] | ^^^^^^^^^^^^ help: replace it with: `value` [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: this `else { if .. }` block can be collapsed [INFO] [stderr] --> src/map.rs:166:16 [INFO] [stderr] | [INFO] [stderr] 166 | } else { [INFO] [stderr] | ________________^ [INFO] [stderr] 167 | | if !self.is_none() { [INFO] [stderr] 168 | | let (i, hash) = split_lo_hi(self.index); [INFO] [stderr] 169 | | Some((i as usize, ShortHashProxy::new(hash as usize))) [INFO] [stderr] ... | [INFO] [stderr] 172 | | } [INFO] [stderr] 173 | | } [INFO] [stderr] | |_________^ [INFO] [stderr] | [INFO] [stderr] = note: #[warn(clippy::collapsible_if)] on by default [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#collapsible_if [INFO] [stderr] help: try [INFO] [stderr] | [INFO] [stderr] 166 | } else if !self.is_none() { [INFO] [stderr] 167 | let (i, hash) = split_lo_hi(self.index); [INFO] [stderr] 168 | Some((i as usize, ShortHashProxy::new(hash as usize))) [INFO] [stderr] 169 | } else { [INFO] [stderr] 170 | None [INFO] [stderr] 171 | } [INFO] [stderr] | [INFO] [stderr] [INFO] [stderr] error[E0554]: #![feature] may not be used on the stable release channel [INFO] [stderr] --> benches/faststring.rs:1:1 [INFO] [stderr] | [INFO] [stderr] 1 | #![feature(test)] [INFO] [stderr] | ^^^^^^^^^^^^^^^^^ [INFO] [stderr] [INFO] [stderr] error: aborting due to previous error [INFO] [stderr] [INFO] [stderr] For more information about this error, try `rustc --explain E0554`. [INFO] [stderr] error: Could not compile `indexmap`. [INFO] [stderr] warning: build failed, waiting for other jobs to finish... [INFO] [stderr] error[E0554]: #![feature] may not be used on the stable release channel [INFO] [stderr] --> benches/bench.rs:1:1 [INFO] [stderr] | [INFO] [stderr] 1 | #![feature(test)] [INFO] [stderr] | ^^^^^^^^^^^^^^^^^ [INFO] [stderr] [INFO] [stderr] error: aborting due to previous error [INFO] [stderr] [INFO] [stderr] For more information about this error, try `rustc --explain E0554`. [INFO] [stderr] error: Could not compile `indexmap`. [INFO] [stderr] warning: build failed, waiting for other jobs to finish... [INFO] [stderr] warning: Constants have by default a `'static` lifetime [INFO] [stderr] --> tests/quick.rs:329:18 [INFO] [stderr] | [INFO] [stderr] 329 | const ALPHABET: &'static [u8] = b"abcdefghijklmnopqrstuvwxyz"; [INFO] [stderr] | -^^^^^^^----- help: consider removing `'static`: `&[u8]` [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: this boolean expression can be simplified [INFO] [stderr] --> tests/equivalent_trait.rs:16:9 [INFO] [stderr] | [INFO] [stderr] 16 | / self.0 == rhs.0 && [INFO] [stderr] 17 | | self.1 == rhs.1 && [INFO] [stderr] 18 | | true [INFO] [stderr] | |____________^ help: try: `self.0 == rhs.0 && self.1 == rhs.1` [INFO] [stderr] | [INFO] [stderr] = note: #[warn(clippy::nonminimal_bool)] on by default [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#nonminimal_bool [INFO] [stderr] [INFO] [stderr] warning: use of `or_insert` followed by a function call [INFO] [stderr] --> tests/quick.rs:158:36 [INFO] [stderr] | [INFO] [stderr] 158 | b.entry(k.clone()).or_insert(v.clone()); [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^ help: try this: `or_insert_with(|| v.clone())` [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: you seem to be trying to use match for destructuring a single pattern. Consider using `if let` [INFO] [stderr] --> tests/quick.rs:165:17 [INFO] [stderr] | [INFO] [stderr] 165 | / match a.entry(k.clone()) { [INFO] [stderr] 166 | | OEntry::Occupied(ent) => { ent.remove_entry(); }, [INFO] [stderr] 167 | | _ => { } [INFO] [stderr] 168 | | } [INFO] [stderr] | |_________________^ help: try this: `if let OEntry::Occupied(ent) = a.entry(k.clone()) { ent.remove_entry(); }` [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: you seem to be trying to use match for destructuring a single pattern. Consider using `if let` [INFO] [stderr] --> tests/quick.rs:169:17 [INFO] [stderr] | [INFO] [stderr] 169 | / match b.entry(k.clone()) { [INFO] [stderr] 170 | | HEntry::Occupied(ent) => { ent.remove_entry(); }, [INFO] [stderr] 171 | | _ => { } [INFO] [stderr] 172 | | } [INFO] [stderr] | |_________________^ help: try this: `if let HEntry::Occupied(ent) = b.entry(k.clone()) { ent.remove_entry(); }` [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 are using an explicit closure for cloning elements [INFO] [stderr] --> src/set.rs:536:20 [INFO] [stderr] | [INFO] [stderr] 536 | let iter = iterable.into_iter().map(|&x| x); [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: Consider calling the dedicated `cloned` method: `iterable.into_iter().cloned()` [INFO] [stderr] | [INFO] [stderr] = note: #[warn(clippy::map_clone)] on by default [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#map_clone [INFO] [stderr] [INFO] [stderr] warning: this boolean expression can be simplified [INFO] [stderr] --> src/set.rs:892:20 [INFO] [stderr] | [INFO] [stderr] 892 | if !set.get(value).is_some() { [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^^^^ help: try: `set.get(value).is_none()` [INFO] [stderr] | [INFO] [stderr] = note: #[warn(clippy::nonminimal_bool)] on by default [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#nonminimal_bool [INFO] [stderr] [INFO] [stderr] warning: the function has a cyclomatic complexity of 29 [INFO] [stderr] --> src/set.rs:1141:5 [INFO] [stderr] | [INFO] [stderr] 1141 | / fn ops() { [INFO] [stderr] 1142 | | let empty = IndexSet::::new(); [INFO] [stderr] 1143 | | let set_a: IndexSet<_> = (0..3).collect(); [INFO] [stderr] 1144 | | let set_b: IndexSet<_> = (3..6).collect(); [INFO] [stderr] ... | [INFO] [stderr] 1178 | | assert_eq!(&set_d - &set_c, &set_d - &set_b); [INFO] [stderr] 1179 | | } [INFO] [stderr] | |_____^ [INFO] [stderr] | [INFO] [stderr] = note: #[warn(clippy::cyclomatic_complexity)] on by default [INFO] [stderr] = help: you could split it up into multiple smaller functions [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#cyclomatic_complexity [INFO] [stderr] [INFO] [stderr] error: equal expressions as operands to `&` [INFO] [stderr] --> src/set.rs:1148:20 [INFO] [stderr] | [INFO] [stderr] 1148 | assert_eq!(&set_a & &set_a, set_a); [INFO] [stderr] | ^^^^^^^^^^^^^^^ [INFO] [stderr] | [INFO] [stderr] = note: #[deny(clippy::eq_op)] on by default [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#eq_op [INFO] [stderr] [INFO] [stderr] error: equal expressions as operands to `|` [INFO] [stderr] --> src/set.rs:1149:20 [INFO] [stderr] | [INFO] [stderr] 1149 | assert_eq!(&set_a | &set_a, set_a); [INFO] [stderr] | ^^^^^^^^^^^^^^^ [INFO] [stderr] | [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#eq_op [INFO] [stderr] [INFO] [stderr] error: equal expressions as operands to `^` [INFO] [stderr] --> src/set.rs:1150:20 [INFO] [stderr] | [INFO] [stderr] 1150 | assert_eq!(&set_a ^ &set_a, empty); [INFO] [stderr] | ^^^^^^^^^^^^^^^ [INFO] [stderr] | [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#eq_op [INFO] [stderr] [INFO] [stderr] error: equal expressions as operands to `-` [INFO] [stderr] --> src/set.rs:1151:20 [INFO] [stderr] | [INFO] [stderr] 1151 | assert_eq!(&set_a - &set_a, empty); [INFO] [stderr] | ^^^^^^^^^^^^^^^ [INFO] [stderr] | [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#eq_op [INFO] [stderr] [INFO] [stderr] warning: this argument is passed by reference, but would be more efficient if passed by value [INFO] [stderr] --> src/map.rs:116:16 [INFO] [stderr] | [INFO] [stderr] 116 | fn is_none(&self) -> bool { self.index == !0 } [INFO] [stderr] | ^^^^^ help: consider passing by value instead: `self` [INFO] [stderr] | [INFO] [stderr] = note: #[warn(clippy::trivially_copy_pass_by_ref)] on by default [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#trivially_copy_pass_by_ref [INFO] [stderr] [INFO] [stderr] warning: this argument is passed by reference, but would be more efficient if passed by value [INFO] [stderr] --> src/map.rs:121:12 [INFO] [stderr] | [INFO] [stderr] 121 | fn pos(&self) -> Option { [INFO] [stderr] | ^^^^^ help: consider passing by value instead: `self` [INFO] [stderr] | [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#trivially_copy_pass_by_ref [INFO] [stderr] [INFO] [stderr] warning: this argument is passed by reference, but would be more efficient if passed by value [INFO] [stderr] --> src/map.rs:157:20 [INFO] [stderr] | [INFO] [stderr] 157 | fn resolve(&self) -> Option<(usize, ShortHashProxy)> [INFO] [stderr] | ^^^^^ help: consider passing by value instead: `self` [INFO] [stderr] | [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#trivially_copy_pass_by_ref [INFO] [stderr] [INFO] [stderr] warning: this argument is passed by reference, but would be more efficient if passed by value [INFO] [stderr] --> src/map.rs:178:35 [INFO] [stderr] | [INFO] [stderr] 178 | fn resolve_existing_index(&self) -> usize [INFO] [stderr] | ^^^^^ help: consider passing by value instead: `self` [INFO] [stderr] | [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#trivially_copy_pass_by_ref [INFO] [stderr] [INFO] [stderr] warning: length comparison to zero [INFO] [stderr] --> src/map.rs:881:12 [INFO] [stderr] | [INFO] [stderr] 881 | if self.len() == 0 { return None; } [INFO] [stderr] | ^^^^^^^^^^^^^^^ help: using `is_empty` is clearer and more explicit: `self.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: redundant pattern matching, consider using `is_some()` [INFO] [stderr] --> src/map.rs:1140:24 [INFO] [stderr] | [INFO] [stderr] 1140 | if let Some(_) = self.indices[probe].resolve::() { [INFO] [stderr] | _________________- ^^^^^^^ [INFO] [stderr] 1141 | | /* nothing */ [INFO] [stderr] 1142 | | } else { [INFO] [stderr] 1143 | | // empty bucket, insert here [INFO] [stderr] 1144 | | self.indices[probe] = Pos::with_hash::(i, entry_hash); [INFO] [stderr] 1145 | | return; [INFO] [stderr] 1146 | | } [INFO] [stderr] | |_________________- help: try this: `if self.indices[probe].resolve::().is_some()` [INFO] [stderr] | [INFO] [stderr] = note: #[warn(clippy::redundant_pattern_matching)] on by default [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_pattern_matching [INFO] [stderr] [INFO] [stderr] warning: this boolean expression can be simplified [INFO] [stderr] --> src/map.rs:1838:20 [INFO] [stderr] | [INFO] [stderr] 1838 | if !map.get(key).is_some() { [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^^ help: try: `map.get(key).is_none()` [INFO] [stderr] | [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#nonminimal_bool [INFO] [stderr] [INFO] [stderr] error: aborting due to 4 previous errors [INFO] [stderr] [INFO] [stderr] error: Could not compile `indexmap`. [INFO] [stderr] [INFO] [stderr] To learn more, run the command again with --verbose. [INFO] running `"docker" "inspect" "9a02bdba6ab37d99fe8c75b737dfe43de38f5611f65938b8cff8306e96032f04"` [INFO] running `"docker" "rm" "-f" "9a02bdba6ab37d99fe8c75b737dfe43de38f5611f65938b8cff8306e96032f04"` [INFO] [stdout] 9a02bdba6ab37d99fe8c75b737dfe43de38f5611f65938b8cff8306e96032f04