💥 Rename Packet::clock to clock_secs

src/models/data/packet.rs

@@ -7,12 +7,21 @@ use serde_with::serde_as;
 use crate::models::data::payload::Payload;
 use crate::result::Result;
 
+/// «Packet» is a single traffic message in a replay,
+/// packets tell us what is going on during the battle.
 #[serde_as]
 #[derive(Debug, Serialize)]
 pub struct Packet {
-    pub clock: f32,
+    /// Replay time, starting from zero.
+    #[serde(rename = "clock")]
+    pub clock_secs: f32,
+
+    /// Parsed packet payload.
     pub payload: Payload,
 
+    /// Raw packet payload, excluding the clock and packet type, and including optional subtype.
+    ///
+    /// This attribute requires the `raw-payload` feature.
     #[cfg(feature = "raw-payload")]
     #[serde_as(as = "serde_with::hex::Hex")]
     pub raw_payload: Vec<u8>,
@@ -22,19 +31,19 @@ impl Packet {
     pub fn from_reader(reader: &mut impl Read) -> Result<Option<Self>> {
         let Ok(length) = reader.read_u32::<LittleEndian>() else { return Ok(None) };
         let type_ = reader.read_u32::<LittleEndian>()?;
-        let clock = reader.read_f32::<LittleEndian>()?;
-        let payload = Self::read_payload(reader, length as usize)?;
+        let clock_secs = reader.read_f32::<LittleEndian>()?;
+        let raw_payload = Self::read_raw_payload(reader, length as usize)?;
         let this = Self {
-            clock,
-            payload: Payload::new(type_, &payload)?,
+            clock_secs,
+            payload: Payload::new(type_, &raw_payload)?,
 
             #[cfg(feature = "raw-payload")]
-            raw_payload: payload,
+            raw_payload,
         };
         Ok(Some(this))
     }
 
-    fn read_payload(reader: &mut impl Read, length: usize) -> Result<Vec<u8>> {
+    fn read_raw_payload(reader: &mut impl Read, length: usize) -> Result<Vec<u8>> {
         let mut buffer = Vec::new();
         buffer.resize(length, 0);
         reader.read_exact(&mut buffer)?;