commit ca041a8af5bc72eff0ac165229d7e3129f23fbfc
author Pavithra Barithaya <pavithrabarithaya07@gmail.com> Thu Aug 29 10:11:54 2024 +0530
committer Pavithra Barithaya <pavithrabarithaya07@gmail.com> Thu Aug 29 10:33:40 2024 +0530
tree d9b099b7c0a67083fc614c20803e1c868873eb4a
parent 90f28d7b91a8909ca17eb7874593f02c8f61aa1f

clang-tidy: Enable bugprone-branch-clone check

This checks for repeated branches in if/else if/else chains,
consecutive repeated branches in switch statements and identical
true and false branches in conditional operators.

Change-Id: I8776742c25f3f8c97617976ec4f6fc799aa249ae
Signed-off-by: Pavithra Barithaya <pavithrabarithaya07@gmail.com>

.clang-tidy

diff --git a/.clang-tidy b/.clang-tidy
index 5b44e8d..ce3d01c 100644
--- a/.clang-tidy
+++ b/.clang-tidy
@@ -3,6 +3,7 @@
 bugprone-assert-side-effect,
 bugprone-bad-signal-to-kill-thread,
 bugprone-bool-pointer-implicit-conversion,
+bugprone-branch-clone,
 bugprone-copy-constructor-init,
 bugprone-dangling-handle,
 bugprone-dynamic-static-initializers,

pldmtool/pldm_platform_cmd.cpp

diff --git a/pldmtool/pldm_platform_cmd.cpp b/pldmtool/pldm_platform_cmd.cpp
index 6a07a38..0a96274 100644
--- a/pldmtool/pldm_platform_cmd.cpp
+++ b/pldmtool/pldm_platform_cmd.cpp
@@ -1560,8 +1560,6 @@
                 printNumericSensorPDR(data, respCnt, output);
                 break;
             case PLDM_SENSOR_AUXILIARY_NAMES_PDR:
-                printAuxNamePDR(data, output);
-                break;
             case PLDM_EFFECTER_AUXILIARY_NAMES_PDR:
                 printAuxNamePDR(data, output);
                 break;