commit c185afd6bec454ad86535f6c649ee498a6c6dd25
author Austin Schuh <austin.linux@gmail.com> Sun Oct 16 15:46:24 2022 -0700
committer Austin Schuh <austin.linux@gmail.com> Mon Oct 17 17:37:26 2022 -0700
tree df3078679db14f21920c7a00812e1a1d6594e933
parent 23d4115512dc21c966e1106cc58a08ed8503b943

Make crc32 use absl::Span<const uint8_t>

We had a const span, not a span of constant data.  Whops, fixed.

Change-Id: I746428365e9b119880806c64a661e371b3c0e9f8
Signed-off-by: Austin Schuh <austin.linux@gmail.com>

aos/util/crc32.cc

diff --git a/aos/util/crc32.cc b/aos/util/crc32.cc
index 7f13f30..0249c61 100644
--- a/aos/util/crc32.cc
+++ b/aos/util/crc32.cc
@@ -49,11 +49,11 @@
     0xb40bbe37, 0xc30c8ea1, 0x5a05df1b, 0x2d02ef8d};
 }
 
-uint32_t ComputeCrc32(const absl::Span<uint8_t> data) {
+uint32_t ComputeCrc32(const absl::Span<const uint8_t> data) {
   return AccumulateCrc32(data, std::nullopt);
 }
 
-uint32_t AccumulateCrc32(const absl::Span<uint8_t> data,
+uint32_t AccumulateCrc32(const absl::Span<const uint8_t> data,
                          std::optional<uint32_t> current_checksum) {
   uint32_t crc =
       current_checksum.has_value() ? current_checksum.value() : 0xFF'FF'FF'FF;

aos/util/crc32.h

diff --git a/aos/util/crc32.h b/aos/util/crc32.h
index b193f61..5f130d7 100644
--- a/aos/util/crc32.h
+++ b/aos/util/crc32.h
@@ -9,10 +9,10 @@
 
 namespace aos {
 
-uint32_t ComputeCrc32(const absl::Span<uint8_t> data);
+uint32_t ComputeCrc32(const absl::Span<const uint8_t> data);
 
 uint32_t AccumulateCrc32(
-    const absl::Span<uint8_t> data,
+    const absl::Span<const uint8_t> data,
     std::optional<uint32_t> current_checksum = std::nullopt);
 
 }  // namespace aos