inflate/constants: Bump OFFSET_ENOUGH and clarify reason for bump

Signed-off-by: caleb <etemesicaleb@gmail.com>
etemesi254 · Dec 12, 2022 · 5877c2f · 5877c2f
1 parent 033aafc
commit 5877c2f
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/zune-inflate/src/constants.rs b/zune-inflate/src/constants.rs
@@ -30,7 +30,9 @@ pub const LITLEN_ENOUGH: usize = 2342;
 /// Maximum bits found in the lookup table for offsets
 /// offsets larger than this require a lookup into a sub-table
 pub const OFFSET_TABLEBITS: usize = 8;
-pub const OFFSET_ENOUGH: usize = 402;
+/// Note, default libdeflate value is 402, but with 512,
+/// we can remove a branch check by simply doing & 511, and I'll take that.
+pub const OFFSET_ENOUGH: usize = 512;
 /// Maximum number of symbols across all codes
 pub const DEFLATE_MAX_NUM_SYMS: usize = 288;