remove warning for dead code that isn't turned on normally (#73)

* remove warning for code that isn't turned on normally * rearrange to avoid triggering dead code warning
microsoft · May 7, 2024 · 407a296 · 407a296
1 parent 280bae5
commit 407a296
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 18 deletions.
diff --git a/src/metrics.rs b/src/metrics.rs
@@ -64,16 +64,8 @@ pub struct Metrics {
     cpu_time_worker_time: Duration,
 }
 
-pub trait ModelStatsCollector {
-    fn record_compression_stats(
-        &mut self,
-        cmp: ModelComponent,
-        total_bits: i64,
-        total_compressed: i64,
-    );
-}
-
-impl ModelStatsCollector for Metrics {
+impl Metrics {
+    #[allow(dead_code)]
     fn record_compression_stats(
         &mut self,
         cmp: ModelComponent,
@@ -87,9 +79,7 @@ impl ModelStatsCollector for Metrics {
         e.total_bits += total_bits;
         e.total_compressed += total_compressed;
     }
-}
 
-impl Metrics {
     pub fn record_cpu_worker_time(&mut self, duration: Duration) {
         self.cpu_time_worker_time += duration;
     }

diff --git a/src/structs/vpx_bool_reader.rs b/src/structs/vpx_bool_reader.rs
@@ -26,9 +26,6 @@ use std::io::{Read, Result};
 
 use crate::metrics::{Metrics, ModelComponent};
 
-#[cfg(feature = "compression_stats")]
-use crate::metrics::ModelStatsCollector;
-
 use super::{branch::Branch, simple_hash::SimpleHash};
 
 const BITS_IN_BYTE: i32 = 8;

diff --git a/src/structs/vpx_bool_writer.rs b/src/structs/vpx_bool_writer.rs
@@ -26,9 +26,6 @@ use std::io::{Result, Write};
 
 use crate::metrics::{Metrics, ModelComponent};
 
-#[cfg(feature = "compression_stats")]
-use crate::metrics::ModelStatsCollector;
-
 use super::{branch::Branch, simple_hash::SimpleHash};
 
 pub struct VPXBoolWriter<W> {