You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@datasketches.apache.org by al...@apache.org on 2019/06/26 19:02:13 UTC

[incubator-datasketches-cpp] branch serde_arithmetic updated: clarification in comment

This is an automated email from the ASF dual-hosted git repository.

alsay pushed a commit to branch serde_arithmetic
in repository https://gitbox.apache.org/repos/asf/incubator-datasketches-cpp.git


The following commit(s) were added to refs/heads/serde_arithmetic by this push:
     new 4c9dc5a  clarification in comment
4c9dc5a is described below

commit 4c9dc5ae9681bfe61b7eb20b3b1f29130305c5ff
Author: AlexanderSaydakov <Al...@users.noreply.github.com>
AuthorDate: Wed Jun 26 12:02:05 2019 -0700

    clarification in comment
---
 common/include/serde.hpp | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/common/include/serde.hpp b/common/include/serde.hpp
index 49961c3..1e27acd 100644
--- a/common/include/serde.hpp
+++ b/common/include/serde.hpp
@@ -37,7 +37,7 @@ template<typename T, typename Enable = void> struct serde {
 };
 
 // serde for all fixed-size arithmetic types (int and float of different sizes)
-// in particular, serde for signed 64-bit integers should produce sketches binary-compatible
+// in particular, kll_sketch<int64_t> should produce sketches binary-compatible
 // with LongsSketch and ItemsSketch<Long> with ArrayOfLongsSerDe in Java
 template<typename T>
 struct serde<T, typename std::enable_if<std::is_arithmetic<T>::value>::type> {


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@datasketches.apache.org
For additional commands, e-mail: commits-help@datasketches.apache.org