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



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

Reply via email to