Add debug asserts

Author: friendlymatthew

src/font/parser.rs

@@ -1,58 +1,18 @@
 use super::grammar::{
-    CMapFormat0,
-    CMapFormat12,
-    CMapFormat4,
-    CMapIndividualGroup,
-    CMapSubtable,
-    CMapTable,
-    ComponentGlyph,
-    ComponentGlyphArgument,
-    ComponentGlyphFlag,
-    ComponentGlyphTransformation,
-    CompoundGlyph,
-    F2Dot14,
-    FWord,
-    Fixed,
-    FontDirectory,
-    Glyph,
-    GlyphData,
-    GlyphDescription,
-    GlyphTable,
-    HHeaTable,
-    HMtxTable,
-    HeadTable,
-    LongDateTime,
-    LongHorizontalMetric,
-    MaxPTable,
-    OffsetSubTable,
-    ScalarType,
-    SimpleGlyph,
-    SimpleGlyphFlag,
-    TableRecord,
-    TableTag,
-    TrueTypeFontFile,
-    UnsignedFWord,
+    CMapFormat0, CMapFormat12, CMapFormat4, CMapIndividualGroup, CMapSubtable, CMapTable,
+    ComponentGlyph, ComponentGlyphArgument, ComponentGlyphFlag, ComponentGlyphTransformation,
+    CompoundGlyph, F2Dot14, FWord, Fixed, FontDirectory, Glyph, GlyphData, GlyphDescription,
+    GlyphTable, HHeaTable, HMtxTable, HeadTable, LongDateTime, LongHorizontalMetric, MaxPTable,
+    OffsetSubTable, ScalarType, SimpleGlyph, SimpleGlyphFlag, TableRecord, TableTag,
+    TrueTypeFontFile, UnsignedFWord,
 };
 use crate::{
     eof,
-    font::grammar::{
-        IndexToLocFormat,
-        Platform,
-        PlatformDouble,
-    },
+    font::grammar::{IndexToLocFormat, Platform, PlatformDouble},
     read,
-    util::read_bytes::{
-        U16_BYTES,
-        U32_BYTES,
-        U64_BYTES,
-        U8_BYTES,
-    },
-};
-use anyhow::{
-    bail,
-    ensure,
-    Result,
+    util::read_bytes::{U16_BYTES, U32_BYTES, U64_BYTES, U8_BYTES},
 };
+use anyhow::{bail, ensure, Result};
 use std::collections::BTreeMap;
 
 #[derive(Debug)]
@@ -75,7 +35,7 @@ impl<'a> TrueTypeFontParser<'a> {
 
             let offset = self.cursor;
             let head = self.parse_head_table()?;
-            assert_eq!(self.cursor - offset, head_table_record.length as usize);
+            debug_assert_eq!(self.cursor - offset, head_table_record.length as usize);
 
             head
         };
@@ -86,7 +46,7 @@ impl<'a> TrueTypeFontParser<'a> {
 
             let offset = self.cursor;
             let hhea = self.parse_hhea_table()?;
-            assert_eq!(self.cursor - offset, hhea_table_record.length as usize);
+            debug_assert_eq!(self.cursor - offset, hhea_table_record.length as usize);
 
             hhea
         };
@@ -97,7 +57,7 @@ impl<'a> TrueTypeFontParser<'a> {
 
             let offset = self.cursor;
             let maxp = self.parse_maxp_table()?;
-            assert_eq!(self.cursor - offset, maxp_table_record.length as usize);
+            debug_assert_eq!(self.cursor - offset, maxp_table_record.length as usize);
 
             maxp
         };
@@ -110,7 +70,7 @@ impl<'a> TrueTypeFontParser<'a> {
             let loca =
                 self.parse_loca_table(&head_table.index_to_loc_format, &maxp_table.num_glyphs)?;
 
-            assert_eq!(self.cursor - offset, loca_table_record.length as usize);
+            debug_assert_eq!(self.cursor - offset, loca_table_record.length as usize);
 
             loca
         };
@@ -122,7 +82,7 @@ impl<'a> TrueTypeFontParser<'a> {
             let offset = self.cursor;
             let htmx =
                 self.parse_hmtx_table(hhea_table.num_of_long_hor_metrics, maxp_table.num_glyphs)?;
-            assert_eq!(self.cursor - offset, hmtx_table_record.length as usize);
+            debug_assert_eq!(self.cursor - offset, hmtx_table_record.length as usize);
 
             htmx
         };
@@ -502,7 +462,7 @@ impl<'a> TrueTypeFontParser<'a> {
             glyphs.push(Glyph { description, data });
         }
 
-        assert_eq!(glyphs.len(), num_glyphs);
+        debug_assert_eq!(glyphs.len(), num_glyphs);
 
         Ok(GlyphTable { glyphs })
     }

src/png/scanline_reader.rs

@@ -1,10 +1,7 @@
 #![allow(clippy::needless_lifetimes)]
 
 use crate::png::{
-    grammar::{
-        Filter,
-        ImageHeader,
-    },
+    grammar::{Filter, ImageHeader},
     interlace::compute_pass_counts,
 };
 use anyhow::Result;
@@ -19,7 +16,7 @@ impl<'a> ScanlineReader<'a> {
     pub(crate) fn new(input_buffer: &'a [u8], image_header: &'a ImageHeader) -> Self {
         let row_bytes = image_header.num_bytes_per_pixel() * image_header.width as usize;
 
-        assert_eq!(
+        debug_assert_eq!(
             input_buffer.len() - image_header.height as usize,
             row_bytes * image_header.height as usize
         );

src/png/ssim.rs

@@ -1,9 +1,6 @@
 #![allow(clippy::suboptimal_flops)]
 
-use crate::png::grammar::{
-    ColorType,
-    Png,
-};
+use crate::png::grammar::{ColorType, Png};
 use anyhow::ensure;
 
 const K1: f32 = 0.01;
@@ -97,7 +94,7 @@ impl Png {
         let reference_luma_buffer = reference_image.luma_buffer();
         let test_luma_buffer = self.luma_buffer();
 
-        assert_eq!(
+        debug_assert_eq!(
             reference_luma_buffer.lumas.len(),
             test_luma_buffer.lumas.len()
         );