Minor cleanup

Signed-off-by: Luca Della Vedova <[email protected]>

Author: luca-della-vedova

rclrs/src/dynamic_message.rs

@@ -474,7 +474,7 @@ impl DynamicMessage {
                 unsafe { std::slice::from_raw_parts_mut(dest_ptr, std::mem::size_of::<T>()) };
             // This creates a shallow copy, with ownership of the "deep" (or inner) parts moving
             // into the destination.
-            dest_slice.copy_from_slice(&*self.storage);
+            dest_slice.copy_from_slice(&self.storage);
             // Don't run the fini function on the src data anymore, because the inner parts would be
             // double-freed by dst and src.
             self.needs_fini = false;

rclrs/src/dynamic_message/message_structure.rs

@@ -1068,21 +1068,5 @@ mod tests {
                 );
             }
         }
-
-        // Explicitly drop to avoid clippy warnings
-        drop(arrays_structure);
-        drop(builtins_structure);
-        drop(duration_structure);
-        drop(empty_structure);
-        drop(time_structure);
-        drop(basic_types_structure);
-        drop(bounded_sequences_structure);
-        drop(constants_structure);
-        drop(multi_nested_structure);
-        drop(nested_structure);
-        drop(defaults_structure);
-        drop(strings_structure);
-        drop(wstrings_structure);
-        drop(unbounded_sequences_structure);
     }
 }