<div><div dir="auto">He does add “b” to the mode of fopen as the workaround.  The problem is that he already had opened with open(), which doesn’t take a mide string, only mode flags.  So  in order to use fopen, which i hypothesize is because open didn’t have _O_BINARY, it had to be closed first and reopened with “b”.</div><div dir="auto"><br></div><div dir="auto">Point is, if you use _O_BINARY with open(), then “b” becomes irrelevant/unnecessary </div><br><div class="gmail_quote"><div>On Tue, Oct 17, 2017 at 7:22 AM Joerg Sonnenberger <<a href="mailto:joerg@bec.de">joerg@bec.de</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Tue, Oct 17, 2017 at 02:17:15PM +0000, Zachary Turner via llvm-commits wrote:<br>
> Did you confirm this fixes it? Because i was just speculating :)<br>
<br>
...and if it does, adding "b" to the mode would result in the same<br>
behavior for fopen.<br>
<br>
Joerg<br>
</blockquote></div></div>