<div dir="rtl"><div dir="ltr">Hi,</div><div dir="ltr"><br></div><div dir="ltr">1. Yes, already implemented per Nick suggestion.</div><div dir="ltr">2. Added.</div><div dir="ltr"><br></div><div dir="ltr">I would have blended the implementations if there were any code shared between the implementations. </div>

<div dir="ltr"><br></div><div dir="ltr">Since the the only code duplication is the function declarations I find the "Win32 code" and the "Unix code" being separate easier to read rather than seeing  #if-#else#-#endif in every function.</div>

<div dir="ltr"><br></div><div dir="ltr">Moreover the constructor declaration will not be the same due to the unused input argument in Win32.</div><div dir="ltr"><br></div><div dir="ltr">Yaron<br></div><div dir="ltr"><br>
</div>
<div dir="ltr"><br></div></div><div class="gmail_extra"><div dir="ltr"><br><br><div class="gmail_quote">2013/10/9 G M <span dir="ltr"><<a href="mailto:gmisocpp@gmail.com" target="_blank">gmisocpp@gmail.com</a>></span><br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div>Hi Yaron<br> <br>Your random_device patch looks good. I'd consider making a few more changes though:<br> <br>
1. I'd #if ! defined(_WIN32) the file handle f in the <random> header<br>
as the Win32 version's wont use it AFAIKT.<br>
 <br>2. Accordingly, I'd change random.cpp to remove the includes<br>relating to files for win32 to emphasis that further. i.e.:<br><br>#if ! defined(_WIN32)<br><br>#include <fcntl.h><br><br>#include <unistd.h><br>


<br>#endif</div><div> </div><div>I did wonder if it was worth blending the implementations a bit more to avoid replicating the function specifications etc. but on balance I'm not suggesting that but others might have an opinion.</div>


</blockquote></div></div></div>