unom/punktfunk
1
0
Fork 0
Code Issues Pull Requests Actions 13 Packages Projects Releases 31 Wiki Activity

style(pf-encode): clear the clippy gate on the HDR/PyroWave additions

Browse Source 
`cargo clippy --workspace --all-targets --locked -- -D warnings` was red on
main — three lints landed with the GNOME 50 HDR + PyroWave 4:4:4 work:

* pyrowave_wire.rs: `aw / 2 >> level` tripped clippy::precedence. Rust already
  binds `/` tighter than `>>`, so this always parsed as `(aw / 2) >> level`
  (subband dim at half res, then one halving per DWT level) — the parens are
  purely explicit, no change in behaviour.
* linux/mod.rs: `probe_can_encode_10bit` sat after `mod hdr_tests`
  (clippy::items_after_test_module) — moved above the test module, unchanged.

Lint-only; no functional change. fmt/clippy/test all green afterwards.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
This commit is contained in:
Enrico Bühler
2026-07-19 10:23:47 +02:00
parent c131603f0d
commit 00f759ec72
2 changed files with 39 additions and 39 deletions
Download Patch File Download Diff File Expand all files Collapse all files
crates/pf-encode/src/enc/pyrowave_wire.rs
+2 -2
View File
@@ -226,8 +226,8 @@ mod tests {
let (aw, ah) = (align(w), align(h));
let mut n = 0u32;
for level in (0..5u32).rev() {
let per = ((aw / 2 >> level).div_ceil(8).div_ceil(4))
* ((ah / 2 >> level).div_ceil(8).div_ceil(4));
let per = (((aw / 2) >> level).div_ceil(8).div_ceil(4))
* (((ah / 2) >> level).div_ceil(8).div_ceil(4));
let bands = if level == 4 { 4 } else { 3 };
for c in 0..3 {
if level == 0 && c != 0 && !c444 {