[PATCH] libc++: Add support for arc4random() to random_device.

JF Bastien jfb at chromium.org
Mon Mar 9 20:21:27 PDT 2015


The changes LGTM with the Windows implementation fix.

Please update the commit message to indicate that this is an API break on Windows but it's probably inconsequential: any string used to be accepted, and now only the default string in the `random_device` ctor is accepted.


REPOSITORY
  rL LLVM

================
Comment at: src/random.cpp:122
@@ -83,4 +121,3 @@
 
-random_device::random_device(const string& __token)
-    : __f_(open(__token.c_str(), O_RDONLY))
+random_device::random_device(const string&)
 {
----------------
`__token`?

http://reviews.llvm.org/D8134

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the cfe-commits mailing list