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 2020/07/10 00:46:14 UTC

[incubator-datasketches-cpp] 01/02: removed redundant check

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

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

commit 117c33d7e917d51f16940a3c736be59f33f8e9c3
Author: AlexanderSaydakov <Al...@users.noreply.github.com>
AuthorDate: Thu Jul 9 14:37:56 2020 -0700

    removed redundant check
---
 tuple/include/theta_update_sketch_base_impl.hpp | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tuple/include/theta_update_sketch_base_impl.hpp b/tuple/include/theta_update_sketch_base_impl.hpp
index 62a1e0f..60d4084 100644
--- a/tuple/include/theta_update_sketch_base_impl.hpp
+++ b/tuple/include/theta_update_sketch_base_impl.hpp
@@ -75,7 +75,7 @@ template<typename EN, typename EK, typename A>
 uint64_t theta_update_sketch_base<EN, EK, A>::hash_and_screen(const void* data, size_t length) {
   is_empty_ = false;
   const uint64_t hash = compute_hash(data, length, seed_);
-  if (hash >= theta_ || hash == 0) return 0; // hash == 0 is reserved to mark empty slots in the table
+  if (hash >= theta_) return 0; // hash == 0 is reserved to mark empty slots in the table
   return hash;
 }
 


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