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

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


The following commit(s) were added to refs/heads/master by this push:
     new 4014233  Fix count-min sketches test snapshots should use binary mode 
(#505)
4014233 is described below

commit 401423367055acdf7502e8ed3126730a08039d91
Author: tison <[email protected]>
AuthorDate: Thu Jun 25 23:02:50 2026 +0800

    Fix count-min sketches test snapshots should use binary mode (#505)
    
    Signed-off-by: tison <[email protected]>
---
 count/test/count_min_test.cpp | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/count/test/count_min_test.cpp b/count/test/count_min_test.cpp
index 8b7ae0a..64ffd4c 100644
--- a/count/test/count_min_test.cpp
+++ b/count/test/count_min_test.cpp
@@ -215,7 +215,7 @@ TEST_CASE("CountMin sketch: serialize-deserialize empty", 
"[cm_sketch]") {
     for (auto di: d) {
       REQUIRE(di == 0);
     }
-    std::ofstream os("count_min-empty.bin");
+    std::ofstream os("count_min_empty_cpp.sk", std::ios::binary);
     c.serialize(os);
 }
 
@@ -243,7 +243,7 @@ TEST_CASE("CountMin sketch: serialize-deserialize 
non-empty", "[cm_sketch]") {
     ++d_it;
   }
 
-  std::ofstream os("count_min-non-empty.bin");
+  std::ofstream os("count_min_non_empty_cpp.sk", std::ios::binary);
   c.serialize(os);
 }
 


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to