VintfObject::Get*: move logging logic

... from fetchAllInformation to Get() functions, because logging
logic in headers may cause problems.

Test: manual
Bug: 71504062
Change-Id: I99d6e2728b24b4e40959a6cece71bd9b4625711f

1 files changed, 4 insertions, 2 deletions

diff --git a/check_vintf.cpp b/check_vintf.cpp
index 211702e..67ecd4e 100644
--- a/check_vintf.cpp
+++ b/check_vintf.cpp
@@ -25,9 +25,11 @@ namespace vintf {
 template <typename T>
 std::unique_ptr<T> readObject(const std::string& path, const XmlConverter<T>& converter) {
     std::string xml;
-    status_t err = details::gFetcher->fetch(path, xml);
+    std::string error;
+    status_t err = details::gFetcher->fetch(path, xml, &error);
     if (err != OK) {
-        std::cerr << "Error: Cannot read '" << path << "': " << strerror(-err) << std::endl;
+        std::cerr << "Error: Cannot read '" << path << "' (" << strerror(-err) << "): " << error
+                  << std::endl;
         return nullptr;
     }
     auto ret = std::make_unique<T>();