No need to copy an empty object into another empty object.

2 files changed, 2 insertions, 3 deletions

diff --git a/messagebus/src/vespa/messagebus/routing/routingnode.cpp b/messagebus/src/vespa/messagebus/routing/routingnode.cpp
index f24afbc07ca..3a433b05bc5 100644
--- a/messagebus/src/vespa/messagebus/routing/routingnode.cpp
+++ b/messagebus/src/vespa/messagebus/routing/routingnode.cpp
@@ -184,7 +184,7 @@ RoutingNode::setReply(Reply::UP reply)
 {
     if (reply) {
         _shouldRetry = _resender != nullptr && _resender->shouldRetry(*reply);
-        if ( ! reply->getTrace().getRoot().isEmpty()) {
+        if ( ! reply->getTrace().isEmpty()) {
             _trace.getRoot().addChild(std::move(reply->getTrace().getRoot()));
             reply->getTrace().clear();
         }
diff --git a/messagebus/src/vespa/messagebus/sendproxy.cpp b/messagebus/src/vespa/messagebus/sendproxy.cpp
index c884932664b..fa485e19bb8 100644
--- a/messagebus/src/vespa/messagebus/sendproxy.cpp
+++ b/messagebus/src/vespa/messagebus/sendproxy.cpp
@@ -53,8 +53,7 @@ SendProxy::handleReply(Reply::UP reply)
         } else if (logger.wants(ns_log::Logger::spam)) {
             LOG(spam, "Trace for reply:\n%s", reply->getTrace().toString().c_str());
         }
-        Trace empty;
-        trace.swap(empty);
+        trace.clear();
     } else if (trace.getLevel() > 0) {
         trace.getRoot().addChild(reply->getTrace().getRoot());
         trace.getRoot().normalize();