Merge pull request #32

This commit is contained in:
Erik Johnston
2021-10-13 10:18:33 +01:00

View File

@@ -27,7 +27,7 @@ use clap::{crate_authors, crate_description, crate_name, crate_version, value_t,
use indicatif::{ProgressBar, ProgressStyle}; use indicatif::{ProgressBar, ProgressStyle};
use rayon::prelude::*; use rayon::prelude::*;
use state_map::StateMap; use state_map::StateMap;
use std::{collections::BTreeMap, fs::File, io::Write, str::FromStr}; use std::{collections::BTreeMap, convert::TryInto, fs::File, io::Write, str::FromStr};
use string_cache::DefaultAtom as Atom; use string_cache::DefaultAtom as Atom;
mod compressor; mod compressor;
@@ -362,8 +362,8 @@ pub fn run(mut config: Config) {
} }
if let Some(min) = config.min_saved_rows { if let Some(min) = config.min_saved_rows {
let saving = (original_summed_size - compressed_summed_size) as i32; let saving = original_summed_size.saturating_sub(compressed_summed_size);
if saving < min { if saving < min.try_into().unwrap_or(0) {
warn!( warn!(
"Only {} rows would be saved by this compression. Skipping output.", "Only {} rows would be saved by this compression. Skipping output.",
saving saving