[INFO] cloning repository https://github.com/bassomatic32/seahaven-rust
[INFO] running `Command { std: "git" "-c" "credential.helper=" "-c" "credential.helper=/workspace/cargo-home/bin/git-credential-null" "clone" "--bare" "https://github.com/bassomatic32/seahaven-rust" "/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2Fbassomatic32%2Fseahaven-rust", kill_on_drop: false }`
[INFO] [stderr] Cloning into bare repository '/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2Fbassomatic32%2Fseahaven-rust'...
[INFO] running `Command { std: "git" "rev-parse" "HEAD", kill_on_drop: false }`
[INFO] [stdout] 20eb2db807134f57cb4b8547f8866e0b8f628258
[INFO] linting bassomatic32/seahaven-rust against nightly for clippy-nonminimal_bool-denied
[INFO] running `Command { std: "git" "clone" "/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2Fbassomatic32%2Fseahaven-rust" "/workspace/builds/worker-0-tc1/source", kill_on_drop: false }`
[INFO] [stderr] Cloning into '/workspace/builds/worker-0-tc1/source'...
[INFO] [stderr] done.
[INFO] started tweaking git repo https://github.com/bassomatic32/seahaven-rust
[INFO] finished tweaking git repo https://github.com/bassomatic32/seahaven-rust
[INFO] tweaked toml for git repo https://github.com/bassomatic32/seahaven-rust written to /workspace/builds/worker-0-tc1/source/Cargo.toml
[INFO] validating manifest of git repo https://github.com/bassomatic32/seahaven-rust on toolchain nightly
[INFO] running `Command { std: CARGO_HOME="/workspace/cargo-home" RUSTUP_HOME="/workspace/rustup-home" "/workspace/cargo-home/bin/cargo" "+nightly" "metadata" "--manifest-path" "Cargo.toml" "--no-deps", kill_on_drop: false }`
[INFO] crate git repo https://github.com/bassomatic32/seahaven-rust already has a lockfile, it will not be regenerated
[INFO] running `Command { std: CARGO_HOME="/workspace/cargo-home" RUSTUP_HOME="/workspace/rustup-home" "/workspace/cargo-home/bin/cargo" "+nightly" "fetch" "--manifest-path" "Cargo.toml", kill_on_drop: false }`
[INFO] [stderr]     Updating crates.io index
[INFO] [stderr]  Downloading crates ...
[INFO] [stderr]   Downloaded crossterm_winapi v0.6.2
[INFO] [stderr]   Downloaded terminal v0.2.1
[INFO] [stderr]   Downloaded parking_lot_core v0.7.3
[INFO] [stderr]   Downloaded crossterm v0.15.0
[INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-0-tc1/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-0-tc1/source:/opt/rustwide/workdir:ro,Z" "-v" "/var/lib/crater-agent-workspace/cargo-home:/opt/rustwide/cargo-home:ro,Z" "-v" "/var/lib/crater-agent-workspace/rustup-home:/opt/rustwide/rustup-home:ro,Z" "-e" "SOURCE_DIR=/opt/rustwide/workdir" "-e" "CARGO_TARGET_DIR=/opt/rustwide/target" "-e" "CARGO_HOME=/opt/rustwide/cargo-home" "-e" "RUSTUP_HOME=/opt/rustwide/rustup-home" "-w" "/opt/rustwide/workdir" "-m" "1610612736" "--user" "0:0" "--network" "none" "ghcr.io/rust-lang/crates-build-env/linux@sha256:d429b63d4308055ea97f60fb1d3dfca48854a00942f1bd2ad806beaf015945ec" "/opt/rustwide/cargo-home/bin/cargo" "+nightly" "metadata" "--no-deps" "--format-version=1", kill_on_drop: false }`
[INFO] [stdout] f81e393538198976bc1e72286ddcf99422cbfb23cff6c2fca4c4052791648ecb
[INFO] running `Command { std: "docker" "start" "-a" "f81e393538198976bc1e72286ddcf99422cbfb23cff6c2fca4c4052791648ecb", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "inspect" "f81e393538198976bc1e72286ddcf99422cbfb23cff6c2fca4c4052791648ecb", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "rm" "-f" "f81e393538198976bc1e72286ddcf99422cbfb23cff6c2fca4c4052791648ecb", kill_on_drop: false }`
[INFO] [stdout] f81e393538198976bc1e72286ddcf99422cbfb23cff6c2fca4c4052791648ecb
[INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-0-tc1/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-0-tc1/source:/opt/rustwide/workdir:ro,Z" "-v" "/var/lib/crater-agent-workspace/cargo-home:/opt/rustwide/cargo-home:ro,Z" "-v" "/var/lib/crater-agent-workspace/rustup-home:/opt/rustwide/rustup-home:ro,Z" "-e" "SOURCE_DIR=/opt/rustwide/workdir" "-e" "CARGO_TARGET_DIR=/opt/rustwide/target" "-e" "CARGO_INCREMENTAL=0" "-e" "RUST_BACKTRACE=full" "-e" "RUSTFLAGS=--cap-lints=forbid" "-e" "RUSTDOCFLAGS=--cap-lints=forbid" "-e" "CARGO_HOME=/opt/rustwide/cargo-home" "-e" "RUSTUP_HOME=/opt/rustwide/rustup-home" "-w" "/opt/rustwide/workdir" "-m" "1610612736" "--user" "0:0" "--network" "none" "ghcr.io/rust-lang/crates-build-env/linux@sha256:d429b63d4308055ea97f60fb1d3dfca48854a00942f1bd2ad806beaf015945ec" "/opt/rustwide/cargo-home/bin/cargo" "+nightly" "clippy" "--frozen" "--all" "--all-targets" "--message-format=json", kill_on_drop: false }`
[INFO] [stdout] 2c4bfd000cc330073b3206eb58d69595263c92f1d4285cce2cef3e90de18f315
[INFO] running `Command { std: "docker" "start" "-a" "2c4bfd000cc330073b3206eb58d69595263c92f1d4285cce2cef3e90de18f315", kill_on_drop: false }`
[INFO] [stderr]     Checking iovec v0.1.4
[INFO] [stderr]     Checking net2 v0.2.39
[INFO] [stderr]     Checking parking_lot_core v0.7.3
[INFO] [stderr]     Checking parking_lot v0.10.2
[INFO] [stderr]     Checking mio v0.6.23
[INFO] [stderr]     Checking signal-hook v0.1.17
[INFO] [stderr]     Checking crossterm v0.15.0
[INFO] [stderr]     Checking terminal v0.2.1
[INFO] [stderr]     Checking seahaven v0.1.0 (/opt/rustwide/workdir)
[INFO] [stdout] warning: this import is redundant
[INFO] [stdout]  --> src/main.rs:5:1
[INFO] [stdout]   |
[INFO] [stdout] 5 | use array_init;
[INFO] [stdout]   | ^^^^^^^^^^^^^^^ help: remove it entirely
[INFO] [stdout]   |
[INFO] [stdout]   = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#single_component_path_imports
[INFO] [stdout]   = note: `#[warn(clippy::single_component_path_imports)]` on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused import: `base64::prelude::*`
[INFO] [stdout]  --> src/main.rs:8:5
[INFO] [stdout]   |
[INFO] [stdout] 8 | use base64::prelude::*;
[INFO] [stdout]   |     ^^^^^^^^^^^^^^^^^^
[INFO] [stdout]   |
[INFO] [stdout]   = note: `#[warn(unused_imports)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused import: `Sha256`
[INFO] [stdout]  --> src/main.rs:9:12
[INFO] [stdout]   |
[INFO] [stdout] 9 | use sha2::{Sha256, Digest};
[INFO] [stdout]   |            ^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unnecessary parentheses around `return` value
[INFO] [stdout]    --> src/main.rs:576:12
[INFO] [stdout]     |
[INFO] [stdout] 576 |                 return (card.value == 1)                
[INFO] [stdout]     |                        ^               ^
[INFO] [stdout]     |
[INFO] [stdout]     = note: `#[warn(unused_parens)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] help: remove these parentheses
[INFO] [stdout]     |
[INFO] [stdout] 576 -                 return (card.value == 1)                
[INFO] [stdout] 576 +                 return card.value == 1                
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unnecessary parentheses around `if` condition
[INFO] [stdout]    --> src/main.rs:603:6
[INFO] [stdout]     |
[INFO] [stdout] 603 |         if (sourceStack.len() > 0) { // cannot move anything from an empty stack
[INFO] [stdout]     |            ^                     ^
[INFO] [stdout]     |
[INFO] [stdout] help: remove these parentheses
[INFO] [stdout]     |
[INFO] [stdout] 603 -         if (sourceStack.len() > 0) { // cannot move anything from an empty stack
[INFO] [stdout] 603 +         if sourceStack.len() > 0 { // cannot move anything from an empty stack
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unnecessary parentheses around `if` condition
[INFO] [stdout]    --> src/main.rs:779:6
[INFO] [stdout]     |
[INFO] [stdout] 779 |         if (success) { tally.winnable += 1 }
[INFO] [stdout]     |            ^       ^
[INFO] [stdout]     |
[INFO] [stdout] help: remove these parentheses
[INFO] [stdout]     |
[INFO] [stdout] 779 -         if (success) { tally.winnable += 1 }
[INFO] [stdout] 779 +         if success { tally.winnable += 1 }
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unnecessary parentheses around `if` condition
[INFO] [stdout]    --> src/main.rs:782:6
[INFO] [stdout]     |
[INFO] [stdout] 782 |         if (game.abandoned) { tally.abandoned += 1 }
[INFO] [stdout]     |            ^              ^
[INFO] [stdout]     |
[INFO] [stdout] help: remove these parentheses
[INFO] [stdout]     |
[INFO] [stdout] 782 -         if (game.abandoned) { tally.abandoned += 1 }
[INFO] [stdout] 782 +         if game.abandoned { tally.abandoned += 1 }
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unnecessary parentheses
[INFO] [stdout]    --> src/main.rs:783:14
[INFO] [stdout]     |
[INFO] [stdout] 783 |         game.print(("Finished"));
[INFO] [stdout]     |                    ^^^^^^^^^^^^ help: remove them: `"Finished"`
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#double_parens
[INFO] [stdout]     = note: `#[warn(clippy::double_parens)]` on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unnecessary parentheses around method argument
[INFO] [stdout]    --> src/main.rs:783:14
[INFO] [stdout]     |
[INFO] [stdout] 783 |         game.print(("Finished"));
[INFO] [stdout]     |                    ^          ^
[INFO] [stdout]     |
[INFO] [stdout] help: remove these parentheses
[INFO] [stdout]     |
[INFO] [stdout] 783 -         game.print(("Finished"));
[INFO] [stdout] 783 +         game.print("Finished");
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: this import is redundant
[INFO] [stdout]  --> src/main.rs:5:1
[INFO] [stdout]   |
[INFO] [stdout] 5 | use array_init;
[INFO] [stdout]   | ^^^^^^^^^^^^^^^ help: remove it entirely
[INFO] [stdout]   |
[INFO] [stdout]   = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#single_component_path_imports
[INFO] [stdout]   = note: `#[warn(clippy::single_component_path_imports)]` on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused import: `base64::prelude::*`
[INFO] [stdout]  --> src/main.rs:8:5
[INFO] [stdout]   |
[INFO] [stdout] 8 | use base64::prelude::*;
[INFO] [stdout]   |     ^^^^^^^^^^^^^^^^^^
[INFO] [stdout]   |
[INFO] [stdout]   = note: `#[warn(unused_imports)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused import: `Sha256`
[INFO] [stdout]  --> src/main.rs:9:12
[INFO] [stdout]   |
[INFO] [stdout] 9 | use sha2::{Sha256, Digest};
[INFO] [stdout]   |            ^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unnecessary parentheses around `return` value
[INFO] [stdout]    --> src/main.rs:576:12
[INFO] [stdout]     |
[INFO] [stdout] 576 |                 return (card.value == 1)                
[INFO] [stdout]     |                        ^               ^
[INFO] [stdout]     |
[INFO] [stdout]     = note: `#[warn(unused_parens)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] help: remove these parentheses
[INFO] [stdout]     |
[INFO] [stdout] 576 -                 return (card.value == 1)                
[INFO] [stdout] 576 +                 return card.value == 1                
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unnecessary parentheses around `if` condition
[INFO] [stdout]    --> src/main.rs:603:6
[INFO] [stdout]     |
[INFO] [stdout] 603 |         if (sourceStack.len() > 0) { // cannot move anything from an empty stack
[INFO] [stdout]     |            ^                     ^
[INFO] [stdout]     |
[INFO] [stdout] help: remove these parentheses
[INFO] [stdout]     |
[INFO] [stdout] 603 -         if (sourceStack.len() > 0) { // cannot move anything from an empty stack
[INFO] [stdout] 603 +         if sourceStack.len() > 0 { // cannot move anything from an empty stack
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unnecessary parentheses around `if` condition
[INFO] [stdout]    --> src/main.rs:779:6
[INFO] [stdout]     |
[INFO] [stdout] 779 |         if (success) { tally.winnable += 1 }
[INFO] [stdout]     |            ^       ^
[INFO] [stdout]     |
[INFO] [stdout] help: remove these parentheses
[INFO] [stdout]     |
[INFO] [stdout] 779 -         if (success) { tally.winnable += 1 }
[INFO] [stdout] 779 +         if success { tally.winnable += 1 }
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unnecessary parentheses around `if` condition
[INFO] [stdout]    --> src/main.rs:782:6
[INFO] [stdout]     |
[INFO] [stdout] 782 |         if (game.abandoned) { tally.abandoned += 1 }
[INFO] [stdout]     |            ^              ^
[INFO] [stdout]     |
[INFO] [stdout] help: remove these parentheses
[INFO] [stdout]     |
[INFO] [stdout] 782 -         if (game.abandoned) { tally.abandoned += 1 }
[INFO] [stdout] 782 +         if game.abandoned { tally.abandoned += 1 }
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unnecessary parentheses
[INFO] [stdout]    --> src/main.rs:783:14
[INFO] [stdout]     |
[INFO] [stdout] 783 |         game.print(("Finished"));
[INFO] [stdout]     |                    ^^^^^^^^^^^^ help: remove them: `"Finished"`
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#double_parens
[INFO] [stdout]     = note: `#[warn(clippy::double_parens)]` on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unnecessary parentheses around method argument
[INFO] [stdout]    --> src/main.rs:783:14
[INFO] [stdout]     |
[INFO] [stdout] 783 |         game.print(("Finished"));
[INFO] [stdout]     |                    ^          ^
[INFO] [stdout]     |
[INFO] [stdout] help: remove these parentheses
[INFO] [stdout]     |
[INFO] [stdout] 783 -         game.print(("Finished"));
[INFO] [stdout] 783 +         game.print("Finished");
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused import: `Digest`
[INFO] [stdout]  --> src/main.rs:9:20
[INFO] [stdout]   |
[INFO] [stdout] 9 | use sha2::{Sha256, Digest};
[INFO] [stdout]   |                    ^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused variable: `i`
[INFO] [stdout]    --> src/main.rs:259:7
[INFO] [stdout]     |
[INFO] [stdout] 259 |     for (i,stackCard) in stack.iter().rev().enumerate() {
[INFO] [stdout]     |          ^ help: if this is intentional, prefix it with an underscore: `_i`
[INFO] [stdout]     |
[INFO] [stdout]     = note: `#[warn(unused_variables)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused variable: `targetStack`
[INFO] [stdout]    --> src/main.rs:487:7
[INFO] [stdout]     |
[INFO] [stdout] 487 |         let targetStack = self.resolvePosition(target);
[INFO] [stdout]     |             ^^^^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_targetStack`
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused variable: `cardName`
[INFO] [stdout]    --> src/main.rs:489:7
[INFO] [stdout]     |
[INFO] [stdout] 489 |         let cardName = cardName(sourceStack.last(), " ".to_string());
[INFO] [stdout]     |             ^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_cardName`
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused variable: `goalStack`
[INFO] [stdout]    --> src/main.rs:607:20
[INFO] [stdout]     |
[INFO] [stdout] 607 |             for (stackIndex,goalStack) in self.board.goals.iter().enumerate() {
[INFO] [stdout]     |                             ^^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_goalStack`
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused variable: `targetStack`
[INFO] [stdout]    --> src/main.rs:628:11
[INFO] [stdout]     |
[INFO] [stdout] 628 |             for (i,targetStack) in self.board.stacks.iter().enumerate() {
[INFO] [stdout]     |                    ^^^^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_targetStack`
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused variable: `i`
[INFO] [stdout]    --> src/main.rs:676:8
[INFO] [stdout]     |
[INFO] [stdout] 676 |             for i in 0..totalExtentMoves  {
[INFO] [stdout]     |                 ^ help: if this is intentional, prefix it with an underscore: `_i`
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: variable does not need to be mutable
[INFO] [stdout]    --> src/main.rs:700:8
[INFO] [stdout]     |
[INFO] [stdout] 700 |             let mut source: Position;
[INFO] [stdout]     |                 ----^^^^^^
[INFO] [stdout]     |                 |
[INFO] [stdout]     |                 help: remove this `mut`
[INFO] [stdout]     |
[INFO] [stdout]     = note: `#[warn(unused_mut)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused variable: `i`
[INFO] [stdout]    --> src/main.rs:740:7
[INFO] [stdout]     |
[INFO] [stdout] 740 |         for i in 0..moveCopy.len() {
[INFO] [stdout]     |             ^ help: if this is intentional, prefix it with an underscore: `_i`
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: fields `msg` and `extent` are never read
[INFO] [stdout]   --> src/main.rs:41:2
[INFO] [stdout]    |
[INFO] [stdout] 38 | struct GameMove {
[INFO] [stdout]    |        -------- fields in this struct
[INFO] [stdout] ...
[INFO] [stdout] 41 |     msg: String,
[INFO] [stdout]    |     ^^^
[INFO] [stdout] 42 |     extent: u16
[INFO] [stdout]    |     ^^^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = note: `GameMove` has derived impls for the traits `Debug` and `Clone`, but these are intentionally ignored during dead code analysis
[INFO] [stdout]    = note: `#[warn(dead_code)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: function `isDisconnectedStack` is never used
[INFO] [stdout]    --> src/main.rs:249:4
[INFO] [stdout]     |
[INFO] [stdout] 249 | fn isDisconnectedStack(stack:&Stack) -> bool {
[INFO] [stdout]     |    ^^^^^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: method `replayGame` is never used
[INFO] [stdout]    --> src/main.rs:735:5
[INFO] [stdout]     |
[INFO] [stdout] 365 | impl Game {
[INFO] [stdout]     | --------- method in this implementation
[INFO] [stdout] ...
[INFO] [stdout] 735 |     fn replayGame(&mut self) {
[INFO] [stdout]     |        ^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: name `GOAL` contains a capitalized acronym
[INFO] [stdout]   --> src/main.rs:46:2
[INFO] [stdout]    |
[INFO] [stdout] 46 |     GOAL,
[INFO] [stdout]    |     ^^^^ help: consider making the acronym lowercase, except the initial letter: `Goal`
[INFO] [stdout]    |
[INFO] [stdout]    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#upper_case_acronyms
[INFO] [stdout]    = note: `#[warn(clippy::upper_case_acronyms)]` on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: name `CELL` contains a capitalized acronym
[INFO] [stdout]   --> src/main.rs:47:2
[INFO] [stdout]    |
[INFO] [stdout] 47 |     CELL,
[INFO] [stdout]    |     ^^^^ help: consider making the acronym lowercase, except the initial letter: `Cell`
[INFO] [stdout]    |
[INFO] [stdout]    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#upper_case_acronyms
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: name `TABLEAU` contains a capitalized acronym
[INFO] [stdout]   --> src/main.rs:48:2
[INFO] [stdout]    |
[INFO] [stdout] 48 |     TABLEAU
[INFO] [stdout]    |     ^^^^^^^ help: consider making the acronym lowercase, except the initial letter: `Tableau`
[INFO] [stdout]    |
[INFO] [stdout]    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#upper_case_acronyms
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unneeded `return` statement
[INFO] [stdout]    --> src/main.rs:113:2
[INFO] [stdout]     |
[INFO] [stdout] 113 |     return board;
[INFO] [stdout]     |     ^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return
[INFO] [stdout]     = note: `#[warn(clippy::needless_return)]` on by default
[INFO] [stdout] help: remove `return`
[INFO] [stdout]     |
[INFO] [stdout] 113 -     return board;
[INFO] [stdout] 113 +     board
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unneeded `return` statement
[INFO] [stdout]    --> src/main.rs:148:4
[INFO] [stdout]     |
[INFO] [stdout] 148 |             return cn;
[INFO] [stdout]     |             ^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return
[INFO] [stdout] help: remove `return`
[INFO] [stdout]     |
[INFO] [stdout] 148 -             return cn;
[INFO] [stdout] 148 +             cn
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unneeded `return` statement
[INFO] [stdout]    --> src/main.rs:205:2
[INFO] [stdout]     |
[INFO] [stdout] 205 |     return false
[INFO] [stdout]     |     ^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return
[INFO] [stdout] help: remove `return`
[INFO] [stdout]     |
[INFO] [stdout] 205 -     return false
[INFO] [stdout] 205 +     false
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unneeded `return` statement
[INFO] [stdout]    --> src/main.rs:226:2
[INFO] [stdout]     |
[INFO] [stdout] 226 |     return count
[INFO] [stdout]     |     ^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return
[INFO] [stdout] help: remove `return`
[INFO] [stdout]     |
[INFO] [stdout] 226 -     return count
[INFO] [stdout] 226 +     count
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: length comparison to zero
[INFO] [stdout]    --> src/main.rs:212:5
[INFO] [stdout]     |
[INFO] [stdout] 212 |     if stack.len() == 0 {
[INFO] [stdout]     |        ^^^^^^^^^^^^^^^^ help: using `is_empty` is clearer and more explicit: `stack.is_empty()`
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#len_zero
[INFO] [stdout]     = note: `#[warn(clippy::len_zero)]` on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unneeded `return` statement
[INFO] [stdout]    --> src/main.rs:241:2
[INFO] [stdout]     |
[INFO] [stdout] 241 |     return 0
[INFO] [stdout]     |     ^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return
[INFO] [stdout] help: remove `return`
[INFO] [stdout]     |
[INFO] [stdout] 241 -     return 0
[INFO] [stdout] 241 +     0
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unneeded `return` statement
[INFO] [stdout]    --> src/main.rs:270:2
[INFO] [stdout]     |
[INFO] [stdout] 270 |     return disconnected;
[INFO] [stdout]     |     ^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return
[INFO] [stdout] help: remove `return`
[INFO] [stdout]     |
[INFO] [stdout] 270 -     return disconnected;
[INFO] [stdout] 270 +     disconnected
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unneeded `return` statement
[INFO] [stdout]    --> src/main.rs:285:2
[INFO] [stdout]     |
[INFO] [stdout] 285 |     return freeCells;
[INFO] [stdout]     |     ^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return
[INFO] [stdout] help: remove `return`
[INFO] [stdout]     |
[INFO] [stdout] 285 -     return freeCells;
[INFO] [stdout] 285 +     freeCells
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: length comparison to zero
[INFO] [stdout]    --> src/main.rs:278:6
[INFO] [stdout]     |
[INFO] [stdout] 278 |         if stack.len() == 0 {
[INFO] [stdout]     |            ^^^^^^^^^^^^^^^^ help: using `is_empty` is clearer and more explicit: `stack.is_empty()`
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#len_zero
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unneeded `return` statement
[INFO] [stdout]    --> src/main.rs:293:2
[INFO] [stdout]     |
[INFO] [stdout] 293 |     return count;
[INFO] [stdout]     |     ^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return
[INFO] [stdout] help: remove `return`
[INFO] [stdout]     |
[INFO] [stdout] 293 -     return count;
[INFO] [stdout] 293 +     count
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unneeded `return` statement
[INFO] [stdout]    --> src/main.rs:297:82
[INFO] [stdout]     |
[INFO] [stdout] 297 |     let tot = board.goals.iter().map(|s| s.len()).reduce(|acc,e| acc + e).unwrap();    return tot as u16;
[INFO] [stdout]     |                                                                                        ^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return
[INFO] [stdout] help: remove `return`
[INFO] [stdout]     |
[INFO] [stdout] 297 -     let tot = board.goals.iter().map(|s| s.len()).reduce(|acc,e| acc + e).unwrap();    return tot as u16;
[INFO] [stdout] 297 +     let tot = board.goals.iter().map(|s| s.len()).reduce(|acc,e| acc + e).unwrap();    tot as u16
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unneeded `return` statement
[INFO] [stdout]    --> src/main.rs:301:2
[INFO] [stdout]     |
[INFO] [stdout] 301 |     return countGoal(board) == 52 // goal will have 52 cards if game is over
[INFO] [stdout]     |     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return
[INFO] [stdout] help: remove `return`
[INFO] [stdout]     |
[INFO] [stdout] 301 -     return countGoal(board) == 52 // goal will have 52 cards if game is over
[INFO] [stdout] 301 +     countGoal(board) == 52 // goal will have 52 cards if game is over
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unneeded `return` statement
[INFO] [stdout]    --> src/main.rs:324:2
[INFO] [stdout]     |
[INFO] [stdout] 324 |     return key;
[INFO] [stdout]     |     ^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return
[INFO] [stdout] help: remove `return`
[INFO] [stdout]     |
[INFO] [stdout] 324 -     return key;
[INFO] [stdout] 324 +     key
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused import: `Digest`
[INFO] [stdout]  --> src/main.rs:9:20
[INFO] [stdout]   |
[INFO] [stdout] 9 | use sha2::{Sha256, Digest};
[INFO] [stdout]   |                    ^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: length comparison to zero
[INFO] [stdout]    --> src/main.rs:320:28
[INFO] [stdout]     |
[INFO] [stdout] 320 |     stacks.sort_by_key(|k| if k.len() == 0 { 0 } else { k[0] });
[INFO] [stdout]     |                               ^^^^^^^^^^^^ help: using `is_empty` is clearer and more explicit: `k.is_empty()`
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#len_zero
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: this boolean expression can be simplified
[INFO] [stdout]    --> src/main.rs:340:5
[INFO] [stdout]     |
[INFO] [stdout] 340 |     if !(stack.len() as u16 > (freeCells + 1) ) { // impossible to be fully ordered unless stack size is greater than the available...
[INFO] [stdout]     |        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: try: `(stack.len() as u16 <= (freeCells + 1))`
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#nonminimal_bool
[INFO] [stdout]     = note: `#[warn(clippy::nonminimal_bool)]` on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unneeded `return` statement
[INFO] [stdout]    --> src/main.rs:350:2
[INFO] [stdout]     |
[INFO] [stdout] 350 |     return false
[INFO] [stdout]     |     ^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return
[INFO] [stdout] help: remove `return`
[INFO] [stdout]     |
[INFO] [stdout] 350 -     return false
[INFO] [stdout] 350 +     false
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: length comparison to zero
[INFO] [stdout]    --> src/main.rs:335:5
[INFO] [stdout]     |
[INFO] [stdout] 335 |     if stack.len() == 0 {
[INFO] [stdout]     |        ^^^^^^^^^^^^^^^^ help: using `is_empty` is clearer and more explicit: `stack.is_empty()`
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#len_zero
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused variable: `i`
[INFO] [stdout]    --> src/main.rs:259:7
[INFO] [stdout]     |
[INFO] [stdout] 259 |     for (i,stackCard) in stack.iter().rev().enumerate() {
[INFO] [stdout]     |          ^ help: if this is intentional, prefix it with an underscore: `_i`
[INFO] [stdout]     |
[INFO] [stdout]     = note: `#[warn(unused_variables)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unneeded `return` statement
[INFO] [stdout]    --> src/main.rs:443:3
[INFO] [stdout]     |
[INFO] [stdout] 443 |         return stack;
[INFO] [stdout]     |         ^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return
[INFO] [stdout] help: remove `return`
[INFO] [stdout]     |
[INFO] [stdout] 443 -         return stack;
[INFO] [stdout] 443 +         stack
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unneeded `return` statement
[INFO] [stdout]    --> src/main.rs:454:3
[INFO] [stdout]     |
[INFO] [stdout] 454 |         return stack;
[INFO] [stdout]     |         ^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return
[INFO] [stdout] help: remove `return`
[INFO] [stdout]     |
[INFO] [stdout] 454 -         return stack;
[INFO] [stdout] 454 +         stack
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unneeded `return` statement
[INFO] [stdout]    --> src/main.rs:459:3
[INFO] [stdout]     |
[INFO] [stdout] 459 |         return stack.pop().unwrap()
[INFO] [stdout]     |         ^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return
[INFO] [stdout] help: remove `return`
[INFO] [stdout]     |
[INFO] [stdout] 459 -         return stack.pop().unwrap()
[INFO] [stdout] 459 +         stack.pop().unwrap()
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unneeded `return` statement
[INFO] [stdout]    --> src/main.rs:482:3
[INFO] [stdout]     |
[INFO] [stdout] 482 |         return false
[INFO] [stdout]     |         ^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return
[INFO] [stdout] help: remove `return`
[INFO] [stdout]     |
[INFO] [stdout] 482 -         return false
[INFO] [stdout] 482 +         false
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: manual implementation of an assign operation
[INFO] [stdout]    --> src/main.rs:472:4
[INFO] [stdout]     |
[INFO] [stdout] 472 |             self.repeatsAvoided = self.repeatsAvoided + 1;
[INFO] [stdout]     |             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: replace it with: `self.repeatsAvoided += 1`
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#assign_op_pattern
[INFO] [stdout]     = note: `#[warn(clippy::assign_op_pattern)]` on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: the loop variable `i` is only used to index `freeCells`
[INFO] [stdout]    --> src/main.rs:513:13
[INFO] [stdout]     |
[INFO] [stdout] 513 |             for i in 0..extent as usize -1 {
[INFO] [stdout]     |                      ^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_range_loop
[INFO] [stdout]     = note: `#[warn(clippy::needless_range_loop)]` on by default
[INFO] [stdout] help: consider using an iterator
[INFO] [stdout]     |
[INFO] [stdout] 513 -             for i in 0..extent as usize -1 {
[INFO] [stdout] 513 +             for <item> in freeCells.iter().take(extent as usize -1) {
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: length comparison to zero
[INFO] [stdout]    --> src/main.rs:531:6
[INFO] [stdout]     |
[INFO] [stdout] 531 |         if sourceStack.len() == 0 {
[INFO] [stdout]     |            ^^^^^^^^^^^^^^^^^^^^^^ help: using `is_empty` is clearer and more explicit: `sourceStack.is_empty()`
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#len_zero
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: manual implementation of `.is_multiple_of()`
[INFO] [stdout]    --> src/main.rs:543:6
[INFO] [stdout]     |
[INFO] [stdout] 543 |         if self.totalMoves % 1000 == 0 {
[INFO] [stdout]     |            ^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: replace with: `self.totalMoves.is_multiple_of(1000)`
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#manual_is_multiple_of
[INFO] [stdout]     = note: `#[warn(clippy::manual_is_multiple_of)]` on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: length comparison to zero
[INFO] [stdout]    --> src/main.rs:550:6
[INFO] [stdout]     |
[INFO] [stdout] 550 |         if self.gameMoves.len() > 0 {
[INFO] [stdout]     |            ^^^^^^^^^^^^^^^^^^^^^^^^ help: using `!is_empty` is clearer and more explicit: `!self.gameMoves.is_empty()`
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#len_zero
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused variable: `targetStack`
[INFO] [stdout]    --> src/main.rs:487:7
[INFO] [stdout]     |
[INFO] [stdout] 487 |         let targetStack = self.resolvePosition(target);
[INFO] [stdout]     |             ^^^^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_targetStack`
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused variable: `cardName`
[INFO] [stdout]    --> src/main.rs:489:7
[INFO] [stdout]     |
[INFO] [stdout] 489 |         let cardName = cardName(sourceStack.last(), " ".to_string());
[INFO] [stdout]     |             ^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_cardName`
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unneeded `return` statement
[INFO] [stdout]    --> src/main.rs:594:3
[INFO] [stdout]     |
[INFO] [stdout] 594 |         return targetCard.suit == card.suit && targetCard.value == (card.value+1) && !isBlockingMove(card, targetStack, extentLength);
[INFO] [stdout]     |         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return
[INFO] [stdout] help: remove `return`
[INFO] [stdout]     |
[INFO] [stdout] 594 -         return targetCard.suit == card.suit && targetCard.value == (card.value+1) && !isBlockingMove(card, targetStack, extentLength);
[INFO] [stdout] 594 +         targetCard.suit == card.suit && targetCard.value == (card.value+1) && !isBlockingMove(card, targetStack, extentLength)
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: length comparison to zero
[INFO] [stdout]    --> src/main.rs:575:7
[INFO] [stdout]     |
[INFO] [stdout] 575 |             if targetStack.len() == 0 {
[INFO] [stdout]     |                ^^^^^^^^^^^^^^^^^^^^^^ help: using `is_empty` is clearer and more explicit: `targetStack.is_empty()`
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#len_zero
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: length comparison to zero
[INFO] [stdout]    --> src/main.rs:584:11
[INFO] [stdout]     |
[INFO] [stdout] 584 |             return targetStack.len() == 0 // our only requiremnt if the target is a Cell is that the stack is empty
[INFO] [stdout]     |                    ^^^^^^^^^^^^^^^^^^^^^^ help: using `is_empty` is clearer and more explicit: `targetStack.is_empty()`
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#len_zero
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: length comparison to zero
[INFO] [stdout]    --> src/main.rs:588:6
[INFO] [stdout]     |
[INFO] [stdout] 588 |         if targetStack.len() == 0 {
[INFO] [stdout]     |            ^^^^^^^^^^^^^^^^^^^^^^ help: using `is_empty` is clearer and more explicit: `targetStack.is_empty()`
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#len_zero
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unneeded `return` statement
[INFO] [stdout]    --> src/main.rs:645:3
[INFO] [stdout]     |
[INFO] [stdout] 645 |         return None;
[INFO] [stdout]     |         ^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return
[INFO] [stdout] help: remove `return`
[INFO] [stdout]     |
[INFO] [stdout] 645 -         return None;
[INFO] [stdout] 645 +         None
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: length comparison to zero
[INFO] [stdout]    --> src/main.rs:603:7
[INFO] [stdout]     |
[INFO] [stdout] 603 |         if (sourceStack.len() > 0) { // cannot move anything from an empty stack
[INFO] [stdout]     |             ^^^^^^^^^^^^^^^^^^^^^ help: using `!is_empty` is clearer and more explicit: `!sourceStack.is_empty()`
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#len_zero
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: this expression creates a reference which is immediately dereferenced by the compiler
[INFO] [stdout]    --> src/main.rs:619:37
[INFO] [stdout]     |
[INFO] [stdout] 619 |                 extent = findExtent(&self.board,&sourceStack);
[INFO] [stdout]     |                                                 ^^^^^^^^^^^^ help: change this to: `sourceStack`
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrow
[INFO] [stdout]     = note: `#[warn(clippy::needless_borrow)]` on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused variable: `goalStack`
[INFO] [stdout]    --> src/main.rs:607:20
[INFO] [stdout]     |
[INFO] [stdout] 607 |             for (stackIndex,goalStack) in self.board.goals.iter().enumerate() {
[INFO] [stdout]     |                             ^^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_goalStack`
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused variable: `targetStack`
[INFO] [stdout]    --> src/main.rs:628:11
[INFO] [stdout]     |
[INFO] [stdout] 628 |             for (i,targetStack) in self.board.stacks.iter().enumerate() {
[INFO] [stdout]     |                    ^^^^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_targetStack`
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused variable: `i`
[INFO] [stdout]    --> src/main.rs:676:8
[INFO] [stdout]     |
[INFO] [stdout] 676 |             for i in 0..totalExtentMoves  {
[INFO] [stdout]     |                 ^ help: if this is intentional, prefix it with an underscore: `_i`
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: this expression creates a reference which is immediately dereferenced by the compiler
[INFO] [stdout]    --> src/main.rs:630:25
[INFO] [stdout]     |
[INFO] [stdout] 630 |                 if self.isLegalMove(&card, target, extent) { return Some(LegalMove{source,target,extent}) }
[INFO] [stdout]     |                                     ^^^^^ help: change this to: `card`
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrow
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: length comparison to zero
[INFO] [stdout]    --> src/main.rs:637:7
[INFO] [stdout]     |
[INFO] [stdout] 637 |             if freeCells.len() > 0 && extent <= 1 {
[INFO] [stdout]     |                ^^^^^^^^^^^^^^^^^^^ help: using `!is_empty` is clearer and more explicit: `!freeCells.is_empty()`
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#len_zero
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: accessing first element with `freeCells.get(0)`
[INFO] [stdout]    --> src/main.rs:639:14
[INFO] [stdout]     |
[INFO] [stdout] 639 |                     target: freeCells.get(0).copied().unwrap(),
[INFO] [stdout]     |                             ^^^^^^^^^^^^^^^^ help: try: `freeCells.first()`
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#get_first
[INFO] [stdout]     = note: `#[warn(clippy::get_first)]` on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: variable does not need to be mutable
[INFO] [stdout]    --> src/main.rs:700:8
[INFO] [stdout]     |
[INFO] [stdout] 700 |             let mut source: Position;
[INFO] [stdout]     |                 ----^^^^^^
[INFO] [stdout]     |                 |
[INFO] [stdout]     |                 help: remove this `mut`
[INFO] [stdout]     |
[INFO] [stdout]     = note: `#[warn(unused_mut)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unneeded `return` statement
[INFO] [stdout]    --> src/main.rs:686:3
[INFO] [stdout]     |
[INFO] [stdout] 686 |         return false // return the fact that this did not succeed
[INFO] [stdout]     |         ^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return
[INFO] [stdout] help: remove `return`
[INFO] [stdout]     |
[INFO] [stdout] 686 -         return false // return the fact that this did not succeed
[INFO] [stdout] 686 +         false // return the fact that this did not succeed
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused variable: `i`
[INFO] [stdout]    --> src/main.rs:740:7
[INFO] [stdout]     |
[INFO] [stdout] 740 |         for i in 0..moveCopy.len() {
[INFO] [stdout]     |             ^ help: if this is intentional, prefix it with an underscore: `_i`
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unneeded `return` statement
[INFO] [stdout]    --> src/main.rs:732:3
[INFO] [stdout]     |
[INFO] [stdout] 732 |         return success;
[INFO] [stdout]     |         ^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return
[INFO] [stdout] help: remove `return`
[INFO] [stdout]     |
[INFO] [stdout] 732 -         return success;
[INFO] [stdout] 732 +         success
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: fields `msg` and `extent` are never read
[INFO] [stdout]   --> src/main.rs:41:2
[INFO] [stdout]    |
[INFO] [stdout] 38 | struct GameMove {
[INFO] [stdout]    |        -------- fields in this struct
[INFO] [stdout] ...
[INFO] [stdout] 41 |     msg: String,
[INFO] [stdout]    |     ^^^
[INFO] [stdout] 42 |     extent: u16
[INFO] [stdout]    |     ^^^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = note: `GameMove` has derived impls for the traits `Debug` and `Clone`, but these are intentionally ignored during dead code analysis
[INFO] [stdout]    = note: `#[warn(dead_code)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: function `isDisconnectedStack` is never used
[INFO] [stdout]    --> src/main.rs:249:4
[INFO] [stdout]     |
[INFO] [stdout] 249 | fn isDisconnectedStack(stack:&Stack) -> bool {
[INFO] [stdout]     |    ^^^^^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: method `replayGame` is never used
[INFO] [stdout]    --> src/main.rs:735:5
[INFO] [stdout]     |
[INFO] [stdout] 365 | impl Game {
[INFO] [stdout]     | --------- method in this implementation
[INFO] [stdout] ...
[INFO] [stdout] 735 |     fn replayGame(&mut self) {
[INFO] [stdout]     |        ^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: you seem to be trying to use `match` for destructuring a single pattern. Consider using `if let`
[INFO] [stdout]    --> src/main.rs:709:4
[INFO] [stdout]     |
[INFO] [stdout] 709 | /             match self.findLegalMove(source) {
[INFO] [stdout] 710 | |                 Some(lm) => {
[INFO] [stdout] 711 | |                     allMoves.push(lm);
[INFO] [stdout] 712 | |                 },
[INFO] [stdout] 713 | |                 None => {}
[INFO] [stdout] 714 | |             }            
[INFO] [stdout]     | |_____________^
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#single_match
[INFO] [stdout]     = note: `#[warn(clippy::single_match)]` on by default
[INFO] [stdout] help: try
[INFO] [stdout]     |
[INFO] [stdout] 709 ~             if let Some(lm) = self.findLegalMove(source) {
[INFO] [stdout] 710 ~                        allMoves.push(lm);
[INFO] [stdout] 711 ~                    }            
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: name `GOAL` contains a capitalized acronym
[INFO] [stdout]   --> src/main.rs:46:2
[INFO] [stdout]    |
[INFO] [stdout] 46 |     GOAL,
[INFO] [stdout]    |     ^^^^ help: consider making the acronym lowercase, except the initial letter: `Goal`
[INFO] [stdout]    |
[INFO] [stdout]    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#upper_case_acronyms
[INFO] [stdout]    = note: `#[warn(clippy::upper_case_acronyms)]` on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: name `CELL` contains a capitalized acronym
[INFO] [stdout]   --> src/main.rs:47:2
[INFO] [stdout]    |
[INFO] [stdout] 47 |     CELL,
[INFO] [stdout]    |     ^^^^ help: consider making the acronym lowercase, except the initial letter: `Cell`
[INFO] [stdout]    |
[INFO] [stdout]    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#upper_case_acronyms
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: name `TABLEAU` contains a capitalized acronym
[INFO] [stdout]   --> src/main.rs:48:2
[INFO] [stdout]    |
[INFO] [stdout] 48 |     TABLEAU
[INFO] [stdout]    |     ^^^^^^^ help: consider making the acronym lowercase, except the initial letter: `Tableau`
[INFO] [stdout]    |
[INFO] [stdout]    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#upper_case_acronyms
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unneeded `return` statement
[INFO] [stdout]    --> src/main.rs:113:2
[INFO] [stdout]     |
[INFO] [stdout] 113 |     return board;
[INFO] [stdout]     |     ^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return
[INFO] [stdout]     = note: `#[warn(clippy::needless_return)]` on by default
[INFO] [stdout] help: remove `return`
[INFO] [stdout]     |
[INFO] [stdout] 113 -     return board;
[INFO] [stdout] 113 +     board
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unneeded `return` statement
[INFO] [stdout]    --> src/main.rs:148:4
[INFO] [stdout]     |
[INFO] [stdout] 148 |             return cn;
[INFO] [stdout]     |             ^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return
[INFO] [stdout] help: remove `return`
[INFO] [stdout]     |
[INFO] [stdout] 148 -             return cn;
[INFO] [stdout] 148 +             cn
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unneeded `return` statement
[INFO] [stdout]    --> src/main.rs:205:2
[INFO] [stdout]     |
[INFO] [stdout] 205 |     return false
[INFO] [stdout]     |     ^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return
[INFO] [stdout] help: remove `return`
[INFO] [stdout]     |
[INFO] [stdout] 205 -     return false
[INFO] [stdout] 205 +     false
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unneeded `return` statement
[INFO] [stdout]    --> src/main.rs:226:2
[INFO] [stdout]     |
[INFO] [stdout] 226 |     return count
[INFO] [stdout]     |     ^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return
[INFO] [stdout] help: remove `return`
[INFO] [stdout]     |
[INFO] [stdout] 226 -     return count
[INFO] [stdout] 226 +     count
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: length comparison to zero
[INFO] [stdout]    --> src/main.rs:212:5
[INFO] [stdout]     |
[INFO] [stdout] 212 |     if stack.len() == 0 {
[INFO] [stdout]     |        ^^^^^^^^^^^^^^^^ help: using `is_empty` is clearer and more explicit: `stack.is_empty()`
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#len_zero
[INFO] [stdout]     = note: `#[warn(clippy::len_zero)]` on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unneeded `return` statement
[INFO] [stdout]    --> src/main.rs:241:2
[INFO] [stdout]     |
[INFO] [stdout] 241 |     return 0
[INFO] [stdout]     |     ^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return
[INFO] [stdout] help: remove `return`
[INFO] [stdout]     |
[INFO] [stdout] 241 -     return 0
[INFO] [stdout] 241 +     0
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unneeded `return` statement
[INFO] [stdout]    --> src/main.rs:270:2
[INFO] [stdout]     |
[INFO] [stdout] 270 |     return disconnected;
[INFO] [stdout]     |     ^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return
[INFO] [stdout] help: remove `return`
[INFO] [stdout]     |
[INFO] [stdout] 270 -     return disconnected;
[INFO] [stdout] 270 +     disconnected
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unneeded `return` statement
[INFO] [stdout]    --> src/main.rs:285:2
[INFO] [stdout]     |
[INFO] [stdout] 285 |     return freeCells;
[INFO] [stdout]     |     ^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return
[INFO] [stdout] help: remove `return`
[INFO] [stdout]     |
[INFO] [stdout] 285 -     return freeCells;
[INFO] [stdout] 285 +     freeCells
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: length comparison to zero
[INFO] [stdout]    --> src/main.rs:278:6
[INFO] [stdout]     |
[INFO] [stdout] 278 |         if stack.len() == 0 {
[INFO] [stdout]     |            ^^^^^^^^^^^^^^^^ help: using `is_empty` is clearer and more explicit: `stack.is_empty()`
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#len_zero
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unneeded `return` statement
[INFO] [stdout]    --> src/main.rs:293:2
[INFO] [stdout]     |
[INFO] [stdout] 293 |     return count;
[INFO] [stdout]     |     ^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return
[INFO] [stdout] help: remove `return`
[INFO] [stdout]     |
[INFO] [stdout] 293 -     return count;
[INFO] [stdout] 293 +     count
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unneeded `return` statement
[INFO] [stdout]    --> src/main.rs:297:82
[INFO] [stdout]     |
[INFO] [stdout] 297 |     let tot = board.goals.iter().map(|s| s.len()).reduce(|acc,e| acc + e).unwrap();    return tot as u16;
[INFO] [stdout]     |                                                                                        ^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return
[INFO] [stdout] help: remove `return`
[INFO] [stdout]     |
[INFO] [stdout] 297 -     let tot = board.goals.iter().map(|s| s.len()).reduce(|acc,e| acc + e).unwrap();    return tot as u16;
[INFO] [stdout] 297 +     let tot = board.goals.iter().map(|s| s.len()).reduce(|acc,e| acc + e).unwrap();    tot as u16
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused `std::result::Result` that must be used
[INFO] [stdout]    --> src/main.rs:384:3
[INFO] [stdout]     |
[INFO] [stdout] 384 |         term.act(Action::MoveCursorTo(1,1));
[INFO] [stdout]     |         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this `Result` may be an `Err` variant, which should be handled
[INFO] [stdout]     = note: `#[warn(unused_must_use)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] help: use `let _ = ...` to ignore the resulting value
[INFO] [stdout]     |
[INFO] [stdout] 384 |         let _ = term.act(Action::MoveCursorTo(1,1));
[INFO] [stdout]     |         +++++++
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused `std::result::Result` that must be used
[INFO] [stdout]    --> src/main.rs:385:3
[INFO] [stdout]     |
[INFO] [stdout] 385 |         term.act(Action::SetForegroundColor(Color::Reset));
[INFO] [stdout]     |         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this `Result` may be an `Err` variant, which should be handled
[INFO] [stdout] help: use `let _ = ...` to ignore the resulting value
[INFO] [stdout]     |
[INFO] [stdout] 385 |         let _ = term.act(Action::SetForegroundColor(Color::Reset));
[INFO] [stdout]     |         +++++++
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unneeded `return` statement
[INFO] [stdout]    --> src/main.rs:301:2
[INFO] [stdout]     |
[INFO] [stdout] 301 |     return countGoal(board) == 52 // goal will have 52 cards if game is over
[INFO] [stdout]     |     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return
[INFO] [stdout] help: remove `return`
[INFO] [stdout]     |
[INFO] [stdout] 301 -     return countGoal(board) == 52 // goal will have 52 cards if game is over
[INFO] [stdout] 301 +     countGoal(board) == 52 // goal will have 52 cards if game is over
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unneeded `return` statement
[INFO] [stdout]    --> src/main.rs:324:2
[INFO] [stdout]     |
[INFO] [stdout] 324 |     return key;
[INFO] [stdout]     |     ^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return
[INFO] [stdout] help: remove `return`
[INFO] [stdout]     |
[INFO] [stdout] 324 -     return key;
[INFO] [stdout] 324 +     key
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused `std::result::Result` that must be used
[INFO] [stdout]    --> src/main.rs:392:4
[INFO] [stdout]     |
[INFO] [stdout] 392 |             term.act(Action::MoveCursorTo(1+(i as u16 * 4),offsetY+1));
[INFO] [stdout]     |             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this `Result` may be an `Err` variant, which should be handled
[INFO] [stdout] help: use `let _ = ...` to ignore the resulting value
[INFO] [stdout]     |
[INFO] [stdout] 392 |             let _ = term.act(Action::MoveCursorTo(1+(i as u16 * 4),offsetY+1));
[INFO] [stdout]     |             +++++++
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused `std::result::Result` that must be used
[INFO] [stdout]    --> src/main.rs:393:4
[INFO] [stdout]     |
[INFO] [stdout] 393 |             term.act(Action::SetForegroundColor(colorCard(goalStack.last())));
[INFO] [stdout]     |             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this `Result` may be an `Err` variant, which should be handled
[INFO] [stdout] help: use `let _ = ...` to ignore the resulting value
[INFO] [stdout]     |
[INFO] [stdout] 393 |             let _ = term.act(Action::SetForegroundColor(colorCard(goalStack.last())));
[INFO] [stdout]     |             +++++++
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused `std::result::Result` that must be used
[INFO] [stdout]    --> src/main.rs:400:4
[INFO] [stdout]     |
[INFO] [stdout] 400 |             term.act(Action::MoveCursorTo(30+(i as u16 * 4),offsetY+1));
[INFO] [stdout]     |             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this `Result` may be an `Err` variant, which should be handled
[INFO] [stdout] help: use `let _ = ...` to ignore the resulting value
[INFO] [stdout]     |
[INFO] [stdout] 400 |             let _ = term.act(Action::MoveCursorTo(30+(i as u16 * 4),offsetY+1));
[INFO] [stdout]     |             +++++++
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: length comparison to zero
[INFO] [stdout]    --> src/main.rs:320:28
[INFO] [stdout]     |
[INFO] [stdout] 320 |     stacks.sort_by_key(|k| if k.len() == 0 { 0 } else { k[0] });
[INFO] [stdout]     |                               ^^^^^^^^^^^^ help: using `is_empty` is clearer and more explicit: `k.is_empty()`
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#len_zero
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: this boolean expression can be simplified
[INFO] [stdout]    --> src/main.rs:340:5
[INFO] [stdout]     |
[INFO] [stdout] 340 |     if !(stack.len() as u16 > (freeCells + 1) ) { // impossible to be fully ordered unless stack size is greater than the available...
[INFO] [stdout]     |        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: try: `(stack.len() as u16 <= (freeCells + 1))`
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#nonminimal_bool
[INFO] [stdout]     = note: `#[warn(clippy::nonminimal_bool)]` on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unneeded `return` statement
[INFO] [stdout]    --> src/main.rs:350:2
[INFO] [stdout]     |
[INFO] [stdout] 350 |     return false
[INFO] [stdout]     |     ^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return
[INFO] [stdout] help: remove `return`
[INFO] [stdout]     |
[INFO] [stdout] 350 -     return false
[INFO] [stdout] 350 +     false
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: length comparison to zero
[INFO] [stdout]    --> src/main.rs:335:5
[INFO] [stdout]     |
[INFO] [stdout] 335 |     if stack.len() == 0 {
[INFO] [stdout]     |        ^^^^^^^^^^^^^^^^ help: using `is_empty` is clearer and more explicit: `stack.is_empty()`
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#len_zero
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused `std::result::Result` that must be used
[INFO] [stdout]    --> src/main.rs:401:4
[INFO] [stdout]     |
[INFO] [stdout] 401 |             term.act(Action::SetForegroundColor(colorCard(cellStack.last())));
[INFO] [stdout]     |             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this `Result` may be an `Err` variant, which should be handled
[INFO] [stdout] help: use `let _ = ...` to ignore the resulting value
[INFO] [stdout]     |
[INFO] [stdout] 401 |             let _ = term.act(Action::SetForegroundColor(colorCard(cellStack.last())));
[INFO] [stdout]     |             +++++++
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused `std::result::Result` that must be used
[INFO] [stdout]    --> src/main.rs:410:5
[INFO] [stdout]     |
[INFO] [stdout] 410 |                 term.act(Action::MoveCursorTo(1+((col as u16)*4),offsetY+3+(row as u16)));            
[INFO] [stdout]     |                 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this `Result` may be an `Err` variant, which should be handled
[INFO] [stdout] help: use `let _ = ...` to ignore the resulting value
[INFO] [stdout]     |
[INFO] [stdout] 410 |                 let _ = term.act(Action::MoveCursorTo(1+((col as u16)*4),offsetY+3+(row as u16)));            
[INFO] [stdout]     |                 +++++++
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused `std::result::Result` that must be used
[INFO] [stdout]    --> src/main.rs:412:5
[INFO] [stdout]     |
[INFO] [stdout] 412 |                 term.act(Action::SetForegroundColor(colorCard(card)));
[INFO] [stdout]     |                 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this `Result` may be an `Err` variant, which should be handled
[INFO] [stdout] help: use `let _ = ...` to ignore the resulting value
[INFO] [stdout]     |
[INFO] [stdout] 412 |                 let _ = term.act(Action::SetForegroundColor(colorCard(card)));
[INFO] [stdout]     |                 +++++++
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused `std::result::Result` that must be used
[INFO] [stdout]    --> src/main.rs:418:3
[INFO] [stdout]     |
[INFO] [stdout] 418 |         term.act(Action::SetForegroundColor(Color::Reset));
[INFO] [stdout]     |         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this `Result` may be an `Err` variant, which should be handled
[INFO] [stdout] help: use `let _ = ...` to ignore the resulting value
[INFO] [stdout]     |
[INFO] [stdout] 418 |         let _ = term.act(Action::SetForegroundColor(Color::Reset));
[INFO] [stdout]     |         +++++++
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused `std::result::Result` that must be used
[INFO] [stdout]    --> src/main.rs:420:3
[INFO] [stdout]     |
[INFO] [stdout] 420 |         term.act(Action::MoveCursorTo(50,offsetY+2));
[INFO] [stdout]     |         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this `Result` may be an `Err` variant, which should be handled
[INFO] [stdout] help: use `let _ = ...` to ignore the resulting value
[INFO] [stdout]     |
[INFO] [stdout] 420 |         let _ = term.act(Action::MoveCursorTo(50,offsetY+2));
[INFO] [stdout]     |         +++++++
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused `std::result::Result` that must be used
[INFO] [stdout]    --> src/main.rs:422:3
[INFO] [stdout]     |
[INFO] [stdout] 422 |         term.act(Action::MoveCursorTo(50,offsetY+4));
[INFO] [stdout]     |         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this `Result` may be an `Err` variant, which should be handled
[INFO] [stdout] help: use `let _ = ...` to ignore the resulting value
[INFO] [stdout]     |
[INFO] [stdout] 422 |         let _ = term.act(Action::MoveCursorTo(50,offsetY+4));
[INFO] [stdout]     |         +++++++
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused `std::result::Result` that must be used
[INFO] [stdout]    --> src/main.rs:424:3
[INFO] [stdout]     |
[INFO] [stdout] 424 |         term.act(Action::MoveCursorTo(50,offsetY+6));
[INFO] [stdout]     |         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this `Result` may be an `Err` variant, which should be handled
[INFO] [stdout] help: use `let _ = ...` to ignore the resulting value
[INFO] [stdout]     |
[INFO] [stdout] 424 |         let _ = term.act(Action::MoveCursorTo(50,offsetY+6));
[INFO] [stdout]     |         +++++++
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused `std::result::Result` that must be used
[INFO] [stdout]    --> src/main.rs:426:3
[INFO] [stdout]     |
[INFO] [stdout] 426 |         term.act(Action::MoveCursorTo(50,offsetY+8));
[INFO] [stdout]     |         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this `Result` may be an `Err` variant, which should be handled
[INFO] [stdout] help: use `let _ = ...` to ignore the resulting value
[INFO] [stdout]     |
[INFO] [stdout] 426 |         let _ = term.act(Action::MoveCursorTo(50,offsetY+8));
[INFO] [stdout]     |         +++++++
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused `std::result::Result` that must be used
[INFO] [stdout]    --> src/main.rs:428:3
[INFO] [stdout]     |
[INFO] [stdout] 428 |         term.act(Action::MoveCursorTo(50,offsetY+10));
[INFO] [stdout]     |         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this `Result` may be an `Err` variant, which should be handled
[INFO] [stdout] help: use `let _ = ...` to ignore the resulting value
[INFO] [stdout]     |
[INFO] [stdout] 428 |         let _ = term.act(Action::MoveCursorTo(50,offsetY+10));
[INFO] [stdout]     |         +++++++
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused `std::result::Result` that must be used
[INFO] [stdout]    --> src/main.rs:763:2
[INFO] [stdout]     |
[INFO] [stdout] 763 |     term.act(Action::ClearTerminal(Clear::All));
[INFO] [stdout]     |     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this `Result` may be an `Err` variant, which should be handled
[INFO] [stdout] help: use `let _ = ...` to ignore the resulting value
[INFO] [stdout]     |
[INFO] [stdout] 763 |     let _ = term.act(Action::ClearTerminal(Clear::All));
[INFO] [stdout]     |     +++++++
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unneeded `return` statement
[INFO] [stdout]    --> src/main.rs:443:3
[INFO] [stdout]     |
[INFO] [stdout] 443 |         return stack;
[INFO] [stdout]     |         ^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return
[INFO] [stdout] help: remove `return`
[INFO] [stdout]     |
[INFO] [stdout] 443 -         return stack;
[INFO] [stdout] 443 +         stack
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unneeded `return` statement
[INFO] [stdout]    --> src/main.rs:454:3
[INFO] [stdout]     |
[INFO] [stdout] 454 |         return stack;
[INFO] [stdout]     |         ^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return
[INFO] [stdout] help: remove `return`
[INFO] [stdout]     |
[INFO] [stdout] 454 -         return stack;
[INFO] [stdout] 454 +         stack
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unneeded `return` statement
[INFO] [stdout]    --> src/main.rs:459:3
[INFO] [stdout]     |
[INFO] [stdout] 459 |         return stack.pop().unwrap()
[INFO] [stdout]     |         ^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return
[INFO] [stdout] help: remove `return`
[INFO] [stdout]     |
[INFO] [stdout] 459 -         return stack.pop().unwrap()
[INFO] [stdout] 459 +         stack.pop().unwrap()
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unneeded `return` statement
[INFO] [stdout]    --> src/main.rs:482:3
[INFO] [stdout]     |
[INFO] [stdout] 482 |         return false
[INFO] [stdout]     |         ^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return
[INFO] [stdout] help: remove `return`
[INFO] [stdout]     |
[INFO] [stdout] 482 -         return false
[INFO] [stdout] 482 +         false
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: manual implementation of an assign operation
[INFO] [stdout]    --> src/main.rs:472:4
[INFO] [stdout]     |
[INFO] [stdout] 472 |             self.repeatsAvoided = self.repeatsAvoided + 1;
[INFO] [stdout]     |             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: replace it with: `self.repeatsAvoided += 1`
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#assign_op_pattern
[INFO] [stdout]     = note: `#[warn(clippy::assign_op_pattern)]` on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: the loop variable `i` is only used to index `freeCells`
[INFO] [stdout]    --> src/main.rs:513:13
[INFO] [stdout]     |
[INFO] [stdout] 513 |             for i in 0..extent as usize -1 {
[INFO] [stdout]     |                      ^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_range_loop
[INFO] [stdout]     = note: `#[warn(clippy::needless_range_loop)]` on by default
[INFO] [stdout] help: consider using an iterator
[INFO] [stdout]     |
[INFO] [stdout] 513 -             for i in 0..extent as usize -1 {
[INFO] [stdout] 513 +             for <item> in freeCells.iter().take(extent as usize -1) {
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: length comparison to zero
[INFO] [stdout]    --> src/main.rs:531:6
[INFO] [stdout]     |
[INFO] [stdout] 531 |         if sourceStack.len() == 0 {
[INFO] [stdout]     |            ^^^^^^^^^^^^^^^^^^^^^^ help: using `is_empty` is clearer and more explicit: `sourceStack.is_empty()`
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#len_zero
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: manual implementation of `.is_multiple_of()`
[INFO] [stdout]    --> src/main.rs:543:6
[INFO] [stdout]     |
[INFO] [stdout] 543 |         if self.totalMoves % 1000 == 0 {
[INFO] [stdout]     |            ^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: replace with: `self.totalMoves.is_multiple_of(1000)`
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#manual_is_multiple_of
[INFO] [stdout]     = note: `#[warn(clippy::manual_is_multiple_of)]` on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: length comparison to zero
[INFO] [stdout]    --> src/main.rs:550:6
[INFO] [stdout]     |
[INFO] [stdout] 550 |         if self.gameMoves.len() > 0 {
[INFO] [stdout]     |            ^^^^^^^^^^^^^^^^^^^^^^^^ help: using `!is_empty` is clearer and more explicit: `!self.gameMoves.is_empty()`
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#len_zero
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unneeded `return` statement
[INFO] [stdout]    --> src/main.rs:594:3
[INFO] [stdout]     |
[INFO] [stdout] 594 |         return targetCard.suit == card.suit && targetCard.value == (card.value+1) && !isBlockingMove(card, targetStack, extentLength);
[INFO] [stdout]     |         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return
[INFO] [stdout] help: remove `return`
[INFO] [stdout]     |
[INFO] [stdout] 594 -         return targetCard.suit == card.suit && targetCard.value == (card.value+1) && !isBlockingMove(card, targetStack, extentLength);
[INFO] [stdout] 594 +         targetCard.suit == card.suit && targetCard.value == (card.value+1) && !isBlockingMove(card, targetStack, extentLength)
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: length comparison to zero
[INFO] [stdout]    --> src/main.rs:575:7
[INFO] [stdout]     |
[INFO] [stdout] 575 |             if targetStack.len() == 0 {
[INFO] [stdout]     |                ^^^^^^^^^^^^^^^^^^^^^^ help: using `is_empty` is clearer and more explicit: `targetStack.is_empty()`
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#len_zero
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: length comparison to zero
[INFO] [stdout]    --> src/main.rs:584:11
[INFO] [stdout]     |
[INFO] [stdout] 584 |             return targetStack.len() == 0 // our only requiremnt if the target is a Cell is that the stack is empty
[INFO] [stdout]     |                    ^^^^^^^^^^^^^^^^^^^^^^ help: using `is_empty` is clearer and more explicit: `targetStack.is_empty()`
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#len_zero
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: length comparison to zero
[INFO] [stdout]    --> src/main.rs:588:6
[INFO] [stdout]     |
[INFO] [stdout] 588 |         if targetStack.len() == 0 {
[INFO] [stdout]     |            ^^^^^^^^^^^^^^^^^^^^^^ help: using `is_empty` is clearer and more explicit: `targetStack.is_empty()`
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#len_zero
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unneeded `return` statement
[INFO] [stdout]    --> src/main.rs:645:3
[INFO] [stdout]     |
[INFO] [stdout] 645 |         return None;
[INFO] [stdout]     |         ^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return
[INFO] [stdout] help: remove `return`
[INFO] [stdout]     |
[INFO] [stdout] 645 -         return None;
[INFO] [stdout] 645 +         None
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: length comparison to zero
[INFO] [stdout]    --> src/main.rs:603:7
[INFO] [stdout]     |
[INFO] [stdout] 603 |         if (sourceStack.len() > 0) { // cannot move anything from an empty stack
[INFO] [stdout]     |             ^^^^^^^^^^^^^^^^^^^^^ help: using `!is_empty` is clearer and more explicit: `!sourceStack.is_empty()`
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#len_zero
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: this expression creates a reference which is immediately dereferenced by the compiler
[INFO] [stdout]    --> src/main.rs:619:37
[INFO] [stdout]     |
[INFO] [stdout] 619 |                 extent = findExtent(&self.board,&sourceStack);
[INFO] [stdout]     |                                                 ^^^^^^^^^^^^ help: change this to: `sourceStack`
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrow
[INFO] [stdout]     = note: `#[warn(clippy::needless_borrow)]` on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: this expression creates a reference which is immediately dereferenced by the compiler
[INFO] [stdout]    --> src/main.rs:630:25
[INFO] [stdout]     |
[INFO] [stdout] 630 |                 if self.isLegalMove(&card, target, extent) { return Some(LegalMove{source,target,extent}) }
[INFO] [stdout]     |                                     ^^^^^ help: change this to: `card`
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrow
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: length comparison to zero
[INFO] [stdout]    --> src/main.rs:637:7
[INFO] [stdout]     |
[INFO] [stdout] 637 |             if freeCells.len() > 0 && extent <= 1 {
[INFO] [stdout]     |                ^^^^^^^^^^^^^^^^^^^ help: using `!is_empty` is clearer and more explicit: `!freeCells.is_empty()`
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#len_zero
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: accessing first element with `freeCells.get(0)`
[INFO] [stdout]    --> src/main.rs:639:14
[INFO] [stdout]     |
[INFO] [stdout] 639 |                     target: freeCells.get(0).copied().unwrap(),
[INFO] [stdout]     |                             ^^^^^^^^^^^^^^^^ help: try: `freeCells.first()`
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#get_first
[INFO] [stdout]     = note: `#[warn(clippy::get_first)]` on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unneeded `return` statement
[INFO] [stdout]    --> src/main.rs:686:3
[INFO] [stdout]     |
[INFO] [stdout] 686 |         return false // return the fact that this did not succeed
[INFO] [stdout]     |         ^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return
[INFO] [stdout] help: remove `return`
[INFO] [stdout]     |
[INFO] [stdout] 686 -         return false // return the fact that this did not succeed
[INFO] [stdout] 686 +         false // return the fact that this did not succeed
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unneeded `return` statement
[INFO] [stdout]    --> src/main.rs:732:3
[INFO] [stdout]     |
[INFO] [stdout] 732 |         return success;
[INFO] [stdout]     |         ^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return
[INFO] [stdout] help: remove `return`
[INFO] [stdout]     |
[INFO] [stdout] 732 -         return success;
[INFO] [stdout] 732 +         success
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: you seem to be trying to use `match` for destructuring a single pattern. Consider using `if let`
[INFO] [stdout]    --> src/main.rs:709:4
[INFO] [stdout]     |
[INFO] [stdout] 709 | /             match self.findLegalMove(source) {
[INFO] [stdout] 710 | |                 Some(lm) => {
[INFO] [stdout] 711 | |                     allMoves.push(lm);
[INFO] [stdout] 712 | |                 },
[INFO] [stdout] 713 | |                 None => {}
[INFO] [stdout] 714 | |             }            
[INFO] [stdout]     | |_____________^
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#single_match
[INFO] [stdout]     = note: `#[warn(clippy::single_match)]` on by default
[INFO] [stdout] help: try
[INFO] [stdout]     |
[INFO] [stdout] 709 ~             if let Some(lm) = self.findLegalMove(source) {
[INFO] [stdout] 710 ~                        allMoves.push(lm);
[INFO] [stdout] 711 ~                    }            
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused `std::result::Result` that must be used
[INFO] [stdout]    --> src/main.rs:384:3
[INFO] [stdout]     |
[INFO] [stdout] 384 |         term.act(Action::MoveCursorTo(1,1));
[INFO] [stdout]     |         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this `Result` may be an `Err` variant, which should be handled
[INFO] [stdout]     = note: `#[warn(unused_must_use)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] help: use `let _ = ...` to ignore the resulting value
[INFO] [stdout]     |
[INFO] [stdout] 384 |         let _ = term.act(Action::MoveCursorTo(1,1));
[INFO] [stdout]     |         +++++++
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused `std::result::Result` that must be used
[INFO] [stdout]    --> src/main.rs:385:3
[INFO] [stdout]     |
[INFO] [stdout] 385 |         term.act(Action::SetForegroundColor(Color::Reset));
[INFO] [stdout]     |         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this `Result` may be an `Err` variant, which should be handled
[INFO] [stdout] help: use `let _ = ...` to ignore the resulting value
[INFO] [stdout]     |
[INFO] [stdout] 385 |         let _ = term.act(Action::SetForegroundColor(Color::Reset));
[INFO] [stdout]     |         +++++++
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused `std::result::Result` that must be used
[INFO] [stdout]    --> src/main.rs:392:4
[INFO] [stdout]     |
[INFO] [stdout] 392 |             term.act(Action::MoveCursorTo(1+(i as u16 * 4),offsetY+1));
[INFO] [stdout]     |             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this `Result` may be an `Err` variant, which should be handled
[INFO] [stdout] help: use `let _ = ...` to ignore the resulting value
[INFO] [stdout]     |
[INFO] [stdout] 392 |             let _ = term.act(Action::MoveCursorTo(1+(i as u16 * 4),offsetY+1));
[INFO] [stdout]     |             +++++++
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused `std::result::Result` that must be used
[INFO] [stdout]    --> src/main.rs:393:4
[INFO] [stdout]     |
[INFO] [stdout] 393 |             term.act(Action::SetForegroundColor(colorCard(goalStack.last())));
[INFO] [stdout]     |             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this `Result` may be an `Err` variant, which should be handled
[INFO] [stdout] help: use `let _ = ...` to ignore the resulting value
[INFO] [stdout]     |
[INFO] [stdout] 393 |             let _ = term.act(Action::SetForegroundColor(colorCard(goalStack.last())));
[INFO] [stdout]     |             +++++++
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused `std::result::Result` that must be used
[INFO] [stdout]    --> src/main.rs:400:4
[INFO] [stdout]     |
[INFO] [stdout] 400 |             term.act(Action::MoveCursorTo(30+(i as u16 * 4),offsetY+1));
[INFO] [stdout]     |             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this `Result` may be an `Err` variant, which should be handled
[INFO] [stdout] help: use `let _ = ...` to ignore the resulting value
[INFO] [stdout]     |
[INFO] [stdout] 400 |             let _ = term.act(Action::MoveCursorTo(30+(i as u16 * 4),offsetY+1));
[INFO] [stdout]     |             +++++++
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused `std::result::Result` that must be used
[INFO] [stdout]    --> src/main.rs:401:4
[INFO] [stdout]     |
[INFO] [stdout] 401 |             term.act(Action::SetForegroundColor(colorCard(cellStack.last())));
[INFO] [stdout]     |             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this `Result` may be an `Err` variant, which should be handled
[INFO] [stdout] help: use `let _ = ...` to ignore the resulting value
[INFO] [stdout]     |
[INFO] [stdout] 401 |             let _ = term.act(Action::SetForegroundColor(colorCard(cellStack.last())));
[INFO] [stdout]     |             +++++++
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused `std::result::Result` that must be used
[INFO] [stdout]    --> src/main.rs:410:5
[INFO] [stdout]     |
[INFO] [stdout] 410 |                 term.act(Action::MoveCursorTo(1+((col as u16)*4),offsetY+3+(row as u16)));            
[INFO] [stdout]     |                 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this `Result` may be an `Err` variant, which should be handled
[INFO] [stdout] help: use `let _ = ...` to ignore the resulting value
[INFO] [stdout]     |
[INFO] [stdout] 410 |                 let _ = term.act(Action::MoveCursorTo(1+((col as u16)*4),offsetY+3+(row as u16)));            
[INFO] [stdout]     |                 +++++++
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused `std::result::Result` that must be used
[INFO] [stdout]    --> src/main.rs:412:5
[INFO] [stdout]     |
[INFO] [stdout] 412 |                 term.act(Action::SetForegroundColor(colorCard(card)));
[INFO] [stdout]     |                 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this `Result` may be an `Err` variant, which should be handled
[INFO] [stdout] help: use `let _ = ...` to ignore the resulting value
[INFO] [stdout]     |
[INFO] [stdout] 412 |                 let _ = term.act(Action::SetForegroundColor(colorCard(card)));
[INFO] [stdout]     |                 +++++++
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused `std::result::Result` that must be used
[INFO] [stdout]    --> src/main.rs:418:3
[INFO] [stdout]     |
[INFO] [stdout] 418 |         term.act(Action::SetForegroundColor(Color::Reset));
[INFO] [stdout]     |         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this `Result` may be an `Err` variant, which should be handled
[INFO] [stdout] help: use `let _ = ...` to ignore the resulting value
[INFO] [stdout]     |
[INFO] [stdout] 418 |         let _ = term.act(Action::SetForegroundColor(Color::Reset));
[INFO] [stdout]     |         +++++++
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused `std::result::Result` that must be used
[INFO] [stdout]    --> src/main.rs:420:3
[INFO] [stdout]     |
[INFO] [stdout] 420 |         term.act(Action::MoveCursorTo(50,offsetY+2));
[INFO] [stdout]     |         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this `Result` may be an `Err` variant, which should be handled
[INFO] [stdout] help: use `let _ = ...` to ignore the resulting value
[INFO] [stdout]     |
[INFO] [stdout] 420 |         let _ = term.act(Action::MoveCursorTo(50,offsetY+2));
[INFO] [stdout]     |         +++++++
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused `std::result::Result` that must be used
[INFO] [stdout]    --> src/main.rs:422:3
[INFO] [stdout]     |
[INFO] [stdout] 422 |         term.act(Action::MoveCursorTo(50,offsetY+4));
[INFO] [stdout]     |         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this `Result` may be an `Err` variant, which should be handled
[INFO] [stdout] help: use `let _ = ...` to ignore the resulting value
[INFO] [stdout]     |
[INFO] [stdout] 422 |         let _ = term.act(Action::MoveCursorTo(50,offsetY+4));
[INFO] [stdout]     |         +++++++
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused `std::result::Result` that must be used
[INFO] [stdout]    --> src/main.rs:424:3
[INFO] [stdout]     |
[INFO] [stdout] 424 |         term.act(Action::MoveCursorTo(50,offsetY+6));
[INFO] [stdout]     |         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this `Result` may be an `Err` variant, which should be handled
[INFO] [stdout] help: use `let _ = ...` to ignore the resulting value
[INFO] [stdout]     |
[INFO] [stdout] 424 |         let _ = term.act(Action::MoveCursorTo(50,offsetY+6));
[INFO] [stdout]     |         +++++++
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused `std::result::Result` that must be used
[INFO] [stdout]    --> src/main.rs:426:3
[INFO] [stdout]     |
[INFO] [stdout] 426 |         term.act(Action::MoveCursorTo(50,offsetY+8));
[INFO] [stdout]     |         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this `Result` may be an `Err` variant, which should be handled
[INFO] [stdout] help: use `let _ = ...` to ignore the resulting value
[INFO] [stdout]     |
[INFO] [stdout] 426 |         let _ = term.act(Action::MoveCursorTo(50,offsetY+8));
[INFO] [stdout]     |         +++++++
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused `std::result::Result` that must be used
[INFO] [stdout]    --> src/main.rs:428:3
[INFO] [stdout]     |
[INFO] [stdout] 428 |         term.act(Action::MoveCursorTo(50,offsetY+10));
[INFO] [stdout]     |         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this `Result` may be an `Err` variant, which should be handled
[INFO] [stdout] help: use `let _ = ...` to ignore the resulting value
[INFO] [stdout]     |
[INFO] [stdout] 428 |         let _ = term.act(Action::MoveCursorTo(50,offsetY+10));
[INFO] [stdout]     |         +++++++
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused `std::result::Result` that must be used
[INFO] [stdout]    --> src/main.rs:763:2
[INFO] [stdout]     |
[INFO] [stdout] 763 |     term.act(Action::ClearTerminal(Clear::All));
[INFO] [stdout]     |     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this `Result` may be an `Err` variant, which should be handled
[INFO] [stdout] help: use `let _ = ...` to ignore the resulting value
[INFO] [stdout]     |
[INFO] [stdout] 763 |     let _ = term.act(Action::ClearTerminal(Clear::All));
[INFO] [stdout]     |     +++++++
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stderr]     Finished `dev` profile [unoptimized + debuginfo] target(s) in 2.26s
[INFO] running `Command { std: "docker" "inspect" "2c4bfd000cc330073b3206eb58d69595263c92f1d4285cce2cef3e90de18f315", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "rm" "-f" "2c4bfd000cc330073b3206eb58d69595263c92f1d4285cce2cef3e90de18f315", kill_on_drop: false }`
[INFO] [stdout] 2c4bfd000cc330073b3206eb58d69595263c92f1d4285cce2cef3e90de18f315