Windows: use binary I/O everywhere

...because newlines matter, at least in the test suite, and because having a stable file format really helps. No LF/CRLF changes for me, please.
CESNET · Dec 16, 2021 · b1aa77f · b1aa77f
1 parent 13ed812
commit b1aa77f
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 10 deletions.
diff --git a/src/out.c b/src/out.c
@@ -329,7 +329,7 @@ ly_out_new_filepath(const char *filepath, struct ly_out **out)
     LY_CHECK_ERR_RET(!*out, LOGMEM(NULL), LY_EMEM);
 
     (*out)->type = LY_OUT_FILEPATH;
-    (*out)->method.fpath.f = fopen(filepath, "w");
+    (*out)->method.fpath.f = fopen(filepath, "wb");
     if (!(*out)->method.fpath.f) {
         LOGERR(NULL, LY_ESYS, "Failed to open file \"%s\" (%s).", filepath, strerror(errno));
         free(*out);
@@ -353,7 +353,7 @@ ly_out_filepath(struct ly_out *out, const char *filepath)
 
     /* replace filepath */
     f = out->method.fpath.f;
-    out->method.fpath.f = fopen(filepath, "w");
+    out->method.fpath.f = fopen(filepath, "wb");
     if (!out->method.fpath.f) {
         LOGERR(NULL, LY_ESYS, "Failed to open file \"%s\" (%s).", filepath, strerror(errno));
         out->method.fpath.f = f;

diff --git a/tests/utests/basic/test_inout.c b/tests/utests/basic/test_inout.c
@@ -83,8 +83,8 @@ test_input_file(void **UNUSED(state))
     assert_int_equal(LY_EINVAL, ly_in_new_file(NULL, NULL));
     assert_null(ly_in_file(NULL, NULL));
 
-    assert_int_not_equal(-1, f1 = fopen(__FILE__, "r"));
-    assert_int_not_equal(-1, f2 = fopen(__FILE__, "r"));
+    assert_int_not_equal(-1, f1 = fopen(__FILE__, "rb"));
+    assert_int_not_equal(-1, f2 = fopen(__FILE__, "rb"));
 
     assert_int_equal(LY_EINVAL, ly_in_new_file(f1, NULL));
 
@@ -203,8 +203,8 @@ test_output_file(void **UNUSED(state))
     char buf[31] = {0};
     const char *filepath = TESTS_BIN "/libyang_test_output";
 
-    assert_int_not_equal(-1, f1 = fopen(filepath, "w"));
-    assert_int_not_equal(-1, f2 = fopen(filepath, "w"));
+    assert_int_not_equal(-1, f1 = fopen(filepath, "wb"));
+    assert_int_not_equal(-1, f2 = fopen(filepath, "wb"));
 
     /* manipulate with the handler */
     assert_int_equal(LY_SUCCESS, ly_out_new_file(f1, &out));
@@ -218,8 +218,8 @@ test_output_file(void **UNUSED(state))
     ly_out_free(out, NULL, 1);
 
     /* writing data */
-    assert_int_not_equal(-1, f1 = fopen(filepath, "w"));
-    assert_int_not_equal(-1, f2 = fopen(filepath, "r"));
+    assert_int_not_equal(-1, f1 = fopen(filepath, "wb"));
+    assert_int_not_equal(-1, f2 = fopen(filepath, "rb"));
 
     assert_int_equal(LY_SUCCESS, ly_out_new_file(f1, &out));
     assert_int_equal(LY_SUCCESS, ly_print(out, "test %s", "print"));
@@ -260,7 +260,7 @@ test_output_filepath(void **UNUSED(state))
     ly_out_free(out, NULL, 1);
 
     /* writing data */
-    assert_int_not_equal(-1, f1 = fopen(fp1, "r"));
+    assert_int_not_equal(-1, f1 = fopen(fp1, "rb"));
 
     assert_int_equal(LY_SUCCESS, ly_out_new_filepath(fp1, &out));
     assert_int_equal(LY_SUCCESS, ly_print(out, "test %s", "print"));

diff --git a/tools/re/main.c b/tools/re/main.c
@@ -146,7 +146,7 @@ main(int argc, char *argv[])
                 fprintf(stderr, "yangre error: command line patterns cannot be mixed with file input.\n");
                 goto cleanup;
             }
-            infile = fopen(optarg, "r");
+            infile = fopen(optarg, "rb");
             if (!infile) {
                 fprintf(stderr, "yangre error: unable to open input file %s (%s).\n", optarg, strerror(errno));
                 goto cleanup;