From efea6e3c6449c9488547be3222d80365607620b7 Mon Sep 17 00:00:00 2001
From: Graham McIntire <graham@mcintire.me>
Date: Tue, 13 Jan 2026 13:47:24 -0600
Subject: [PATCH] Fix clippy warning: use self by value for Copy type

Change to_unix_timestamp to take self by value instead of &self
since Timestamp is Copy
---
 src/api_client.rs  | 1 -
 src/metrics/mod.rs | 2 +-
 2 files changed, 1 insertion(+), 2 deletions(-)

diff --git a/src/api_client.rs b/src/api_client.rs
index 5170749..74e0397 100644
--- a/src/api_client.rs
+++ b/src/api_client.rs
@@ -2,7 +2,6 @@ use crate::config::{AgentConfig, HeartbeatMetadata};
 use crate::metrics::Metric;
 use crate::proto::agent;
 use prost::Message;
-use serde_json::json;
 use std::time::Duration;
 use thiserror::Error;
 
diff --git a/src/metrics/mod.rs b/src/metrics/mod.rs
index 8db631f..f19d391 100644
--- a/src/metrics/mod.rs
+++ b/src/metrics/mod.rs
@@ -21,7 +21,7 @@ impl Timestamp {
         Self::now().secs - self.secs
     }
 
-    pub fn to_unix_timestamp(&self) -> i64 {
+    pub fn to_unix_timestamp(self) -> i64 {
         self.secs
     }