[PATCH] D44143: [clang-tidy] Create properly seeded random generator check
Aaron Ballman via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed Jul 4 18:23:01 PDT 2018
aaron.ballman closed this revision.
aaron.ballman added a comment.
In https://reviews.llvm.org/D44143#1152550, @boga95 wrote:
> How can I commit it?
You need to have obtained commit access first. I went ahead and committed on your behalf in r336301. Thank you for the patch!
================
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
----------------
boga95 wrote:
> aaron.ballman wrote:
> > Please add double backticks around `srand()` instead of single backticks.
> Should I use double backticks in release notes too?
Good catch; I'll add those when I commit.
https://reviews.llvm.org/D44143
More information about the cfe-commits
mailing list