Skip to content

Commit 686f60e

Browse files
jqnatividadclaude
andcommitted
fix: clarify Records-mode cap comment to avoid arithmetic imprecision
Replace '≈ 200 MB' with '> MAX_RECORDS_BYTES (100 MB)' in the test_records_mode_cap_boundary_ok comment to avoid implying false exactness (200_000 × 1_024 = 195 MiB, not 200 MB). Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
1 parent e686077 commit 686f60e

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

src/sniffer.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -757,7 +757,7 @@ mod tests {
757757
);
758758
let cursor = std::io::Cursor::new(data);
759759
let mut sniffer = Sniffer::new();
760-
// Records(200_000): estimated_size = 200_000 * 1024 ≈ 200 MB, clamped to MAX_RECORDS_BYTES.
760+
// Records(200_000): estimated_size = 200_000 * 1024 > MAX_RECORDS_BYTES (100 MB), clamped to MAX_RECORDS_BYTES.
761761
sniffer.sample_size(SampleSize::Records(200_000));
762762
let result = sniffer.sniff_reader(cursor);
763763
assert!(

0 commit comments

Comments
 (0)