<div dir="ltr"><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>
</div>