don't panic, but return None

mozilla · Aug 20, 2024 · cca50a1 · cca50a1
1 parent 187f635
commit cca50a1
Showing 1 changed file with 4 additions and 10 deletions.
diff --git a/neqo-common/src/qlog.rs b/neqo-common/src/qlog.rs
@@ -153,7 +153,6 @@ impl Drop for NeqoQlogShared {
     }
 }
 
-#[allow(clippy::missing_panics_doc)]
 #[must_use]
 pub fn new_trace(role: Role) -> qlog::TraceSeq {
     TraceSeq {
@@ -174,15 +173,10 @@ pub fn new_trace(role: Role) -> qlog::TraceSeq {
         common_fields: Some(CommonFields {
             group_id: None,
             protocol_type: None,
-            reference_time: {
-                Some(
-                    SystemTime::now()
-                        .duration_since(SystemTime::UNIX_EPOCH)
-                        .expect("expect UNIX_EPOCH to always be earlier than now")
-                        .as_secs_f64()
-                        * 1_000.0,
-                )
-            },
+            reference_time: SystemTime::now()
+                .duration_since(SystemTime::UNIX_EPOCH)
+                .map(|d| d.as_secs_f64() * 1_000.0)
+                .ok(),
             time_format: Some("relative".to_string()),
         }),
     }