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

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

commit af4436280bdab53e0063268e92ff29b3fdcb1b07
Author: tison <[email protected]>
AuthorDate: Thu Jun 25 02:42:14 2026 +0800

    Fix count-min sketches test snapshots should use binary mode
    
    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..5ce0685 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.bin", 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.bin", std::ios::binary);
   c.serialize(os);
 }
 


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

Reply via email to