Fix assertion against entering lossless mode

When skipping, we may ignore the quantizer value. Fixes #3209.
xiph · Oct 16, 2023 · e678ad1 · e678ad1
1 parent 77d2a08
commit e678ad1
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/src/encoder.rs b/src/encoder.rs
@@ -2239,7 +2239,11 @@ pub fn write_tx_blocks<T: Pixel, W: Writer>(
   let bw = bsize.width_mi() / tx_size.width_mi();
   let bh = bsize.height_mi() / tx_size.height_mi();
   let qidx = get_qidx(fi, ts, cw, tile_bo);
-  assert_ne!(qidx, 0); // lossless is not yet supported
+
+  // TODO: Lossless is not yet supported.
+  if !skip {
+    assert_ne!(qidx, 0);
+  }
 
   let PlaneConfig { xdec, ydec, .. } = ts.input.planes[1].cfg;
   // SAFETY: We write to the array below before reading from it.