commit 00f39b84037a840a1b169a8017be75ac6636cb6f
author Brandon Kim <brandonkim@google.com> Fri Oct 18 11:00:00 2019 -0700
committer Brandon Kim <brandonkim@google.com> Fri Oct 18 11:00:00 2019 -0700
tree 210f2b655078db6e811f8454eb3c906145662413
parent cc4ef0c5a492968e6e7e7df7e41acbd599f0e23f

blob_handler: remove redundant returns

ClangTidy was not happy with redundant returns for void functions.

Signed-off-by: Brandon Kim <brandonkim@google.com>
Change-Id: I72f1773ea041f890ca0dd2c6c3ab8fbb279390c9

src/ipmiblob/blob_handler.cpp

diff --git a/src/ipmiblob/blob_handler.cpp b/src/ipmiblob/blob_handler.cpp
index d93f5c3..f7b0151 100644
--- a/src/ipmiblob/blob_handler.cpp
+++ b/src/ipmiblob/blob_handler.cpp
@@ -210,14 +210,13 @@
 void BlobHandler::writeMeta(std::uint16_t session, std::uint32_t offset,
                             const std::vector<std::uint8_t>& bytes)
 {
-    return writeGeneric(BlobOEMCommands::bmcBlobWriteMeta, session, offset,
-                        bytes);
+    writeGeneric(BlobOEMCommands::bmcBlobWriteMeta, session, offset, bytes);
 }
 
 void BlobHandler::writeBytes(std::uint16_t session, std::uint32_t offset,
                              const std::vector<std::uint8_t>& bytes)
 {
-    return writeGeneric(BlobOEMCommands::bmcBlobWrite, session, offset, bytes);
+    writeGeneric(BlobOEMCommands::bmcBlobWrite, session, offset, bytes);
 }
 
 std::vector<std::string> BlobHandler::getBlobList()
@@ -330,8 +329,6 @@
     {
         std::fprintf(stderr, "Received failure on close: %s\n", b.what());
     }
-
-    return;
 }
 
 void BlobHandler::deleteBlob(const std::string& id)
@@ -348,8 +345,6 @@
     {
         std::fprintf(stderr, "Received failure on delete: %s\n", b.what());
     }
-
-    return;
 }
 
 std::vector<std::uint8_t> BlobHandler::readBytes(std::uint16_t session,