Eugene.Zelenko added inline comments.

================
Comment at: docs/ReleaseNotes.rst:78
+
+  Finds and fixes usage of random_shuffle as the function has been removed 
from C++17.
+
----------------
Please add std:: and enclose random_shuffle in ``.


================
Comment at: docs/clang-tidy/checks/modernize-replace-random-shuffle.rst:6
+
+This check will find occurences of ``random_shuffle`` and replace it with 
``shuffle``. In C++17 ``random_shuffle`` will no longer be available and thus 
we need to replace it.
+
----------------
I think std:: should be added.


https://reviews.llvm.org/D30158



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to