commit a725706d167d79169f411105e32c317026ed4410
author Brad Bishop <bradbish@qti.qualcomm.com> Tue Jul 29 14:12:35 2025 -0400
committer Brad Bishop <bradbish@qti.qualcomm.com> Tue Nov 04 06:50:53 2025 -0500
tree 2bf8ceab56fa6a2cb4ea5149f748f483627f13a6
parent 9d2c63445cae145ae0cb7baab3f6b65d5040dfe5

tidy: fix readability-container-contains

Tested by building and running the unit tests.

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

src/associations.cpp

diff --git a/src/associations.cpp b/src/associations.cpp
index 880b382..fc2703f 100644
--- a/src/associations.cpp
+++ b/src/associations.cpp
@@ -259,7 +259,7 @@
         }
 
         // Can't create this association if the endpoint isn't on D-Bus.
-        if (interfaceMap.find(objectPath) == interfaceMap.end())
+        if (!interfaceMap.contains(objectPath))
         {
             addPendingAssociation(objectPath, reverse, path, forward, owner,
                                   assocMaps);
@@ -425,7 +425,7 @@
         return;
     }
 
-    if (interfaceMap.find(objectPath) == interfaceMap.end())
+    if (!interfaceMap.contains(objectPath))
     {
         return;
     }
@@ -437,8 +437,7 @@
         const auto& e = std::get<assocPos>(*endpoint);
 
         // Ensure the other side of the association still exists
-        if (interfaceMap.find(std::get<reversePathPos>(e)) ==
-            interfaceMap.end())
+        if (!interfaceMap.contains(std::get<reversePathPos>(e)))
         {
             endpoint++;
             continue;

src/handler.cpp

diff --git a/src/handler.cpp b/src/handler.cpp
index 902ae66..5575f2d 100644
--- a/src/handler.cpp
+++ b/src/handler.cpp
@@ -48,7 +48,7 @@
     {
         reqPath.pop_back();
     }
-    if (!reqPath.empty() && interfaceMap.find(reqPath) == interfaceMap.end())
+    if (!reqPath.empty() && !interfaceMap.contains(reqPath))
     {
         throw sdbusplus::xyz::openbmc_project::Common::Error::
             ResourceNotFound();