commit a79f6c8fa6bc8fec117f64d14ae7f62fc8b0d48b
author Zane Shelley <zshelle@us.ibm.com> Tue Jan 12 16:38:49 2021 -0600
committer Zane Shelley <zshelle@us.ibm.com> Tue Jan 12 16:38:49 2021 -0600
tree 4332c9f3f53b5d19b528860c5ff855904b4ce909
parent 9fb657f9bac9adf2bad14ebfc564b2660f5cd275

Clean up warnings from Jenkins CI

Signed-off-by: Zane Shelley <zshelle@us.ibm.com>
Change-Id: I6d9dd7909a4ec6c07c820fc3fa208226927c04c9

attn/attn_handler.cpp

diff --git a/attn/attn_handler.cpp b/attn/attn_handler.cpp
index caf5265..a4abf74 100644
--- a/attn/attn_handler.cpp
+++ b/attn/attn_handler.cpp
@@ -52,7 +52,6 @@
     std::vector<Attention> active_attentions;
 
     uint32_t isr_val, isr_mask;
-    uint32_t proc;
 
     std::stringstream ss; // for trace messages
 
@@ -64,7 +63,7 @@
     {
         if (PDBG_TARGET_ENABLED == pdbg_target_probe(target))
         {
-            proc = pdbg_target_index(target); // get processor number
+            auto proc = pdbg_target_index(target); // get processor number
 
             // Use PIB target to determine if a processor is enabled
             char path[16];

attn/attn_logging.cpp

diff --git a/attn/attn_logging.cpp b/attn/attn_logging.cpp
index 48ee5be..3aa0889 100644
--- a/attn/attn_logging.cpp
+++ b/attn/attn_logging.cpp
@@ -36,12 +36,7 @@
     createFFDCTuples(const std::vector<util::FFDCFile>& files)
 {
     std::vector<FFDCTuple> ffdcTuples{};
-    for (const util::FFDCFile& file : files)
-    {
-        ffdcTuples.emplace_back(
-            file.getFormat(), file.getSubType(), file.getVersion(),
-            sdbusplus::message::unix_fd(file.getFileDescriptor()));
-    }
+    util::transformFFDC(files, ffdcTuples);
 
     return ffdcTuples;
 }