Merge pull request #63 from smithlabcode/non-hts-mode-bugfix

Fixing a bug that checked BAM things even when not compiled with HTS support
smithlabcode · Jul 15, 2024 · 5d29eec · 5d29eec
2 parents 1f46ff4 + 8b6d8e3
commit 5d29eec
Showing 1 changed file with 5 additions and 4 deletions.
diff --git a/src/falco.cpp b/src/falco.cpp
@@ -77,11 +77,12 @@ read_stream_into_stats(T &in, FastqStats &stats, FalcoConfig &falco_config) {
   size_t file_size = in.load();
   size_t tot_bytes_read = 0;
 
-  // can't report progress for bam files
-  constexpr bool is_bam = std::is_same<T, BamReader>::value;
-
   // decide whether to report progress
-  const bool quiet = falco_config.quiet || is_bam;
+  const bool quiet = falco_config.quiet
+#ifdef USE_HTS
+    || std::is_same<T, BamReader>::value // can't do progress for bam
+#endif
+    ;
 
   ProgressBar progress(file_size, "running falco");
   if (!quiet)