Merge pull request #365 from armansito/pr-non-zero-buffer-sizes

[encoding] Do not allow zero-length buffer sizes

crates/encoding/src/config.rs

@@ -185,7 +185,12 @@ impl<T: Sized> BufferSize<T> {
     /// Creates a new buffer size from number of elements.
     pub const fn new(len: u32) -> Self {
         Self {
-            len,
+            // Each buffer binding must be large enough to hold at least one element to avoid
+            // triggering validation errors.
+            //
+            // Note: not using `Ord::max` here because it doesn't support const eval yet (except
+            // in nightly)
+            len: if len > 0 { len } else { 1 },
             _phantom: std::marker::PhantomData,
         }
     }

src/engine.rs

@@ -676,11 +676,8 @@ impl Recording {
 impl BufProxy {
     pub fn new(size: u64, name: &'static str) -> Self {
         let id = Id::next();
-        BufProxy {
-            id,
-            size: size.max(16),
-            name,
-        }
+        debug_assert!(size > 0);
+        BufProxy { id, size, name }
     }
 }