commit b487eb47e566df7679e740559582f6d25597feba
author Willy Tu <wltu@google.com> Thu Sep 16 21:44:43 2021 -0700
committer Willy Tu <wltu@google.com> Fri Sep 17 08:44:25 2021 -0700
tree 90e95ebcba64e326f7c47d61acf55535efde581c
parent fd16f6dbb0ca64263ed30e6ce18d0c0d53e51bbe

phosphor-ipmi-flash: Fix all C++ warnings

Change-Id: I89236b2dab88ca725e269bb8f5b2cbfd271e807a
Signed-off-by: Willy Tu <wltu@google.com>

bmc/version-handler/version_handler.cpp

diff --git a/bmc/version-handler/version_handler.cpp b/bmc/version-handler/version_handler.cpp
index f2ab881..0efdcf7 100644
--- a/bmc/version-handler/version_handler.cpp
+++ b/bmc/version-handler/version_handler.cpp
@@ -82,12 +82,12 @@
 /**
  * deleteBlob - does nothing, always fails
  */
-bool VersionBlobHandler::deleteBlob(const std::string& path)
+bool VersionBlobHandler::deleteBlob(const std::string&)
 {
     return false;
 }
 
-bool VersionBlobHandler::stat(const std::string& path, blobs::BlobMeta* meta)
+bool VersionBlobHandler::stat(const std::string&, blobs::BlobMeta*)
 {
     return false;
 }

bmc/version-handler/version_handler.hpp

diff --git a/bmc/version-handler/version_handler.hpp b/bmc/version-handler/version_handler.hpp
index 4a3d7d4..b0b9b52 100644
--- a/bmc/version-handler/version_handler.hpp
+++ b/bmc/version-handler/version_handler.hpp
@@ -49,17 +49,15 @@
               const std::string& path) override;
     std::vector<uint8_t> read(uint16_t session, uint32_t offset,
                               uint32_t requestedSize) override;
-    bool write(uint16_t session, uint32_t offset,
-               const std::vector<uint8_t>& data) override
+    bool write(uint16_t, uint32_t, const std::vector<uint8_t>&) override
     {
         return false; /* not supported */
     };
-    bool writeMeta(uint16_t session, uint32_t offset,
-                   const std::vector<uint8_t>& data) override
+    bool writeMeta(uint16_t, uint32_t, const std::vector<uint8_t>&) override
     {
         return false; /* not supported */
     }
-    bool commit(uint16_t session, const std::vector<uint8_t>& data) override
+    bool commit(uint16_t, const std::vector<uint8_t>&) override
     {
         return false; // not supported
     }