fix: Default for Line/Location

Author: morrisonlevi

datadog-profiling/src/profiles/datatypes/location.rs

@@ -4,18 +4,28 @@
 use crate::profiles::collections::ParallelSet;
 use crate::profiles::datatypes::{OptionalFunctionId, OptionalMappingId};
 use std::ffi::c_void;
+use std::ptr::null_mut;
 
 /// A representation of a location that is an intersection of the Otel and
 /// Pprof representations. Omits some fields to save space because Datadog
 /// doesn't use them in any way. Additionally, Datadog only ever sets one Line,
 /// so it's not a Vec.
 #[repr(C)]
-#[derive(Copy, Clone, Debug, Default, Eq, PartialEq, Hash)]
+#[derive(Copy, Clone, Debug, Eq, PartialEq, Hash)]
 pub struct Location {
     pub address: u64,
     pub mapping_id: OptionalMappingId,
     pub line: Line,
 }
+impl Default for Location {
+    fn default() -> Location {
+        Location {
+            address: 0,
+            mapping_id: null_mut(),
+            line: Line::default(),
+        }
+    }
+}
 
 // Avoid NonNull<()> in FFI; see PR:
 // https://github.com/mozilla/cbindgen/pull/1098
@@ -24,10 +34,19 @@ pub type LocationId = std::ptr::NonNull<c_void>;
 /// A representation of a line plus function. It omits the column because it's
 /// not used by Datadog.
 #[repr(C)]
-#[derive(Copy, Clone, Debug, Default, Eq, PartialEq, Hash)]
+#[derive(Copy, Clone, Debug, Eq, PartialEq, Hash)]
 pub struct Line {
     pub line_number: i64,
     pub function_id: OptionalFunctionId,
 }
 
+impl Default for Line {
+    fn default() -> Line {
+        Line {
+            line_number: 0,
+            function_id: null_mut(),
+        }
+    }
+}
+
 pub type LocationSet = ParallelSet<Location, 4>;