Remove TODO about renaming app_metrics field/parameter

The name is fine. When it was tried to be changed in #3636 we could not come up with anything better really. Signed-off-by: Patrick Roy <[email protected]>
firecracker-microvm · May 8, 2024 · 05f7195 · 05f7195
1 parent c60aed6
commit 05f7195
Showing 1 changed file with 0 additions and 2 deletions.
diff --git a/src/vmm/src/logger/metrics.rs b/src/vmm/src/logger/metrics.rs
@@ -94,8 +94,6 @@ pub struct Metrics<T: Serialize, M: Write + Send> {
 
 impl<T: Serialize + Debug, M: Write + Send + Debug> Metrics<T, M> {
  /// Creates a new instance of the current metrics.
- // TODO: We need a better name than app_metrics (something that says that these are the actual
- // values that we are writing to the metrics_buf).
  pub const fn new(app_metrics: T) -> Metrics<T, M> {
  Metrics {
  metrics_buf: OnceLock::new(),