commit 93b6a33320ab0494d4732c1190c6821ec7149cac
author George Liu <liuxiwei@ieisystem.com> Mon Aug 19 08:30:52 2024 +0800
committer George Liu <liuxiwei@ieisystem.com> Mon Aug 19 08:30:52 2024 +0800
tree 7392133901b33da5e100afc7135398201fa0f75d
parent 0917ac423df7a3ae700bbb21fac32f540cb58682

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: I00e5377ec69a75c00c0c620dc6ac395c99fbc812
Signed-off-by: George Liu <liuxiwei@ieisystem.com>

.clang-format

diff --git a/.clang-format b/.clang-format
index d7e0961..27681a9 100644
--- a/.clang-format
+++ b/.clang-format
@@ -87,7 +87,7 @@
 IndentWrappedFunctionNames: true
 InsertNewlineAtEOF: true
 KeepEmptyLinesAtTheStartOfBlocks: false
-LambdaBodyIndentation: OuterScope
+LambdaBodyIndentation: Signature
 LineEnding: LF
 MacroBlockBegin: ''
 MacroBlockEnd:   ''
@@ -98,13 +98,14 @@
 ObjCSpaceBeforeProtocolList: true
 PackConstructorInitializers: BinPack
 PenaltyBreakAssignment: 25
-PenaltyBreakBeforeFirstCallParameter: 19
+PenaltyBreakBeforeFirstCallParameter: 50
 PenaltyBreakComment: 300
 PenaltyBreakFirstLessLess: 120
 PenaltyBreakString: 1000
+PenaltyBreakTemplateDeclaration: 10
 PenaltyExcessCharacter: 1000000
 PenaltyReturnTypeOnItsOwnLine: 60
-PenaltyIndentedWhitespace: 0
+PenaltyIndentedWhitespace: 1
 PointerAlignment: Left
 QualifierAlignment: Left
 ReferenceAlignment: Left

src/asset_info.cpp

diff --git a/src/asset_info.cpp b/src/asset_info.cpp
index af03fce..8f38d6a 100644
--- a/src/asset_info.cpp
+++ b/src/asset_info.cpp
@@ -13,11 +13,11 @@
 {
 constexpr auto FIRMWARE_TYPE_OFFSET = 0;
 constexpr auto FIRMWARE_TYPE_SIZE = 1;
-constexpr auto FIRMWARE_VERSION_OFFSET = FIRMWARE_TYPE_OFFSET +
-                                         FIRMWARE_TYPE_SIZE;
+constexpr auto FIRMWARE_VERSION_OFFSET =
+    FIRMWARE_TYPE_OFFSET + FIRMWARE_TYPE_SIZE;
 constexpr auto FIRMWARE_VERSION_SIZE = 15;
-constexpr auto FIRMWARE_BUILDTIME_OFFSET = FIRMWARE_VERSION_OFFSET +
-                                           FIRMWARE_VERSION_SIZE;
+constexpr auto FIRMWARE_BUILDTIME_OFFSET =
+    FIRMWARE_VERSION_OFFSET + FIRMWARE_VERSION_SIZE;
 constexpr auto FIRMWARE_BUILDTIME_SIZE = 20;
 constexpr auto FIRMWARE_MIN_SIZE = FIRMWARE_BUILDTIME_OFFSET;
 
@@ -176,11 +176,10 @@
            buildTime.c_str());
 }
 
-ipmi_ret_t ipmiOemIEIAssetInfo(ipmi_netfn_t /* netfn */, ipmi_cmd_t /* cmd */,
-                               ipmi_request_t request,
-                               ipmi_response_t /* response */,
-                               ipmi_data_len_t /* data_len */,
-                               ipmi_context_t /* context */)
+ipmi_ret_t ipmiOemIEIAssetInfo(
+    ipmi_netfn_t /* netfn */, ipmi_cmd_t /* cmd */, ipmi_request_t request,
+    ipmi_response_t /* response */, ipmi_data_len_t /* data_len */,
+    ipmi_context_t /* context */)
 {
     auto header = reinterpret_cast<AssetInfoHeader*>(request);
 

src/firmware_version.cpp

diff --git a/src/firmware_version.cpp b/src/firmware_version.cpp
index fced513..85a28b6 100644
--- a/src/firmware_version.cpp
+++ b/src/firmware_version.cpp
@@ -56,9 +56,8 @@
     BIOS = 0x03,
 };
 
-RspType<std::string> ipmiIEIGetFWVersion(ipmi::Context::ptr ctx,
-                                         uint8_t fwIndex,
-                                         uint8_t /* devIndex */)
+RspType<std::string> ipmiIEIGetFWVersion(
+    ipmi::Context::ptr ctx, uint8_t fwIndex, uint8_t /* devIndex */)
 {
     std::string fwPath{};
     std::string value{};