[INFO] updating cached repository Goirad/Rusty-Ramsey [INFO] running `"git" "fetch" "--all"` [INFO] [stdout] Fetching origin [INFO] [stderr] From git://github.com/Goirad/Rusty-Ramsey [INFO] [stderr] * branch HEAD -> FETCH_HEAD [INFO] running `"git" "clone" "work/cache/sources/gh/Goirad/Rusty-Ramsey" "work/ex/clippy-test-run/sources/stable/gh/Goirad/Rusty-Ramsey"` [INFO] [stderr] Cloning into 'work/ex/clippy-test-run/sources/stable/gh/Goirad/Rusty-Ramsey'... [INFO] [stderr] done. [INFO] running `"git" "clone" "work/cache/sources/gh/Goirad/Rusty-Ramsey" "work/ex/clippy-test-run/sources/stable+rustflags=-Dclippy%3A%3Ainto_iter_on_array/gh/Goirad/Rusty-Ramsey"` [INFO] [stderr] Cloning into 'work/ex/clippy-test-run/sources/stable+rustflags=-Dclippy%3A%3Ainto_iter_on_array/gh/Goirad/Rusty-Ramsey'... [INFO] [stderr] done. [INFO] running `"git" "rev-parse" "HEAD"` [INFO] [stdout] f9092e4bfea7b52c5288835a4d2a6a6a00b86cf9 [INFO] sha for GitHub repo Goirad/Rusty-Ramsey: f9092e4bfea7b52c5288835a4d2a6a6a00b86cf9 [INFO] validating manifest of Goirad/Rusty-Ramsey 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 Goirad/Rusty-Ramsey 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 Goirad/Rusty-Ramsey [INFO] finished frobbing Goirad/Rusty-Ramsey [INFO] frobbed toml for Goirad/Rusty-Ramsey written to work/ex/clippy-test-run/sources/stable/gh/Goirad/Rusty-Ramsey/Cargo.toml [INFO] started frobbing Goirad/Rusty-Ramsey [INFO] finished frobbing Goirad/Rusty-Ramsey [INFO] frobbed toml for Goirad/Rusty-Ramsey written to work/ex/clippy-test-run/sources/stable+rustflags=-Dclippy%3A%3Ainto_iter_on_array/gh/Goirad/Rusty-Ramsey/Cargo.toml [INFO] crate Goirad/Rusty-Ramsey 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 Goirad/Rusty-Ramsey against stable for clippy-test-run [INFO] running `"docker" "create" "-v" "/mnt/big/crater/work/local/target-dirs/clippy-test-run/worker-4/stable:/opt/crater/target:rw,Z" "-v" "/mnt/big/crater/work/ex/clippy-test-run/sources/stable/gh/Goirad/Rusty-Ramsey:/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] 329046addb5c24143430e6a04042ed9253aefed050a0422ab3dc2753b57936b4 [INFO] running `"docker" "start" "-a" "329046addb5c24143430e6a04042ed9253aefed050a0422ab3dc2753b57936b4"` [INFO] [stderr] Checking permutohedron v0.2.4 [INFO] [stderr] Checking rusty_ramsey v0.1.0 (/opt/crater/workdir) [INFO] [stderr] warning: redundant field names in struct initialization [INFO] [stderr] --> src/graph_lib.rs:180:21 [INFO] [stderr] | [INFO] [stderr] 180 | edges: edges, [INFO] [stderr] | ^^^^^^^^^^^^ help: replace it with: `edges` [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/graph_lib.rs:371:13 [INFO] [stderr] | [INFO] [stderr] 371 | labels: labels, [INFO] [stderr] | ^^^^^^^^^^^^^^ help: replace it with: `labels` [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/graph_lib.rs:180:21 [INFO] [stderr] | [INFO] [stderr] 180 | edges: edges, [INFO] [stderr] | ^^^^^^^^^^^^ help: replace it with: `edges` [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/graph_lib.rs:371:13 [INFO] [stderr] | [INFO] [stderr] 371 | labels: labels, [INFO] [stderr] | ^^^^^^^^^^^^^^ help: replace it with: `labels` [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: unneeded return statement [INFO] [stderr] --> src/graph_lib.rs:44:13 [INFO] [stderr] | [INFO] [stderr] 44 | return &Edge::None; [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^ help: remove `return` as shown: `&Edge::None` [INFO] [stderr] | [INFO] [stderr] = note: #[warn(clippy::needless_return)] on by default [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return [INFO] [stderr] [INFO] [stderr] warning: unneeded return statement [INFO] [stderr] --> src/graph_lib.rs:426:13 [INFO] [stderr] | [INFO] [stderr] 426 | return 1; [INFO] [stderr] | ^^^^^^^^^ help: remove `return` as shown: `1` [INFO] [stderr] | [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return [INFO] [stderr] [INFO] [stderr] warning: unneeded return statement [INFO] [stderr] --> src/graph_lib.rs:510:13 [INFO] [stderr] | [INFO] [stderr] 510 | return true; [INFO] [stderr] | ^^^^^^^^^^^^ help: remove `return` as shown: `true` [INFO] [stderr] | [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return [INFO] [stderr] [INFO] [stderr] warning: unneeded return statement [INFO] [stderr] --> src/graph_lib.rs:529:13 [INFO] [stderr] | [INFO] [stderr] 529 | return false; [INFO] [stderr] | ^^^^^^^^^^^^^ help: remove `return` as shown: `false` [INFO] [stderr] | [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return [INFO] [stderr] [INFO] [stderr] warning: unneeded return statement [INFO] [stderr] --> src/graph_lib.rs:537:13 [INFO] [stderr] | [INFO] [stderr] 537 | return false; [INFO] [stderr] | ^^^^^^^^^^^^^ help: remove `return` as shown: `false` [INFO] [stderr] | [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return [INFO] [stderr] [INFO] [stderr] warning: unneeded return statement [INFO] [stderr] --> src/graph_lib.rs:559:17 [INFO] [stderr] | [INFO] [stderr] 559 | return true; [INFO] [stderr] | ^^^^^^^^^^^^ help: remove `return` as shown: `true` [INFO] [stderr] | [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return [INFO] [stderr] [INFO] [stderr] warning: unneeded return statement [INFO] [stderr] --> src/graph_lib.rs:599:9 [INFO] [stderr] | [INFO] [stderr] 599 | return 1; [INFO] [stderr] | ^^^^^^^^^ help: remove `return` as shown: `1` [INFO] [stderr] | [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return [INFO] [stderr] [INFO] [stderr] warning: unneeded return statement [INFO] [stderr] --> src/graph_lib.rs:636:9 [INFO] [stderr] | [INFO] [stderr] 636 | return None; [INFO] [stderr] | ^^^^^^^^^^^^ help: remove `return` as shown: `None` [INFO] [stderr] | [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return [INFO] [stderr] [INFO] [stderr] warning: unneeded return statement [INFO] [stderr] --> src/graph_lib.rs:719:17 [INFO] [stderr] | [INFO] [stderr] 719 | return false; [INFO] [stderr] | ^^^^^^^^^^^^^ help: remove `return` as shown: `false` [INFO] [stderr] | [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return [INFO] [stderr] [INFO] [stderr] warning: unneeded return statement [INFO] [stderr] --> src/graph_lib.rs:44:13 [INFO] [stderr] | [INFO] [stderr] 44 | return &Edge::None; [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^ help: remove `return` as shown: `&Edge::None` [INFO] [stderr] | [INFO] [stderr] = note: #[warn(clippy::needless_return)] on by default [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return [INFO] [stderr] [INFO] [stderr] warning: unneeded return statement [INFO] [stderr] --> src/graph_lib.rs:426:13 [INFO] [stderr] | [INFO] [stderr] 426 | return 1; [INFO] [stderr] | ^^^^^^^^^ help: remove `return` as shown: `1` [INFO] [stderr] | [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return [INFO] [stderr] [INFO] [stderr] warning: unneeded return statement [INFO] [stderr] --> src/graph_lib.rs:510:13 [INFO] [stderr] | [INFO] [stderr] 510 | return true; [INFO] [stderr] | ^^^^^^^^^^^^ help: remove `return` as shown: `true` [INFO] [stderr] | [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return [INFO] [stderr] [INFO] [stderr] warning: unneeded return statement [INFO] [stderr] --> src/graph_lib.rs:529:13 [INFO] [stderr] | [INFO] [stderr] 529 | return false; [INFO] [stderr] | ^^^^^^^^^^^^^ help: remove `return` as shown: `false` [INFO] [stderr] | [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return [INFO] [stderr] [INFO] [stderr] warning: unneeded return statement [INFO] [stderr] --> src/graph_lib.rs:537:13 [INFO] [stderr] | [INFO] [stderr] 537 | return false; [INFO] [stderr] | ^^^^^^^^^^^^^ help: remove `return` as shown: `false` [INFO] [stderr] | [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return [INFO] [stderr] [INFO] [stderr] warning: unneeded return statement [INFO] [stderr] --> src/graph_lib.rs:559:17 [INFO] [stderr] | [INFO] [stderr] 559 | return true; [INFO] [stderr] | ^^^^^^^^^^^^ help: remove `return` as shown: `true` [INFO] [stderr] | [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return [INFO] [stderr] [INFO] [stderr] warning: unneeded return statement [INFO] [stderr] --> src/graph_lib.rs:599:9 [INFO] [stderr] | [INFO] [stderr] 599 | return 1; [INFO] [stderr] | ^^^^^^^^^ help: remove `return` as shown: `1` [INFO] [stderr] | [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return [INFO] [stderr] [INFO] [stderr] warning: unneeded return statement [INFO] [stderr] --> src/graph_lib.rs:636:9 [INFO] [stderr] | [INFO] [stderr] 636 | return None; [INFO] [stderr] | ^^^^^^^^^^^^ help: remove `return` as shown: `None` [INFO] [stderr] | [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return [INFO] [stderr] [INFO] [stderr] warning: unneeded return statement [INFO] [stderr] --> src/graph_lib.rs:719:17 [INFO] [stderr] | [INFO] [stderr] 719 | return false; [INFO] [stderr] | ^^^^^^^^^^^^^ help: remove `return` as shown: `false` [INFO] [stderr] | [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return [INFO] [stderr] [INFO] [stderr] warning: unused variable: `start` [INFO] [stderr] --> src/main.rs:86:17 [INFO] [stderr] | [INFO] [stderr] 86 | let start = Instant::now(); [INFO] [stderr] | ^^^^^ help: consider using `_start` instead [INFO] [stderr] | [INFO] [stderr] = note: #[warn(unused_variables)] on by default [INFO] [stderr] [INFO] [stderr] warning: unused variable: `num_old` [INFO] [stderr] --> src/main.rs:89:17 [INFO] [stderr] | [INFO] [stderr] 89 | let num_old = c.graphs.len(); [INFO] [stderr] | ^^^^^^^ help: consider using `_num_old` instead [INFO] [stderr] [INFO] [stderr] warning: unused variable: `i` [INFO] [stderr] --> src/main.rs:85:54 [INFO] [stderr] | [INFO] [stderr] 85 | chunks.par_iter_mut().enumerate().for_each(|(i, c)| { [INFO] [stderr] | ^ help: consider using `_i` instead [INFO] [stderr] [INFO] [stderr] warning: unused variable: `start` [INFO] [stderr] --> src/main.rs:132:21 [INFO] [stderr] | [INFO] [stderr] 132 | let start = Instant::now(); [INFO] [stderr] | ^^^^^ help: consider using `_start` instead [INFO] [stderr] [INFO] [stderr] warning: unused variable: `num_old` [INFO] [stderr] --> src/main.rs:135:21 [INFO] [stderr] | [INFO] [stderr] 135 | let num_old = c.graphs.len(); [INFO] [stderr] | ^^^^^^^ help: consider using `_num_old` instead [INFO] [stderr] [INFO] [stderr] warning: unused variable: `i` [INFO] [stderr] --> src/main.rs:131:63 [INFO] [stderr] | [INFO] [stderr] 131 | hard_chunks.par_iter_mut().enumerate().for_each(|(i, c)| { [INFO] [stderr] | ^ help: consider using `_i` instead [INFO] [stderr] [INFO] [stderr] warning: unused variable: `total_comp` [INFO] [stderr] --> src/main.rs:78:17 [INFO] [stderr] | [INFO] [stderr] 78 | let mut total_comp = 0; [INFO] [stderr] | ^^^^^^^^^^ help: consider using `_total_comp` instead [INFO] [stderr] [INFO] [stderr] warning: unused variable: `total_processed` [INFO] [stderr] --> src/main.rs:79:17 [INFO] [stderr] | [INFO] [stderr] 79 | let mut total_processed = 0; [INFO] [stderr] | ^^^^^^^^^^^^^^^ help: consider using `_total_processed` instead [INFO] [stderr] [INFO] [stderr] warning: unused variable: `total_comp` [INFO] [stderr] --> src/main.rs:126:21 [INFO] [stderr] | [INFO] [stderr] 126 | let mut total_comp = 0; [INFO] [stderr] | ^^^^^^^^^^ help: consider using `_total_comp` instead [INFO] [stderr] [INFO] [stderr] warning: unused variable: `total_processed` [INFO] [stderr] --> src/main.rs:127:21 [INFO] [stderr] | [INFO] [stderr] 127 | let mut total_processed = 0; [INFO] [stderr] | ^^^^^^^^^^^^^^^ help: consider using `_total_processed` instead [INFO] [stderr] [INFO] [stderr] warning: method is never used: `collapse` [INFO] [stderr] --> src/graph_lib.rs:648:5 [INFO] [stderr] | [INFO] [stderr] 648 | fn collapse(list: &Vec>) -> Vec { [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stderr] | [INFO] [stderr] = note: #[warn(dead_code)] on by default [INFO] [stderr] [INFO] [stderr] warning: function is never used: `views` [INFO] [stderr] --> src/main.rs:177:1 [INFO] [stderr] | [INFO] [stderr] 177 | fn views(v: &Vec, n: usize) -> Vec<(usize, usize)> { [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stderr] [INFO] [stderr] warning: function is never used: `fmt_dur` [INFO] [stderr] --> src/main.rs:189:1 [INFO] [stderr] | [INFO] [stderr] 189 | fn fmt_dur(d: Duration) -> String { [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stderr] [INFO] [stderr] warning: function is never used: `dedup` [INFO] [stderr] --> src/main.rs:201:1 [INFO] [stderr] | [INFO] [stderr] 201 | fn dedup(v: &mut Vec) { [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stderr] [INFO] [stderr] warning: function is never used: `k3_free` [INFO] [stderr] --> src/main.rs:210:1 [INFO] [stderr] | [INFO] [stderr] 210 | fn k3_free(g: &Graph, col: &Edge) -> bool { [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stderr] [INFO] [stderr] warning: function is never used: `k5_free` [INFO] [stderr] --> src/main.rs:251:1 [INFO] [stderr] | [INFO] [stderr] 251 | fn k5_free(g: &Graph, col: &Edge) -> bool { [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/graph_lib.rs:254:17 [INFO] [stderr] | [INFO] [stderr] 254 | / match *g.get_edge(i, j) { [INFO] [stderr] 255 | | Edge::Red => red_edges += 1, [INFO] [stderr] 256 | | _ => (), [INFO] [stderr] 257 | | } [INFO] [stderr] | |_________________^ help: try this: `if let Edge::Red = *g.get_edge(i, j) { red_edges += 1 }` [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: the loop variable `j` is used to index `labels` [INFO] [stderr] --> src/graph_lib.rs:282:22 [INFO] [stderr] | [INFO] [stderr] 282 | for j in 0..g.vertices { [INFO] [stderr] | ^^^^^^^^^^^^^ [INFO] [stderr] | [INFO] [stderr] = note: #[warn(clippy::needless_range_loop)] on by default [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_range_loop [INFO] [stderr] help: consider using an iterator [INFO] [stderr] | [INFO] [stderr] 282 | for (j, ) in labels.iter().enumerate().take(g.vertices) { [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/graph_lib.rs:283:17 [INFO] [stderr] | [INFO] [stderr] 283 | / match *g.get_edge(i, j) { [INFO] [stderr] 284 | | Edge::Red => red_edges += labels[j] as u16, [INFO] [stderr] 285 | | _ => (), [INFO] [stderr] 286 | | } [INFO] [stderr] | |_________________^ help: try this: `if let Edge::Red = *g.get_edge(i, j) { red_edges += labels[j] as u16 }` [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: casting u8 to u16 may become silently lossy if types change [INFO] [stderr] --> src/graph_lib.rs:284:47 [INFO] [stderr] | [INFO] [stderr] 284 | Edge::Red => red_edges += labels[j] as u16, [INFO] [stderr] | ^^^^^^^^^^^^^^^^ help: try: `u16::from(labels[j])` [INFO] [stderr] | [INFO] [stderr] = note: #[warn(clippy::cast_lossless)] on by default [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#cast_lossless [INFO] [stderr] [INFO] [stderr] warning: the loop variable `j` is used to index `labels` [INFO] [stderr] --> src/graph_lib.rs:311:22 [INFO] [stderr] | [INFO] [stderr] 311 | for j in 0..g.vertices { [INFO] [stderr] | ^^^^^^^^^^^^^ [INFO] [stderr] | [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_range_loop [INFO] [stderr] help: consider using an iterator [INFO] [stderr] | [INFO] [stderr] 311 | for (j, ) in labels.iter().enumerate().take(g.vertices) { [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/graph_lib.rs:312:17 [INFO] [stderr] | [INFO] [stderr] 312 | / match *g.get_edge(i, j) { [INFO] [stderr] 313 | | Edge::Red => red_edges += labels[j].1 as u16, [INFO] [stderr] 314 | | _ => (), [INFO] [stderr] 315 | | } [INFO] [stderr] | |_________________^ help: try this: `if let Edge::Red = *g.get_edge(i, j) { red_edges += labels[j].1 as u16 }` [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: the loop variable `i` is used to index `labels` [INFO] [stderr] --> src/graph_lib.rs:334:18 [INFO] [stderr] | [INFO] [stderr] 334 | for i in 0..g.vertices { [INFO] [stderr] | ^^^^^^^^^^^^^ [INFO] [stderr] | [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_range_loop [INFO] [stderr] help: consider using an iterator [INFO] [stderr] | [INFO] [stderr] 334 | for (i, ) in labels.iter_mut().enumerate().take(g.vertices) { [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/graph_lib.rs:337:17 [INFO] [stderr] | [INFO] [stderr] 337 | / match *g.get_edge(i, j) { [INFO] [stderr] 338 | | Edge::Red => red_edges += 1, [INFO] [stderr] 339 | | _ => (), [INFO] [stderr] 340 | | } [INFO] [stderr] | |_________________^ help: try this: `if let Edge::Red = *g.get_edge(i, j) { red_edges += 1 }` [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: the loop variable `j` is used to index `labels` [INFO] [stderr] --> src/graph_lib.rs:346:22 [INFO] [stderr] | [INFO] [stderr] 346 | for j in 0..g.vertices { [INFO] [stderr] | ^^^^^^^^^^^^^ [INFO] [stderr] | [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_range_loop [INFO] [stderr] help: consider using an iterator [INFO] [stderr] | [INFO] [stderr] 346 | for (j, ) in labels.iter().enumerate().take(g.vertices) { [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/graph_lib.rs:347:17 [INFO] [stderr] | [INFO] [stderr] 347 | / match *g.get_edge(i, j) { [INFO] [stderr] 348 | | Edge::Red => red_edges += labels[j].0 as u16, [INFO] [stderr] 349 | | _ => (), [INFO] [stderr] 350 | | } [INFO] [stderr] | |_________________^ help: try this: `if let Edge::Red = *g.get_edge(i, j) { red_edges += labels[j].0 as u16 }` [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: casting u8 to u16 may become silently lossy if types change [INFO] [stderr] --> src/graph_lib.rs:348:47 [INFO] [stderr] | [INFO] [stderr] 348 | Edge::Red => red_edges += labels[j].0 as u16, [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^ help: try: `u16::from(labels[j].0)` [INFO] [stderr] | [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#cast_lossless [INFO] [stderr] [INFO] [stderr] warning: the loop variable `j` is used to index `labels` [INFO] [stderr] --> src/graph_lib.rs:356:22 [INFO] [stderr] | [INFO] [stderr] 356 | for j in 0..g.vertices { [INFO] [stderr] | ^^^^^^^^^^^^^ [INFO] [stderr] | [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_range_loop [INFO] [stderr] help: consider using an iterator [INFO] [stderr] | [INFO] [stderr] 356 | for (j, ) in labels.iter().enumerate().take(g.vertices) { [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/graph_lib.rs:357:17 [INFO] [stderr] | [INFO] [stderr] 357 | / match *g.get_edge(i, j) { [INFO] [stderr] 358 | | Edge::Red => red_edges += labels[j].1 as u16, [INFO] [stderr] 359 | | _ => (), [INFO] [stderr] 360 | | } [INFO] [stderr] | |_________________^ help: try this: `if let Edge::Red = *g.get_edge(i, j) { red_edges += labels[j].1 as u16 }` [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: the loop variable `i` is used to index `old_labels` [INFO] [stderr] --> src/graph_lib.rs:383:18 [INFO] [stderr] | [INFO] [stderr] 383 | for i in 0..g.vertices { [INFO] [stderr] | ^^^^^^^^^^^^^ [INFO] [stderr] | [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_range_loop [INFO] [stderr] help: consider using an iterator [INFO] [stderr] | [INFO] [stderr] 383 | for (i, ) in old_labels.iter().enumerate().take(g.vertices) { [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/graph_lib.rs:386:17 [INFO] [stderr] | [INFO] [stderr] 386 | / match *g.get_edge(i, j) { [INFO] [stderr] 387 | | Edge::Red => connected.push(j), [INFO] [stderr] 388 | | _ => (), [INFO] [stderr] 389 | | } [INFO] [stderr] | |_________________^ help: try this: `if let Edge::Red = *g.get_edge(i, j) { connected.push(j) }` [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/graph_lib.rs:395:25 [INFO] [stderr] | [INFO] [stderr] 395 | / match *g.get_edge(connected[j], connected[k]) { [INFO] [stderr] 396 | | Edge::Red => k3s += 1, [INFO] [stderr] 397 | | _ => (), [INFO] [stderr] 398 | | } [INFO] [stderr] | |_________________________^ help: try this: `if let Edge::Red = *g.get_edge(connected[j], connected[k]) { k3s += 1 }` [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: writing `&Vec<_>` instead of `&[_]` involves one more reference and cannot be used with non-Vec-based slices. [INFO] [stderr] --> src/graph_lib.rs:424:24 [INFO] [stderr] | [INFO] [stderr] 424 | fn calc_comp(list: &Vec) -> u64 { [INFO] [stderr] | ^^^^^^^ help: change this to: `&[T]` [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] [INFO] [stderr] warning: writing `&Vec<_>` instead of `&[_]` involves one more reference and cannot be used with non-Vec-based slices. [INFO] [stderr] --> src/graph_lib.rs:455:18 [INFO] [stderr] | [INFO] [stderr] 455 | fn bin(list: &Vec) -> Vec> { [INFO] [stderr] | ^^^^^^^ help: change this to: `&[T]` [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: writing `&Vec<_>` instead of `&[_]` involves one more reference and cannot be used with non-Vec-based slices. [INFO] [stderr] --> src/graph_lib.rs:492:23 [INFO] [stderr] | [INFO] [stderr] 492 | orig_verts_g: &Vec>, //original bins, with more than one element [INFO] [stderr] | ^^^^^^^^^^^^^^^^ help: change this to: `&[Vec]` [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: writing `&Vec<_>` instead of `&[_]` involves one more reference and cannot be used with non-Vec-based slices. [INFO] [stderr] --> src/graph_lib.rs:494:28 [INFO] [stderr] | [INFO] [stderr] 494 | collapsed_verts_h: &Vec, //the vertices of h collapsed into a single vector [INFO] [stderr] | ^^^^^^^^^^^ help: change this to: `&[usize]` [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: this function has too many arguments (8/7) [INFO] [stderr] --> src/graph_lib.rs:489:5 [INFO] [stderr] | [INFO] [stderr] 489 | / fn rec_iso_check( [INFO] [stderr] 490 | | depth: usize, [INFO] [stderr] 491 | | max_depth: usize, [INFO] [stderr] 492 | | orig_verts_g: &Vec>, //original bins, with more than one element [INFO] [stderr] ... | [INFO] [stderr] 530 | | } [INFO] [stderr] 531 | | } [INFO] [stderr] | |_____^ [INFO] [stderr] | [INFO] [stderr] = note: #[warn(clippy::too_many_arguments)] on by default [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#too_many_arguments [INFO] [stderr] [INFO] [stderr] warning: length comparison to zero [INFO] [stderr] --> src/graph_lib.rs:541:16 [INFO] [stderr] | [INFO] [stderr] 541 | if g_bins.len() > 0 { [INFO] [stderr] | ^^^^^^^^^^^^^^^^ help: using `is_empty` is clearer and more explicit: `!g_bins.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: writing `&Vec<_>` instead of `&[_]` involves one more reference and cannot be used with non-Vec-based slices. [INFO] [stderr] --> src/graph_lib.rs:565:19 [INFO] [stderr] | [INFO] [stderr] 565 | fn collapse(list: &Vec>, out: &mut Vec) { [INFO] [stderr] | ^^^^^^^^^^^^^^^^ help: change this to: `&[Vec]` [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: writing `&Vec<_>` instead of `&[_]` involves one more reference and cannot be used with non-Vec-based slices. [INFO] [stderr] --> src/graph_lib.rs:629:22 [INFO] [stderr] | [INFO] [stderr] 629 | fn find_split(v: &Vec>) -> Option { [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^ help: change this to: `&[LabeledGraph]` [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: writing `&Vec<_>` instead of `&[_]` involves one more reference and cannot be used with non-Vec-based slices. [INFO] [stderr] --> src/graph_lib.rs:648:23 [INFO] [stderr] | [INFO] [stderr] 648 | fn collapse(list: &Vec>) -> Vec { [INFO] [stderr] | ^^^^^^^^^^^^^^^^ help: change this to: `&[Vec]` [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: writing `&Vec<_>` instead of `&[_]` involves one more reference and cannot be used with non-Vec-based slices. [INFO] [stderr] --> src/graph_lib.rs:701:23 [INFO] [stderr] | [INFO] [stderr] 701 | orig_verts_g: &Vec>, //original bins, with more than one element [INFO] [stderr] | ^^^^^^^^^^^^^^^^ help: change this to: `&[Vec]` [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: the loop variable `i` is used to index `collapsed_verts_g` [INFO] [stderr] --> src/graph_lib.rs:710:26 [INFO] [stderr] | [INFO] [stderr] 710 | for i in 0..collapsed_verts_g.len() - 1 { [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stderr] | [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_range_loop [INFO] [stderr] help: consider using an iterator [INFO] [stderr] | [INFO] [stderr] 710 | for (i, ) in collapsed_verts_g.iter().enumerate().take(collapsed_verts_g.len() - 1) { [INFO] [stderr] | ^^^^^^^^^^^ ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stderr] [INFO] [stderr] warning: the loop variable `j` is used to index `collapsed_verts_g` [INFO] [stderr] --> src/graph_lib.rs:711:30 [INFO] [stderr] | [INFO] [stderr] 711 | for j in 0..collapsed_verts_g.len() { [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stderr] | [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_range_loop [INFO] [stderr] help: consider using an iterator [INFO] [stderr] | [INFO] [stderr] 711 | for (j, ) in collapsed_verts_g.iter().enumerate() { [INFO] [stderr] | ^^^^^^^^^^^ ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stderr] [INFO] [stderr] warning: length comparison to zero [INFO] [stderr] --> src/main.rs:71:12 [INFO] [stderr] | [INFO] [stderr] 71 | if next.len() == 0 { [INFO] [stderr] | ^^^^^^^^^^^^^^^ help: using `is_empty` is clearer and more explicit: `next.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] error: this boolean expression contains a logic bug [INFO] [stderr] --> src/main.rs:91:16 [INFO] [stderr] | [INFO] [stderr] 91 | if false && comp > 500_000 { [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^^ help: it would look like the following: `false` [INFO] [stderr] | [INFO] [stderr] = note: #[deny(clippy::logic_bug)] on by default [INFO] [stderr] help: this expression can be optimized out by applying boolean operations to the outer expression [INFO] [stderr] --> src/main.rs:91:25 [INFO] [stderr] | [INFO] [stderr] 91 | if false && comp > 500_000 { [INFO] [stderr] | ^^^^^^^^^^^^^^ [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#logic_bug [INFO] [stderr] [INFO] [stderr] warning: length comparison to zero [INFO] [stderr] --> src/main.rs:124:12 [INFO] [stderr] | [INFO] [stderr] 124 | if hard_ones.len() > 0 { [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^ help: using `is_empty` is clearer and more explicit: `!hard_ones.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: writing `&Vec<_>` instead of `&[_]` involves one more reference and cannot be used with non-Vec-based slices. [INFO] [stderr] --> src/main.rs:177:16 [INFO] [stderr] | [INFO] [stderr] 177 | fn views(v: &Vec, n: usize) -> Vec<(usize, usize)> { [INFO] [stderr] | ^^^^^^^ help: change this to: `&[T]` [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: manual implementation of an assign operation [INFO] [stderr] --> src/main.rs:183:9 [INFO] [stderr] | [INFO] [stderr] 183 | curr = curr + size; [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^ help: replace it with: `curr += size` [INFO] [stderr] | [INFO] [stderr] = note: #[warn(clippy::assign_op_pattern)] on by default [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#assign_op_pattern [INFO] [stderr] [INFO] [stderr] warning: Calling `subsec_millis()` is more concise than this calculation [INFO] [stderr] --> src/main.rs:197:9 [INFO] [stderr] | [INFO] [stderr] 197 | d.subsec_nanos() / 1_000_000 [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: try: `d.subsec_millis()` [INFO] [stderr] | [INFO] [stderr] = note: #[warn(clippy::duration_subsec)] on by default [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#duration_subsec [INFO] [stderr] [INFO] [stderr] warning: this argument is passed by reference, but would be more efficient if passed by value [INFO] [stderr] --> src/main.rs:210:28 [INFO] [stderr] | [INFO] [stderr] 210 | fn k3_free(g: &Graph, col: &Edge) -> bool { [INFO] [stderr] | ^^^^^ help: consider passing by value instead: `Edge` [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/main.rs:226:28 [INFO] [stderr] | [INFO] [stderr] 226 | fn k4_free(g: &Graph, col: &Edge) -> bool { [INFO] [stderr] | ^^^^^ help: consider passing by value instead: `Edge` [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/main.rs:251:28 [INFO] [stderr] | [INFO] [stderr] 251 | fn k5_free(g: &Graph, col: &Edge) -> bool { [INFO] [stderr] | ^^^^^ help: consider passing by value instead: `Edge` [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] error: aborting due to previous error [INFO] [stderr] [INFO] [stderr] error: Could not compile `rusty_ramsey`. [INFO] [stderr] warning: build failed, waiting for other jobs to finish... [INFO] [stderr] warning: unused variable: `start` [INFO] [stderr] --> src/main.rs:86:17 [INFO] [stderr] | [INFO] [stderr] 86 | let start = Instant::now(); [INFO] [stderr] | ^^^^^ help: consider using `_start` instead [INFO] [stderr] | [INFO] [stderr] = note: #[warn(unused_variables)] on by default [INFO] [stderr] [INFO] [stderr] warning: unused variable: `num_old` [INFO] [stderr] --> src/main.rs:89:17 [INFO] [stderr] | [INFO] [stderr] 89 | let num_old = c.graphs.len(); [INFO] [stderr] | ^^^^^^^ help: consider using `_num_old` instead [INFO] [stderr] [INFO] [stderr] warning: unused variable: `i` [INFO] [stderr] --> src/main.rs:85:54 [INFO] [stderr] | [INFO] [stderr] 85 | chunks.par_iter_mut().enumerate().for_each(|(i, c)| { [INFO] [stderr] | ^ help: consider using `_i` instead [INFO] [stderr] [INFO] [stderr] warning: unused variable: `start` [INFO] [stderr] --> src/main.rs:132:21 [INFO] [stderr] | [INFO] [stderr] 132 | let start = Instant::now(); [INFO] [stderr] | ^^^^^ help: consider using `_start` instead [INFO] [stderr] [INFO] [stderr] warning: unused variable: `num_old` [INFO] [stderr] --> src/main.rs:135:21 [INFO] [stderr] | [INFO] [stderr] 135 | let num_old = c.graphs.len(); [INFO] [stderr] | ^^^^^^^ help: consider using `_num_old` instead [INFO] [stderr] [INFO] [stderr] warning: unused variable: `i` [INFO] [stderr] --> src/main.rs:131:63 [INFO] [stderr] | [INFO] [stderr] 131 | hard_chunks.par_iter_mut().enumerate().for_each(|(i, c)| { [INFO] [stderr] | ^ help: consider using `_i` instead [INFO] [stderr] [INFO] [stderr] warning: unused variable: `total_comp` [INFO] [stderr] --> src/main.rs:78:17 [INFO] [stderr] | [INFO] [stderr] 78 | let mut total_comp = 0; [INFO] [stderr] | ^^^^^^^^^^ help: consider using `_total_comp` instead [INFO] [stderr] [INFO] [stderr] warning: unused variable: `total_processed` [INFO] [stderr] --> src/main.rs:79:17 [INFO] [stderr] | [INFO] [stderr] 79 | let mut total_processed = 0; [INFO] [stderr] | ^^^^^^^^^^^^^^^ help: consider using `_total_processed` instead [INFO] [stderr] [INFO] [stderr] warning: unused variable: `total_comp` [INFO] [stderr] --> src/main.rs:126:21 [INFO] [stderr] | [INFO] [stderr] 126 | let mut total_comp = 0; [INFO] [stderr] | ^^^^^^^^^^ help: consider using `_total_comp` instead [INFO] [stderr] [INFO] [stderr] warning: unused variable: `total_processed` [INFO] [stderr] --> src/main.rs:127:21 [INFO] [stderr] | [INFO] [stderr] 127 | let mut total_processed = 0; [INFO] [stderr] | ^^^^^^^^^^^^^^^ help: consider using `_total_processed` instead [INFO] [stderr] [INFO] [stderr] warning: method is never used: `collapse` [INFO] [stderr] --> src/graph_lib.rs:648:5 [INFO] [stderr] | [INFO] [stderr] 648 | fn collapse(list: &Vec>) -> Vec { [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stderr] | [INFO] [stderr] = note: #[warn(dead_code)] on by default [INFO] [stderr] [INFO] [stderr] warning: function is never used: `views` [INFO] [stderr] --> src/main.rs:177:1 [INFO] [stderr] | [INFO] [stderr] 177 | fn views(v: &Vec, n: usize) -> Vec<(usize, usize)> { [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stderr] [INFO] [stderr] warning: function is never used: `fmt_dur` [INFO] [stderr] --> src/main.rs:189:1 [INFO] [stderr] | [INFO] [stderr] 189 | fn fmt_dur(d: Duration) -> String { [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stderr] [INFO] [stderr] warning: function is never used: `dedup` [INFO] [stderr] --> src/main.rs:201:1 [INFO] [stderr] | [INFO] [stderr] 201 | fn dedup(v: &mut Vec) { [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stderr] [INFO] [stderr] warning: function is never used: `k3_free` [INFO] [stderr] --> src/main.rs:210:1 [INFO] [stderr] | [INFO] [stderr] 210 | fn k3_free(g: &Graph, col: &Edge) -> bool { [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stderr] [INFO] [stderr] warning: function is never used: `k5_free` [INFO] [stderr] --> src/main.rs:251:1 [INFO] [stderr] | [INFO] [stderr] 251 | fn k5_free(g: &Graph, col: &Edge) -> bool { [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/graph_lib.rs:254:17 [INFO] [stderr] | [INFO] [stderr] 254 | / match *g.get_edge(i, j) { [INFO] [stderr] 255 | | Edge::Red => red_edges += 1, [INFO] [stderr] 256 | | _ => (), [INFO] [stderr] 257 | | } [INFO] [stderr] | |_________________^ help: try this: `if let Edge::Red = *g.get_edge(i, j) { red_edges += 1 }` [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: the loop variable `j` is used to index `labels` [INFO] [stderr] --> src/graph_lib.rs:282:22 [INFO] [stderr] | [INFO] [stderr] 282 | for j in 0..g.vertices { [INFO] [stderr] | ^^^^^^^^^^^^^ [INFO] [stderr] | [INFO] [stderr] = note: #[warn(clippy::needless_range_loop)] on by default [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_range_loop [INFO] [stderr] help: consider using an iterator [INFO] [stderr] | [INFO] [stderr] 282 | for (j, ) in labels.iter().enumerate().take(g.vertices) { [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/graph_lib.rs:283:17 [INFO] [stderr] | [INFO] [stderr] 283 | / match *g.get_edge(i, j) { [INFO] [stderr] 284 | | Edge::Red => red_edges += labels[j] as u16, [INFO] [stderr] 285 | | _ => (), [INFO] [stderr] 286 | | } [INFO] [stderr] | |_________________^ help: try this: `if let Edge::Red = *g.get_edge(i, j) { red_edges += labels[j] as u16 }` [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: casting u8 to u16 may become silently lossy if types change [INFO] [stderr] --> src/graph_lib.rs:284:47 [INFO] [stderr] | [INFO] [stderr] 284 | Edge::Red => red_edges += labels[j] as u16, [INFO] [stderr] | ^^^^^^^^^^^^^^^^ help: try: `u16::from(labels[j])` [INFO] [stderr] | [INFO] [stderr] = note: #[warn(clippy::cast_lossless)] on by default [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#cast_lossless [INFO] [stderr] [INFO] [stderr] warning: the loop variable `j` is used to index `labels` [INFO] [stderr] --> src/graph_lib.rs:311:22 [INFO] [stderr] | [INFO] [stderr] 311 | for j in 0..g.vertices { [INFO] [stderr] | ^^^^^^^^^^^^^ [INFO] [stderr] | [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_range_loop [INFO] [stderr] help: consider using an iterator [INFO] [stderr] | [INFO] [stderr] 311 | for (j, ) in labels.iter().enumerate().take(g.vertices) { [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/graph_lib.rs:312:17 [INFO] [stderr] | [INFO] [stderr] 312 | / match *g.get_edge(i, j) { [INFO] [stderr] 313 | | Edge::Red => red_edges += labels[j].1 as u16, [INFO] [stderr] 314 | | _ => (), [INFO] [stderr] 315 | | } [INFO] [stderr] | |_________________^ help: try this: `if let Edge::Red = *g.get_edge(i, j) { red_edges += labels[j].1 as u16 }` [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: the loop variable `i` is used to index `labels` [INFO] [stderr] --> src/graph_lib.rs:334:18 [INFO] [stderr] | [INFO] [stderr] 334 | for i in 0..g.vertices { [INFO] [stderr] | ^^^^^^^^^^^^^ [INFO] [stderr] | [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_range_loop [INFO] [stderr] help: consider using an iterator [INFO] [stderr] | [INFO] [stderr] 334 | for (i, ) in labels.iter_mut().enumerate().take(g.vertices) { [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/graph_lib.rs:337:17 [INFO] [stderr] | [INFO] [stderr] 337 | / match *g.get_edge(i, j) { [INFO] [stderr] 338 | | Edge::Red => red_edges += 1, [INFO] [stderr] 339 | | _ => (), [INFO] [stderr] 340 | | } [INFO] [stderr] | |_________________^ help: try this: `if let Edge::Red = *g.get_edge(i, j) { red_edges += 1 }` [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: the loop variable `j` is used to index `labels` [INFO] [stderr] --> src/graph_lib.rs:346:22 [INFO] [stderr] | [INFO] [stderr] 346 | for j in 0..g.vertices { [INFO] [stderr] | ^^^^^^^^^^^^^ [INFO] [stderr] | [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_range_loop [INFO] [stderr] help: consider using an iterator [INFO] [stderr] | [INFO] [stderr] 346 | for (j, ) in labels.iter().enumerate().take(g.vertices) { [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/graph_lib.rs:347:17 [INFO] [stderr] | [INFO] [stderr] 347 | / match *g.get_edge(i, j) { [INFO] [stderr] 348 | | Edge::Red => red_edges += labels[j].0 as u16, [INFO] [stderr] 349 | | _ => (), [INFO] [stderr] 350 | | } [INFO] [stderr] | |_________________^ help: try this: `if let Edge::Red = *g.get_edge(i, j) { red_edges += labels[j].0 as u16 }` [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: casting u8 to u16 may become silently lossy if types change [INFO] [stderr] --> src/graph_lib.rs:348:47 [INFO] [stderr] | [INFO] [stderr] 348 | Edge::Red => red_edges += labels[j].0 as u16, [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^ help: try: `u16::from(labels[j].0)` [INFO] [stderr] | [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#cast_lossless [INFO] [stderr] [INFO] [stderr] warning: the loop variable `j` is used to index `labels` [INFO] [stderr] --> src/graph_lib.rs:356:22 [INFO] [stderr] | [INFO] [stderr] 356 | for j in 0..g.vertices { [INFO] [stderr] | ^^^^^^^^^^^^^ [INFO] [stderr] | [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_range_loop [INFO] [stderr] help: consider using an iterator [INFO] [stderr] | [INFO] [stderr] 356 | for (j, ) in labels.iter().enumerate().take(g.vertices) { [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/graph_lib.rs:357:17 [INFO] [stderr] | [INFO] [stderr] 357 | / match *g.get_edge(i, j) { [INFO] [stderr] 358 | | Edge::Red => red_edges += labels[j].1 as u16, [INFO] [stderr] 359 | | _ => (), [INFO] [stderr] 360 | | } [INFO] [stderr] | |_________________^ help: try this: `if let Edge::Red = *g.get_edge(i, j) { red_edges += labels[j].1 as u16 }` [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: the loop variable `i` is used to index `old_labels` [INFO] [stderr] --> src/graph_lib.rs:383:18 [INFO] [stderr] | [INFO] [stderr] 383 | for i in 0..g.vertices { [INFO] [stderr] | ^^^^^^^^^^^^^ [INFO] [stderr] | [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_range_loop [INFO] [stderr] help: consider using an iterator [INFO] [stderr] | [INFO] [stderr] 383 | for (i, ) in old_labels.iter().enumerate().take(g.vertices) { [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/graph_lib.rs:386:17 [INFO] [stderr] | [INFO] [stderr] 386 | / match *g.get_edge(i, j) { [INFO] [stderr] 387 | | Edge::Red => connected.push(j), [INFO] [stderr] 388 | | _ => (), [INFO] [stderr] 389 | | } [INFO] [stderr] | |_________________^ help: try this: `if let Edge::Red = *g.get_edge(i, j) { connected.push(j) }` [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/graph_lib.rs:395:25 [INFO] [stderr] | [INFO] [stderr] 395 | / match *g.get_edge(connected[j], connected[k]) { [INFO] [stderr] 396 | | Edge::Red => k3s += 1, [INFO] [stderr] 397 | | _ => (), [INFO] [stderr] 398 | | } [INFO] [stderr] | |_________________________^ help: try this: `if let Edge::Red = *g.get_edge(connected[j], connected[k]) { k3s += 1 }` [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: writing `&Vec<_>` instead of `&[_]` involves one more reference and cannot be used with non-Vec-based slices. [INFO] [stderr] --> src/graph_lib.rs:424:24 [INFO] [stderr] | [INFO] [stderr] 424 | fn calc_comp(list: &Vec) -> u64 { [INFO] [stderr] | ^^^^^^^ help: change this to: `&[T]` [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] [INFO] [stderr] warning: writing `&Vec<_>` instead of `&[_]` involves one more reference and cannot be used with non-Vec-based slices. [INFO] [stderr] --> src/graph_lib.rs:455:18 [INFO] [stderr] | [INFO] [stderr] 455 | fn bin(list: &Vec) -> Vec> { [INFO] [stderr] | ^^^^^^^ help: change this to: `&[T]` [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: writing `&Vec<_>` instead of `&[_]` involves one more reference and cannot be used with non-Vec-based slices. [INFO] [stderr] --> src/graph_lib.rs:492:23 [INFO] [stderr] | [INFO] [stderr] 492 | orig_verts_g: &Vec>, //original bins, with more than one element [INFO] [stderr] | ^^^^^^^^^^^^^^^^ help: change this to: `&[Vec]` [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: writing `&Vec<_>` instead of `&[_]` involves one more reference and cannot be used with non-Vec-based slices. [INFO] [stderr] --> src/graph_lib.rs:494:28 [INFO] [stderr] | [INFO] [stderr] 494 | collapsed_verts_h: &Vec, //the vertices of h collapsed into a single vector [INFO] [stderr] | ^^^^^^^^^^^ help: change this to: `&[usize]` [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: this function has too many arguments (8/7) [INFO] [stderr] --> src/graph_lib.rs:489:5 [INFO] [stderr] | [INFO] [stderr] 489 | / fn rec_iso_check( [INFO] [stderr] 490 | | depth: usize, [INFO] [stderr] 491 | | max_depth: usize, [INFO] [stderr] 492 | | orig_verts_g: &Vec>, //original bins, with more than one element [INFO] [stderr] ... | [INFO] [stderr] 530 | | } [INFO] [stderr] 531 | | } [INFO] [stderr] | |_____^ [INFO] [stderr] | [INFO] [stderr] = note: #[warn(clippy::too_many_arguments)] on by default [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#too_many_arguments [INFO] [stderr] [INFO] [stderr] warning: length comparison to zero [INFO] [stderr] --> src/graph_lib.rs:541:16 [INFO] [stderr] | [INFO] [stderr] 541 | if g_bins.len() > 0 { [INFO] [stderr] | ^^^^^^^^^^^^^^^^ help: using `is_empty` is clearer and more explicit: `!g_bins.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: writing `&Vec<_>` instead of `&[_]` involves one more reference and cannot be used with non-Vec-based slices. [INFO] [stderr] --> src/graph_lib.rs:565:19 [INFO] [stderr] | [INFO] [stderr] 565 | fn collapse(list: &Vec>, out: &mut Vec) { [INFO] [stderr] | ^^^^^^^^^^^^^^^^ help: change this to: `&[Vec]` [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: writing `&Vec<_>` instead of `&[_]` involves one more reference and cannot be used with non-Vec-based slices. [INFO] [stderr] --> src/graph_lib.rs:629:22 [INFO] [stderr] | [INFO] [stderr] 629 | fn find_split(v: &Vec>) -> Option { [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^ help: change this to: `&[LabeledGraph]` [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: writing `&Vec<_>` instead of `&[_]` involves one more reference and cannot be used with non-Vec-based slices. [INFO] [stderr] --> src/graph_lib.rs:648:23 [INFO] [stderr] | [INFO] [stderr] 648 | fn collapse(list: &Vec>) -> Vec { [INFO] [stderr] | ^^^^^^^^^^^^^^^^ help: change this to: `&[Vec]` [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: writing `&Vec<_>` instead of `&[_]` involves one more reference and cannot be used with non-Vec-based slices. [INFO] [stderr] --> src/graph_lib.rs:701:23 [INFO] [stderr] | [INFO] [stderr] 701 | orig_verts_g: &Vec>, //original bins, with more than one element [INFO] [stderr] | ^^^^^^^^^^^^^^^^ help: change this to: `&[Vec]` [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: the loop variable `i` is used to index `collapsed_verts_g` [INFO] [stderr] --> src/graph_lib.rs:710:26 [INFO] [stderr] | [INFO] [stderr] 710 | for i in 0..collapsed_verts_g.len() - 1 { [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stderr] | [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_range_loop [INFO] [stderr] help: consider using an iterator [INFO] [stderr] | [INFO] [stderr] 710 | for (i, ) in collapsed_verts_g.iter().enumerate().take(collapsed_verts_g.len() - 1) { [INFO] [stderr] | ^^^^^^^^^^^ ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stderr] [INFO] [stderr] warning: the loop variable `j` is used to index `collapsed_verts_g` [INFO] [stderr] --> src/graph_lib.rs:711:30 [INFO] [stderr] | [INFO] [stderr] 711 | for j in 0..collapsed_verts_g.len() { [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stderr] | [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_range_loop [INFO] [stderr] help: consider using an iterator [INFO] [stderr] | [INFO] [stderr] 711 | for (j, ) in collapsed_verts_g.iter().enumerate() { [INFO] [stderr] | ^^^^^^^^^^^ ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stderr] [INFO] [stderr] warning: length comparison to zero [INFO] [stderr] --> src/main.rs:71:12 [INFO] [stderr] | [INFO] [stderr] 71 | if next.len() == 0 { [INFO] [stderr] | ^^^^^^^^^^^^^^^ help: using `is_empty` is clearer and more explicit: `next.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] error: this boolean expression contains a logic bug [INFO] [stderr] --> src/main.rs:91:16 [INFO] [stderr] | [INFO] [stderr] 91 | if false && comp > 500_000 { [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^^ help: it would look like the following: `false` [INFO] [stderr] | [INFO] [stderr] = note: #[deny(clippy::logic_bug)] on by default [INFO] [stderr] help: this expression can be optimized out by applying boolean operations to the outer expression [INFO] [stderr] --> src/main.rs:91:25 [INFO] [stderr] | [INFO] [stderr] 91 | if false && comp > 500_000 { [INFO] [stderr] | ^^^^^^^^^^^^^^ [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#logic_bug [INFO] [stderr] [INFO] [stderr] warning: length comparison to zero [INFO] [stderr] --> src/main.rs:124:12 [INFO] [stderr] | [INFO] [stderr] 124 | if hard_ones.len() > 0 { [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^ help: using `is_empty` is clearer and more explicit: `!hard_ones.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: writing `&Vec<_>` instead of `&[_]` involves one more reference and cannot be used with non-Vec-based slices. [INFO] [stderr] --> src/main.rs:177:16 [INFO] [stderr] | [INFO] [stderr] 177 | fn views(v: &Vec, n: usize) -> Vec<(usize, usize)> { [INFO] [stderr] | ^^^^^^^ help: change this to: `&[T]` [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: manual implementation of an assign operation [INFO] [stderr] --> src/main.rs:183:9 [INFO] [stderr] | [INFO] [stderr] 183 | curr = curr + size; [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^ help: replace it with: `curr += size` [INFO] [stderr] | [INFO] [stderr] = note: #[warn(clippy::assign_op_pattern)] on by default [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#assign_op_pattern [INFO] [stderr] [INFO] [stderr] warning: Calling `subsec_millis()` is more concise than this calculation [INFO] [stderr] --> src/main.rs:197:9 [INFO] [stderr] | [INFO] [stderr] 197 | d.subsec_nanos() / 1_000_000 [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: try: `d.subsec_millis()` [INFO] [stderr] | [INFO] [stderr] = note: #[warn(clippy::duration_subsec)] on by default [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#duration_subsec [INFO] [stderr] [INFO] [stderr] warning: this argument is passed by reference, but would be more efficient if passed by value [INFO] [stderr] --> src/main.rs:210:28 [INFO] [stderr] | [INFO] [stderr] 210 | fn k3_free(g: &Graph, col: &Edge) -> bool { [INFO] [stderr] | ^^^^^ help: consider passing by value instead: `Edge` [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/main.rs:226:28 [INFO] [stderr] | [INFO] [stderr] 226 | fn k4_free(g: &Graph, col: &Edge) -> bool { [INFO] [stderr] | ^^^^^ help: consider passing by value instead: `Edge` [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/main.rs:251:28 [INFO] [stderr] | [INFO] [stderr] 251 | fn k5_free(g: &Graph, col: &Edge) -> bool { [INFO] [stderr] | ^^^^^ help: consider passing by value instead: `Edge` [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] error: aborting due to previous error [INFO] [stderr] [INFO] [stderr] error: Could not compile `rusty_ramsey`. [INFO] [stderr] [INFO] [stderr] To learn more, run the command again with --verbose. [INFO] running `"docker" "inspect" "329046addb5c24143430e6a04042ed9253aefed050a0422ab3dc2753b57936b4"` [INFO] running `"docker" "rm" "-f" "329046addb5c24143430e6a04042ed9253aefed050a0422ab3dc2753b57936b4"` [INFO] [stdout] 329046addb5c24143430e6a04042ed9253aefed050a0422ab3dc2753b57936b4