warning: casting to the same type is unnecessary (`u64` -> `u64`) --> src/static_lut.rs:621:29 | 621 | Lut6::from_blocks(&[table as u64]) | ^^^^^^^^^^^^ help: try: `table` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#unnecessary_cast = note: `#[warn(clippy::unnecessary_cast)]` on by default

the loop variable `i` is used to index `res_perm`: src/canonization.rs#L272

warning: the loop variable `i` is used to index `res_perm` --> src/canonization.rs:272:14 | 272 | for i in 0..res_perm.len() { | ^^^^^^^^^^^^^^^^^ | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_range_loop help: consider using an iterator and enumerate() | 272 | for (i, <item>) in res_perm.iter_mut().enumerate() { | ~~~~~~~~~~~ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

the variable `ind` is used as a loop counter: src/canonization.rs#L233

warning: the variable `ind` is used as a loop counter --> src/canonization.rs:233:5 | 233 | for swap in all_swaps { | ^^^^^^^^^^^^^^^^^^^^^ help: consider using: `for (ind, swap) in all_swaps.iter().enumerate()` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#explicit_counter_loop

the loop variable `i` is used to index `res_perm`: src/canonization.rs#L229

warning: the loop variable `i` is used to index `res_perm` --> src/canonization.rs:229:14 | 229 | for i in 0..res_perm.len() { | ^^^^^^^^^^^^^^^^^ | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_range_loop help: consider using an iterator and enumerate() | 229 | for (i, <item>) in res_perm.iter_mut().enumerate() { | ~~~~~~~~~~~ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

the variable `ind` is used as a loop counter: src/canonization.rs#L163

warning: the variable `ind` is used as a loop counter --> src/canonization.rs:163:5 | 163 | for swap in all_swaps { | ^^^^^^^^^^^^^^^^^^^^^ help: consider using: `for (ind, swap) in all_swaps.iter().enumerate()` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#explicit_counter_loop = note: `#[warn(clippy::explicit_counter_loop)]` on by default

the loop variable `i` is used to index `all_perms`: src/canonization.rs#L117

warning: the loop variable `i` is used to index `all_perms` --> src/canonization.rs:117:18 | 117 | for i in 0..all_perms.len() { | ^^^^^^^^^^^^^^^^^^ | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_range_loop = note: `#[warn(clippy::needless_range_loop)]` on by default help: consider using an iterator and enumerate() | 117 | for (i, <item>) in all_perms.iter().enumerate() { | ~~~~~~~~~~~ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~

associated items `simplify`, `not`, `or`, and `and` are never used: src/sop.rs#L230

warning: associated items `simplify`, `not`, `or`, and `and` are never used --> src/sop.rs:230:8 | 184 | impl Sop { | -------- associated items in this implementation ... 230 | fn simplify(&mut self) { | ^^^^^^^^ ... 237 | fn not(&self) -> Sop { | ^^^ ... 242 | fn or(a: &Sop, b: &Sop) -> Sop { | ^^ ... 249 | fn and(a: &Sop, b: &Sop) -> Sop { | ^^^ | = note: `#[warn(dead_code)]` on by default

Clippy

The following actions uses node12 which is deprecated and will be forced to run on node16: actions-rs/toolchain@v1, actions-rs/clippy-check@v1. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/

Clippy