[flang-commits] [PATCH] D119015: [flang] Debugging of ACCESS='STREAM' I/O (take 2)

Michael Kruse via Phabricator via flang-commits flang-commits at lists.llvm.org
Fri Feb 4 10:51:20 PST 2022


Meinersbur added inline comments.


================
Comment at: flang/runtime/file.cpp:164
+#ifdef _WIN32
+  isWindowsTextFile_ = true;
+#endif
----------------
I have a concern here: If `fd` is `<=2`, (stdout or stderr), it will be in text mode, which expects a single `\n` for a newline, but with `isWindowsTextFile_ = true`, we will write `\r\n` (which the c-runtime expands to `\r\r\n`).


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D119015/new/

https://reviews.llvm.org/D119015



More information about the flang-commits mailing list