commit 543ac9f16ff63c9d18d9d2aab5c9a142e66c19c9
author Patrick Williams <patrick@stwcx.xyz> Fri Aug 16 15:19:59 2024 -0400
committer Patrick Williams <patrick@stwcx.xyz> Fri Aug 16 15:19:59 2024 -0400
tree 73c0148319b170d50ae6b10e69ffc4b3755f8a8f
parent 4a435516a856324839e047607be8ef53855ddc8a

clang-format: re-format for clang-18

clang-format-18 isn't compatible with the clang-format-17 output, so we
need to reformat the code with the latest version.  The way clang-18
handles lambda formatting also changed, so we have made changes to the
organization default style format to better handle lambda formatting.

See I5e08687e696dd240402a2780158664b7113def0e for updated style.
See Iea0776aaa7edd483fa395e23de25ebf5a6288f71 for clang-18 enablement.

Change-Id: I47f3c815913ae34be19ce0d823fa233ed698d1a0
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>

manager/json-config.hpp

diff --git a/manager/json-config.hpp b/manager/json-config.hpp
index eadf4b4..64441b3 100644
--- a/manager/json-config.hpp
+++ b/manager/json-config.hpp
@@ -70,17 +70,17 @@
      */
     bool filePathExists(const std::vector<std::string>& names)
     {
-        auto it = std::find_if(names.begin(), names.end(),
-                               [this](const auto& name) {
-            auto configFileName = name + ".json";
-            auto configFilePath = fs::path{confBasePath} / configFileName;
-            if (fs::exists(configFilePath))
-            {
-                confFile = configFilePath;
-                return true;
-            }
-            return false;
-        });
+        auto it =
+            std::find_if(names.begin(), names.end(), [this](const auto& name) {
+                auto configFileName = name + ".json";
+                auto configFilePath = fs::path{confBasePath} / configFileName;
+                if (fs::exists(configFilePath))
+                {
+                    confFile = configFilePath;
+                    return true;
+                }
+                return false;
+            });
         return it == names.end() ? false : true;
     }
 
@@ -160,8 +160,8 @@
         try
         {
             // Get all objects implementing the compatible interface
-            auto objects = dBusHandler.getSubTreePaths("/",
-                                                       confCompatibleInterface);
+            auto objects =
+                dBusHandler.getSubTreePaths("/", confCompatibleInterface);
             for (const auto& path : objects)
             {
                 try

manager/led-main.cpp

diff --git a/manager/led-main.cpp b/manager/led-main.cpp
index 47d97dc..a18c793 100644
--- a/manager/led-main.cpp
+++ b/manager/led-main.cpp
@@ -80,9 +80,9 @@
     /** Now create so many dbus objects as there are groups */
     std::ranges::transform(systemLedMap, std::back_inserter(groups),
                            [&bus, &manager, serializePtr](auto& grp) {
-        return std::make_unique<phosphor::led::Group>(bus, grp.first, manager,
-                                                      serializePtr);
-    });
+                               return std::make_unique<phosphor::led::Group>(
+                                   bus, grp.first, manager, serializePtr);
+                           });
 
     // Attach the bus to sd_event to service user requests
     bus.attach_event(event.get(), SD_EVENT_PRIORITY_NORMAL);

manager/manager.hpp

diff --git a/manager/manager.hpp b/manager/manager.hpp
index caa7ba8..4e611b8 100644
--- a/manager/manager.hpp
+++ b/manager/manager.hpp
@@ -83,8 +83,8 @@
     Manager(
         sdbusplus::bus_t& bus, const GroupMap& ledLayout,
         const sdeventplus::Event& event = sdeventplus::Event::get_default()) :
-        ledMap(ledLayout),
-        bus(bus), timer(event, [this](auto&) { driveLedsHandler(); })
+        ledMap(ledLayout), bus(bus),
+        timer(event, [this](auto&) { driveLedsHandler(); })
     {
         // Nothing here
     }