commit 8fc6087670430363dbad5a45459a35ce48392497
author Brandon Kim <brandonkim@google.com> Fri Oct 18 15:15:50 2019 -0700
committer Brandon Kim <brandonkim@google.com> Fri Oct 18 16:07:36 2019 -0700
tree 1adec4e723def8541d9bb2fb9585fe8869939e8e
parent 6749ba1c93c9b4f65c9ef7791987fc2f98bc15d1

bmc: Replace fileOpen flag

Instead of keeping track of a fileOpen flag, keep track of fileOpen
using !lookup.empty() instead.

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

bmc/firmware_handler.hpp

diff --git a/bmc/firmware_handler.hpp b/bmc/firmware_handler.hpp
index 472bb8e..3a4b4be 100644
--- a/bmc/firmware_handler.hpp
+++ b/bmc/firmware_handler.hpp
@@ -210,6 +210,11 @@
                       blobIDs.end());
     }
 
+    inline bool fileOpen()
+    {
+        return !lookup.empty();
+    }
+
     ActionStatus getVerifyStatus();
     ActionStatus getActionStatus();
 
@@ -252,9 +257,6 @@
     bool preparationTriggered = false;
     ActionMap actionPacks;
 
-    /** Temporary variable to track whether a blob is open. */
-    bool fileOpen = false;
-
     ActionStatus lastVerificationStatus = ActionStatus::unknown;
 
     ActionStatus lastUpdateStatus = ActionStatus::unknown;