[diagnostics-bridge] Get rid of some unnecessary cloning
Bug: 67960
Change-Id: I8beff26a5f45ca9d6ad45cd4091bcb39cc369d19
Reviewed-on: https://fuchsia-review.googlesource.com/c/fuchsia/+/476223
Fuchsia-Auto-Submit: Miguel Flores <miguelfrde@google.com>
Reviewed-by: Jordon Wing <jwing@google.com>
Commit-Queue: Miguel Flores <miguelfrde@google.com>
diff --git a/src/developer/remote-control/diagnostics-bridge/src/lib.rs b/src/developer/remote-control/diagnostics-bridge/src/lib.rs
index 2781912..5542347 100644
--- a/src/developer/remote-control/diagnostics-bridge/src/lib.rs
+++ b/src/developer/remote-control/diagnostics-bridge/src/lib.rs
@@ -68,8 +68,7 @@
}
};
- let (truncated_logs, truncated_chars) = match truncate_log_msg(logs.clone())
- {
+ let (truncated_logs, truncated_chars) = match truncate_log_msg(logs) {
Ok(t) => t,
Err(err) => {
warn!(%err, "failed to truncate log message");
@@ -139,14 +138,14 @@
logs.payload = logs.payload.map(|p| {
let props = p
.properties
- .iter()
+ .into_iter()
.map(|prop| {
if *prop.key() == LogsField::Msg {
let new_msg = truncate_to_char_boundary(&msg, MAX_DATAGRAM_LEN_BYTES as usize);
return LogsProperty::String(LogsField::Msg, new_msg.to_string());
}
- prop.clone()
+ prop
})
.collect();
LogsHierarchy::new(p.name, props, p.children)