refactor: Use util::Bytes::insert for span<const uint8_t>

diff --git a/src/core/CacheEntry.cpp b/src/core/CacheEntry.cpp
index 0009ecc7fa465ecf8e6f80265a6cfcba9f37f63a..4cfe6bee600900db832d30b4a6a3cb428ab711f2 100644 (file)
--- a/src/core/CacheEntry.cpp
+++ b/src/core/CacheEntry.cpp
@@ -285,7 +285,7 @@ CacheEntry::serialize(const CacheEntry::Header& header,
     [&payload](util::Bytes& result, const CacheEntry::Header& hdr) {
       switch (hdr.compression_type) {
       case CompressionType::none:
-        result.insert(result.end(), payload.begin(), payload.end());
+        result.insert(result.end(), payload);
         break;
 
       case CompressionType::zstd:

diff --git a/src/core/CacheEntryDataWriter.hpp b/src/core/CacheEntryDataWriter.hpp
index 19e91272c649868853241341840869e244fcd1da..6e18be522ee16ec74ec0990a0e3a2711cb9d702c 100644 (file)
--- a/src/core/CacheEntryDataWriter.hpp
+++ b/src/core/CacheEntryDataWriter.hpp
@@ -1,4 +1,4 @@
-// Copyright (C) 2022-2023 Joel Rosdahl and other contributors
+// Copyright (C) 2022-2024 Joel Rosdahl and other contributors
 //
 // See doc/AUTHORS.adoc for a complete list of contributors.
 //
@@ -57,7 +57,7 @@ inline CacheEntryDataWriter::CacheEntryDataWriter(util::Bytes& output)
 inline void
 CacheEntryDataWriter::write_bytes(nonstd::span<const uint8_t> data)
 {
-  m_output.insert(m_output.end(), data.begin(), data.end());
+  m_output.insert(m_output.end(), data);
 }
 
 template<typename T>

diff --git a/src/util/file.cpp b/src/util/file.cpp
index 87600422ff2f5d07646f3cfb2c24c3ca0323fa68..4ef39044171bd26b2b9652cb22a989c7ffb49f47 100644 (file)
--- a/src/util/file.cpp
+++ b/src/util/file.cpp
@@ -211,10 +211,7 @@ tl::expected<Bytes, std::string>
 read_fd(int fd)
 {
   Bytes output;
-  return read_fd(fd,
-                 [&](auto data) {
-                   output.insert(output.end(), data.begin(), data.end());
-                 })
+  return read_fd(fd, [&](auto data) { output.insert(output.end(), data); })
     .transform([&] { return output; });
 }