WIP: Address TracerPayload clippy errors

sidecar/src/service/sidecar_server.rs

@@ -16,7 +16,6 @@ use datadog_ipc::platform::{AsyncChannel, ShmHandle};
 use datadog_ipc::tarpc;
 use datadog_ipc::tarpc::context::Context;
 use datadog_ipc::transport::Transport;
-use datadog_trace_protobuf::pb;
 use datadog_trace_utils::trace_utils::SendData;
 use datadog_trace_utils::tracer_payload;
 use datadog_trace_utils::tracer_payload::TraceEncoding;
@@ -313,8 +312,7 @@ impl SidecarServer {
                 error!(
                     "Failed to collect trace chunks from msgpack with error {:?}",
                     e
-                );
-                return;
+                )
             }
         }
     }

trace-utils/src/tracer_payload.rs

@@ -105,6 +105,8 @@ impl TracerPayloadCollection {
     }
 }
 
+pub type TracerPayloadChunkProcessor<'a> = Box<dyn Fn(&mut pb::TraceChunk, usize) + 'a>;
+
 /// Represents the parameters required to collect trace chunks from msgpack data.
 ///
 /// This struct encapsulates all the necessary parameters for converting msgpack data into
@@ -124,7 +126,7 @@ pub struct TracerPayloadParams<'a> {
     tracer_header_tags: &'a TracerHeaderTags<'a>,
     /// A boxed closure that processes each `TraceChunk`, allowing for custom logic to be applied
     /// during the conversion process.
-    process_chunk: Box<dyn Fn(&mut pb::TraceChunk, usize) + 'a>,
+    process_chunk: TracerPayloadChunkProcessor<'a>,
     /// A boolean indicating whether the agent is running in an agentless mode. This is used to
     /// determine what protocol trace chunks should be serialized into when being sent.
     is_agentless: bool,