bet

Author: itsdevbear

crates/cli/src/args.rs

@@ -139,7 +139,7 @@ pub struct Args {
     pub no_verbose: bool,
 
     /// Number of blocks per file
-    #[arg(short, long, default_value_t = 1000, help_heading = "Output Options")]
+    #[arg(short, long, default_value_t = 1000, help_heading = "Output Options", value_parser = clap::value_parser!(u64).ref_filter(|s| *s > 0, "chunk_size must be greater than 0"))]
     pub chunk_size: u64,
 
     /// Number of files (alternative to --chunk-size)

crates/freeze/src/types/chunks/number_chunk.rs

@@ -74,6 +74,11 @@ impl NumberChunk {
 
     /// align boundaries of chunk to clean boundaries
     pub fn align(self, chunk_size: u64) -> Option<NumberChunk> {
+        if chunk_size == 0 {
+            // Cannot align with a chunk_size of 0, return None or self based on desired behavior.
+            // Returning None seems appropriate as alignment isn't possible.
+            return None;
+        }
         match self {
             NumberChunk::Numbers(numbers) => Some(NumberChunk::Numbers(numbers)),
             NumberChunk::Range(start, end) => {