[PATCH] D44143: [clang-tidy] Create properly seeded random generator check

Eugene Zelenko via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Mar 13 11:28:03 PDT 2018


Eugene.Zelenko added inline comments.


================
Comment at: docs/ReleaseNotes.rst:63
+
+  Detects inappropriate seeding of C++ random generators and C srand function.
+
----------------
Please replace srand  with ``srand ()``.


================
Comment at: docs/clang-tidy/checks/cert-msc51-cpp.rst:7
+This check flags all pseudo-random number engines, engine adaptor
+instantiations and srand when initialized or seeded with default argument,
+constant expression or any user-configurable type. Pseudo-random number
----------------
aaron.ballman wrote:
> Backticks around `srand`
Two of them and please add (). Will be good idea to make first statement same as in Release Notes.


================
Comment at: docs/clang-tidy/checks/cert-msc51-cpp.rst:29
+  }
+
+Options
----------------
Is there guideline documentation available online? If so, please add link. See other checks documentation as example.


Repository:
  rCTE Clang Tools Extra

https://reviews.llvm.org/D44143





More information about the cfe-commits mailing list