don't write an error looking message when no line matches

CHANGELOG.md

@@ -1,3 +1,7 @@
+<a name="v1.5.5"></a>
+### v1.5.5 - 2021-12-21
+- don't write an error when no log line matches the query
+
 <a name="v1.5.4"></a>
 ### v1.5.4 - 2021-11-27
 - fix compilation broken by patch release 1.0.49 of anyhow

Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "rhit"
-version = "1.5.4"
+version = "1.5.5"
 authors = ["dystroy <[email protected]>"]
 repository = "https://github.com/Canop/rhit"
 description = "nginx log analyzer"

src/cli/mod.rs

@@ -11,10 +11,6 @@ use {
 
 fn print_analysis(path: &Path, args: &args::Args) -> Result<()> {
     let mut log_base = time!("LogBase::new", LogBase::new(path, args))?;
-    if log_base.lines.is_empty() {
-        eprintln!("no hit in logs");
-        return Ok(());
-    }
     let printer = md::Printer::new(args, &log_base);
     let base = &mut log_base;
     let trend_computer = time!("Trend computer initialization", TrendComputer::new(base, args))?;

src/nginx_log/file_reader.rs

@@ -137,7 +137,12 @@ impl<'c, C: LineConsumer> FileReader<'c, C> {
         }
         execute!(io::stderr(), Clear(ClearType::CurrentLine))?;
         if !self.silent {
-            eprintln!("I've read {} files in {:?}", total, self.root);
+            eprintln!(
+                "I've read {} file{} in {:?}",
+                total,
+                if total > 1 { "s" } else { "" },
+                self.root,
+            );
         }
         Ok(())
     }

src/nginx_log/log_base.rs

@@ -69,15 +69,15 @@ impl LogBase {
         time!("reading files", file_reader.read_all_files())?;
         let filterer = file_reader.filterer();
         let BaseContent {lines, unfiltered_histogram, filtered_histogram, ..} = base_content;
-        if lines.is_empty() {
-            bail!("no log file found in {:?}", path);
-        }
         let mut unfiltered_count = 0;
         let mut dates = Vec::new();
         for bar in &unfiltered_histogram.bars {
             unfiltered_count += bar.hits;
             dates.push(bar.date);
         }
+        if unfiltered_count == 0 {
+            bail!("no hit found in {:?}", path);
+        }
         let filtered_count = filtered_histogram.total_hits();
         Ok(Self {
             dates,