commit b3e3d64846fd41952d799e1e8c5d1c62a18f6d35
author Patrick Williams <patrick@stwcx.xyz> Wed Dec 18 11:20:22 2024 -0500
committer Patrick Williams <patrick@stwcx.xyz> Wed Dec 18 11:20:22 2024 -0500
tree 1016586b8960aa23cfb77c2a41212dd524084aa9
parent db3d966a75875914f630ff702a27f6db2bf63326

clang-format: re-format for clang-19

clang-format-19 isn't compatible with the clang-format-18 output, so we
need to reformat the code with the latest version.  A few parameters
in clang-tidy have been deprecated, so adjust the style file
accordingly.

See Ie2f6eb3b043f2d655c9df806815afd7971fd0947 for updated style.
See I88192b41ab7a95599a90915013579608af7bc56f for clang-19 enablement.

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

.clang-format

diff --git a/.clang-format b/.clang-format
index 28e3328..a75a2c2 100644
--- a/.clang-format
+++ b/.clang-format
@@ -17,9 +17,7 @@
 AllowShortIfStatementsOnASingleLine: Never
 AllowShortLambdasOnASingleLine: true
 AllowShortLoopsOnASingleLine: false
-AlwaysBreakAfterReturnType: None
 AlwaysBreakBeforeMultilineStrings: false
-AlwaysBreakTemplateDeclarations: Yes
 BinPackArguments: true
 BinPackParameters: true
 BitFieldColonSpacing: None
@@ -43,12 +41,14 @@
   SplitEmptyRecord:     false
   SplitEmptyNamespace:  false
 BreakAfterAttributes: Never
+BreakAfterReturnType: Automatic
 BreakBeforeBinaryOperators: None
 BreakBeforeBraces: Custom
 BreakBeforeTernaryOperators: true
 BreakConstructorInitializers: AfterColon
 BreakInheritanceList: AfterColon
 BreakStringLiterals: false
+BreakTemplateDeclarations: Yes
 ColumnLimit:     80
 CommentPragmas:  '^ IWYU pragma:'
 CompactNamespaces: false

update_manager.cpp

diff --git a/update_manager.cpp b/update_manager.cpp
index 6ced2ca..37bbb85 100644
--- a/update_manager.cpp
+++ b/update_manager.cpp
@@ -54,8 +54,8 @@
 // NOLINTNEXTLINE(readability-static-accessed-through-instance)
 auto Manager::processImage(sdbusplus::message::unix_fd image,
                            ApplyTimeIntf::RequestedApplyTimes applyTime,
-                           std::string id,
-                           std::string objPath) -> sdbusplus::async::task<>
+                           std::string id, std::string objPath)
+    -> sdbusplus::async::task<>
 {
     debug("Processing image {FD}", "FD", image.fd);
     fs::path tmpDirPath(std::string{IMG_UPLOAD_DIR});

update_manager.hpp

diff --git a/update_manager.hpp b/update_manager.hpp
index 5214335..59c185f 100644
--- a/update_manager.hpp
+++ b/update_manager.hpp
@@ -55,8 +55,8 @@
     /* @brief Process the image supplied via image fd */
     auto processImage(sdbusplus::message::unix_fd image,
                       ApplyTimeIntf::RequestedApplyTimes applyTime,
-                      std::string id,
-                      std::string objPath) -> sdbusplus::async::task<>;
+                      std::string id, std::string objPath)
+        -> sdbusplus::async::task<>;
 
     /* @brief The handler for the image processing failure  */
     void processImageFailed(sdbusplus::message::unix_fd image, std::string& id);