fixed XXH3_128Stream so it initializes the state properly

the old version is still present for now, with a _deprecated suffix

1 files changed, 30 insertions, 1 deletions

diff --git a/src/zencore/include/zencore/xxhash.h b/src/zencore/include/zencore/xxhash.h
index 94d98be98..1616e5f93 100644
--- a/src/zencore/include/zencore/xxhash.h
+++ b/src/zencore/include/zencore/xxhash.h
@@ -61,8 +61,10 @@ struct XXH3_128
 
 struct XXH3_128Stream
 {
+	XXH3_128Stream() { Reset(); }
+
 	/// Begin streaming hash compute (not needed on freshly constructed instance)
-	void Reset() { memset(&m_State, 0, sizeof m_State); }
+	void Reset() { XXH3_128bits_reset(&m_State); }
 
 	/// Append another chunk
 	XXH3_128Stream& Append(const void* Data, size_t ByteCount)
@@ -83,6 +85,33 @@ struct XXH3_128Stream
 	}
 
 private:
+	XXH3_state_s m_State;
+};
+
+struct XXH3_128Stream_deprecated
+{
+	/// Begin streaming hash compute (not needed on freshly constructed instance)
+	void Reset() { memset(&m_State, 0, sizeof m_State); }
+
+	/// Append another chunk
+	XXH3_128Stream_deprecated& Append(const void* Data, size_t ByteCount)
+	{
+		XXH3_128bits_update(&m_State, Data, ByteCount);
+		return *this;
+	}
+
+	/// Append another chunk
+	XXH3_128Stream_deprecated& Append(MemoryView Data) { return Append(Data.GetData(), Data.GetSize()); }
+
+	/// Obtain final hash. If you wish to reuse the instance call reset()
+	XXH3_128 GetHash()
+	{
+		XXH3_128 Hash;
+		XXH128_canonicalFromHash((XXH128_canonical_t*)Hash.Hash, XXH3_128bits_digest(&m_State));
+		return Hash;
+	}
+
+private:
 	XXH3_state_s m_State{};
 };