commit 78b0486412e0e296fac119be2e4c20fc040d3e7a
author Brad Bishop <bradbish@qti.qualcomm.com> Tue Jul 08 16:00:37 2025 -0400
committer Brad Bishop <bradbish@qti.qualcomm.com> Tue Nov 04 06:50:54 2025 -0500
tree 6751b337033ce78f3a74ec33d1a306f9a756ade3
parent a725706d167d79169f411105e32c317026ed4410

tidy: fix performance-inefficient-string-concatenation

Tested by building and running the unit tests.

Change-Id: Ia7638172abcd737d6036bb5b2b96d03fb0a85562
Signed-off-by: Brad Bishop <bradbish@qti.qualcomm.com>

src/handler.cpp

diff --git a/src/handler.cpp b/src/handler.cpp
index 5575f2d..89c9913 100644
--- a/src/handler.cpp
+++ b/src/handler.cpp
@@ -1,5 +1,6 @@
 #include "handler.hpp"
 
+#include "path.hpp"
 #include "types.hpp"
 
 #include <xyz/openbmc_project/Common/error.hpp>
@@ -427,7 +428,8 @@
     for (const auto& subtreePath : subtreePaths)
     {
         // Form the association path
-        std::string associationPathStr = subtreePath + "/" + association;
+        std::string associationPathStr =
+            appendPathSegment(subtreePath, association);
         sdbusplus::message::object_path associationPath(associationPathStr);
 
         auto associatedSubTree =
@@ -453,7 +455,8 @@
     for (const auto& subtreePath : subtreePaths)
     {
         // Form the association path
-        std::string associationPathStr = subtreePath + "/" + association;
+        std::string associationPathStr =
+            appendPathSegment(subtreePath, association);
         sdbusplus::message::object_path associationPath(associationPathStr);
 
         auto associatedSubTree = getAssociatedSubTreePaths(