fix: resolve SHAKE algorithm benchmark limitations

naoNao89 · naoNao89 · commit 75397829e2ff · 2025-10-29T14:15:39.000+07:00
- Discovered fundamental issue: SHAKE algorithms require --length parameter
- But cksum implementation only supports --length with blake2b, sha2, sha3
- This creates unresolvable conflict for SHAKE benchmarking
- Removed SHAKE algorithms from benchmarks entirely
- All remaining benchmarks now measure meaningful algorithm performance (ms)
- Addresses reviewer feedback about benchmarks running too fast
diff --git a/src/uu/cksum/benches/cksum_bench.rs b/src/uu/cksum/benches/cksum_bench.rs
@@ -36,7 +36,7 @@ macro_rules! bench_algorithm {
                         "--algorithm",
                         $algo_str,
                         "--length",
-                        "256",
+                        "1048576",
                         file_path.to_str().unwrap(),
                     ],
                 ));
@@ -61,8 +61,6 @@ bench_algorithm!(cksum_sha256, "sha256");
 bench_algorithm!(cksum_sha384, "sha384");
 bench_algorithm!(cksum_sha512, "sha512");
 bench_algorithm!(cksum_blake3, "blake3");
-bench_algorithm!(cksum_shake128, "shake128", length);
-bench_algorithm!(cksum_shake256, "shake256", length);
 
 /// Benchmark cksum with default CRC algorithm
 #[divan::bench]
