CRC: Make crc32c_t as a class for explicit control of operators

The motivation is to explicitly remove and document dangerous
operations like adding crc32c_t to a set, because equality is not
enough to guarantee uniqueness.

PiperOrigin-RevId: 491656425
Change-Id: I7b4dadc1a59ea9861e6ec7a929d64b5746467832

1 files changed, 2 insertions, 2 deletions

diff --git a/absl/crc/internal/crc_memcpy.h b/absl/crc/internal/crc_memcpy.h
index 8e728a6e..ae9cccad 100644
--- a/absl/crc/internal/crc_memcpy.h
+++ b/absl/crc/internal/crc_memcpy.h
@@ -40,7 +40,7 @@ class CrcMemcpy {
  public:
   static crc32c_t CrcAndCopy(void* __restrict dst, const void* __restrict src,
                              std::size_t length,
-                             crc32c_t initial_crc = ToCrc32c(0),
+                             crc32c_t initial_crc = crc32c_t{0},
                              bool non_temporal = false) {
     static const ArchSpecificEngines engines = GetArchSpecificEngines();
     auto* engine = non_temporal ? engines.non_temporal : engines.temporal;
@@ -100,7 +100,7 @@ class CrcNonTemporalMemcpyAVXEngine : public CrcMemcpyEngine {
 // the generic fallback version.
 inline crc32c_t Crc32CAndCopy(void* __restrict dst, const void* __restrict src,
                               std::size_t length,
-                              crc32c_t initial_crc = ToCrc32c(0),
+                              crc32c_t initial_crc = crc32c_t{0},
                               bool non_temporal = false) {
   return CrcMemcpy::CrcAndCopy(dst, src, length, initial_crc, non_temporal);
 }