commit 3cbb6ef689c77cd5dae4dc449a1db6eb75a2dbe8
author Edward Lee <edwarddl@google.com> Wed Mar 15 17:08:40 2023 +0000
committer Edward Lee <edwarddl@google.com> Wed Mar 15 18:12:47 2023 +0000
tree 53bf89dbecadcab5705f817ca6340a32fcfa6ae3
parent aedea9fca37282f1490eed7c93027c8d1289ff41

Remove unused variables

Jenkins presubmit fails for all commits right now due to certain unused
variables. This commit aims to pass presubmit without regressing
functionality.

Tested: Presubmit passes

Change-Id: Ifdc7081b1154ce3ac2f3ab0828429660db35360a
Signed-off-by: Edward Lee <edwarddl@google.com>

src/pci_handler.cpp

diff --git a/src/pci_handler.cpp b/src/pci_handler.cpp
index b3e25bd..9ccb550 100644
--- a/src/pci_handler.cpp
+++ b/src/pci_handler.cpp
@@ -17,8 +17,8 @@
 
 PciDataHandler::PciDataHandler(uint32_t regionAddress, size_t regionSize,
                                std::unique_ptr<stdplus::fd::Fd> fd) :
-    regionAddress(regionAddress),
-    regionSize(regionSize), fd(std::move(fd)),
+    regionSize(regionSize),
+    fd(std::move(fd)),
     mmap(stdplus::fd::MMap(
         *this->fd, regionSize, stdplus::fd::ProtFlags{PROT_READ | PROT_WRITE},
         stdplus::fd::MMapFlags{stdplus::fd::MMapAccess::Shared}, regionAddress))

src/rde/external_storer_file.cpp

diff --git a/src/rde/external_storer_file.cpp b/src/rde/external_storer_file.cpp
index 26aa6d7..4e45993 100644
--- a/src/rde/external_storer_file.cpp
+++ b/src/rde/external_storer_file.cpp
@@ -46,8 +46,8 @@
 ExternalStorerFileInterface::ExternalStorerFileInterface(
     sdbusplus::bus_t& bus, std::string_view rootPath,
     std::unique_ptr<FileHandlerInterface> fileHandler) :
-    bus(bus),
-    rootPath(rootPath), fileHandler(std::move(fileHandler)), logServiceId(""),
+    rootPath(rootPath),
+    fileHandler(std::move(fileHandler)), logServiceId(""),
     cperNotifier(std::make_unique<CperFileNotifierHandler>(bus))
 {}