<div dir="rtl"><div dir="ltr">Hi,</div><div dir="ltr"><br></div><div dir="ltr">I fixed the issues.</div><div dir="ltr">__f_ data member will be used if !_WIN32 as a file descriptor so it's #ifdef-d.</div><div dir="ltr">

<br></div><div dir="ltr">Yaron</div><div dir="ltr"><br></div></div><div class="gmail_extra"><br><br><div class="gmail_quote"><div dir="ltr">2013/10/8 Nico Rieck <span dir="ltr"><<a href="mailto:nico.rieck@gmail.com" target="_blank">nico.rieck@gmail.com</a>></span></div>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"> #  endif<br>
+#  // Must be defined before stdlib.h is included to enable rand_s().<br>
+#  define _CRT_RAND_S<br>
 #endif  // _WIN32<br>
<br>
Since rand_s is never used in a libc++ header, I don't think it should be defined here.<br>
<br>
There are also trailing whitespace in the patch. The __f_ data member is unused and should be removed. (It also can't hold any HANDLE if that's the reason you kept it around.) And I see no need to duplicate entropy().<span class="HOEnZb"><font color="#888888"><br>


<br>
-Nico<br>
</font></span></blockquote></div><br></div>