From d2259f22386e88df8da5cac9fa4895869366ffe2 Mon Sep 17 00:00:00 2001
From: Joel Rosdahl <joel@rosdahl.net>
Date: Mon, 24 Jul 2023 13:51:42 +0200
Subject: [PATCH] refactor: Improve util::read_file usage

---
 src/storage/remote/FileStorage.cpp | 11 +++++------
 1 file changed, 5 insertions(+), 6 deletions(-)

diff --git a/src/storage/remote/FileStorage.cpp b/src/storage/remote/FileStorage.cpp
index ca18dd9f6..da68921b5 100644
--- a/src/storage/remote/FileStorage.cpp
+++ b/src/storage/remote/FileStorage.cpp
@@ -128,12 +128,11 @@ FileStorageBackend::get(const Hash::Digest& key)
     util::set_timestamps(path);
   }
 
-  auto value = util::read_file<util::Bytes>(path);
-  if (!value) {
-    LOG("Failed to read {}: {}", path, value.error());
-    return tl::unexpected(Failure::error);
-  }
-  return std::move(*value);
+  return util::read_file<util::Bytes>(path).transform_error(
+    [&](const auto& error) {
+      LOG("Failed to read {}: {}", path, error);
+      return Failure::error;
+    });
 }
 
 tl::expected<bool, RemoteStorage::Backend::Failure>
-- 
2.47.2