commit 8d5c9cecb6fda7cdbe96087bd8474cc7c94b58fc
author Patrick Venture <venture@google.com> Sat Mar 16 11:20:12 2019 -0700
committer Patrick Venture <venture@google.com> Mon Mar 25 08:42:18 2019 -0700
tree ed2602032d45ff599c8a941aa035277540534f68
parent ab650004954170fdcababf39c60a33657951d85f

util: move read_file to readPropertyFile

Move a method that reads a file and discards any trailing nul-terminator
found.  This is expected in dts property files.

Tested: Only ran unit-tests.
Change-Id: I4a551c0098862bcc1cfb7b20d518d7e2fa890a85
Signed-off-by: Patrick Venture <venture@google.com>

pcie_i2c.cpp

diff --git a/pcie_i2c.cpp b/pcie_i2c.cpp
index c0e7fc7..88eb66c 100644
--- a/pcie_i2c.cpp
+++ b/pcie_i2c.cpp
@@ -17,6 +17,7 @@
 #include "pcie_i2c.hpp"
 
 #include "main.hpp"
+#include "util.hpp"
 
 #include <cstdint>
 #include <cstring>
@@ -43,36 +44,6 @@
 
 std::vector<std::tuple<uint32_t, std::string>> pcie_i2c_map;
 
-std::string read_file(const std::string& file_name)
-{
-    std::ifstream ifs(file_name);
-    std::string file_content;
-    if (!ifs.is_open())
-    {
-        std::fprintf(stderr, "Unable to open file %s.\n", file_name.c_str());
-    }
-    else
-    {
-        if (ifs >> file_content)
-        {
-            // If the last character is a null terminator; remove it.
-            if (!file_content.empty())
-            {
-                char const& back = file_content.back();
-                if (back == '\0')
-                    file_content.pop_back();
-            }
-            return file_content;
-        }
-        else
-        {
-            std::fprintf(stderr, "Unable to read file %s.\n",
-                         file_name.c_str());
-        }
-    }
-    return "";
-}
-
 } // namespace
 
 struct PcieSlotCountRequest
@@ -128,7 +99,7 @@
             std::string pcie_slot_path = i2c_dev_path + "/of_node/pcie-slot";
             std::string pcie_slot;
             // Read the "pcie-slot" name from the "pcie-slot" file.
-            pcie_slot = read_file(pcie_slot_path);
+            pcie_slot = readPropertyFile(pcie_slot_path);
             if (pcie_slot.empty())
             {
                 continue;
@@ -140,7 +111,7 @@
             pcie_slot_full_path.append(pcie_slot);
             // Read the "label" which contains the pcie slot name.
             pcie_slot_full_path.append("/label");
-            pcie_slot_name = read_file(pcie_slot_full_path);
+            pcie_slot_name = readPropertyFile(pcie_slot_full_path);
             if (pcie_slot_name.empty())
             {
                 continue;