diff --git a/README.md b/README.md index 44df649..0ac1397 100644 --- a/README.md +++ b/README.md @@ -13,9 +13,9 @@ Identify which human reference genome was used to align a BAM/SAM/CRAM file.

- - - Fulcrum Genomics + + + Fulcrum Genomics

diff --git a/src/catalog/builder.rs b/src/catalog/builder.rs index 2d8769b..06ed703 100644 --- a/src/catalog/builder.rs +++ b/src/catalog/builder.rs @@ -987,13 +987,7 @@ impl std::fmt::Display for BuildSummary { writeln!(f)?; writeln!(f, "Coverage:")?; - let pct = |n: usize, total: usize| { - if total == 0 { - 0 - } else { - (n * 100) / total - } - }; + let pct = |n: usize, total: usize| (n * 100).checked_div(total).unwrap_or(0); let check = |n: usize, total: usize| { if n == total { "+" diff --git a/src/catalog/index.rs b/src/catalog/index.rs index 1106169..54046f1 100644 --- a/src/catalog/index.rs +++ b/src/catalog/index.rs @@ -30,7 +30,7 @@ impl<'a> CandidateFinder<'a> { } let mut candidates: Vec<_> = ref_counts.into_iter().collect(); - candidates.sort_by(|a, b| b.1.cmp(&a.1)); // Sort by count descending + candidates.sort_by_key(|b| std::cmp::Reverse(b.1)); // Sort by count descending candidates } @@ -68,7 +68,7 @@ impl<'a> CandidateFinder<'a> { } let mut candidates: Vec<_> = ref_counts.into_iter().collect(); - candidates.sort_by(|a, b| b.1.cmp(&a.1)); + candidates.sort_by_key(|b| std::cmp::Reverse(b.1)); candidates }