NoQ added inline comments.
Comment at: www/analyzer/checker_dev_manual.html:708
+Checker Reviewer Checklist
+
I think we actually need two separate checklists:
* for common bugs (careful use of non-fatal error nodes, etc. - stuff we check
for on every review)
NoQ accepted this revision.
NoQ added a comment.
This revision is now accepted and ready to land.
Thx!
Comment at: www/analyzer/checker_dev_manual.html:703-704
Useful Links
+http://lcs.ios.ac.cn/~xuzb/canalyze/memmodel.pdf";>Xu, Zhongxing &
Maybe merge wit
NoQ accepted this revision.
NoQ added a comment.
This revision is now accepted and ready to land.
Thanks!
Comment at: www/analyzer/alpha_checks.html:450
-
-Variable Argument Alpha Checkers
-
I think these are missing on the available checkers list, so we should
NoQ accepted this revision.
NoQ added a comment.
This revision is now accepted and ready to land.
Yay, these look useful. Is there also an attribute for methods that should
never be called on a 'moved-from' object?
Repository:
rC Clang
https://reviews.llvm.org/D52983
_
Author: zinovy.nis
Date: Mon Oct 8 22:40:03 2018
New Revision: 344015
URL: http://llvm.org/viewvc/llvm-project?rev=344015&view=rev
Log:
[clang-tidy] The patch extends the existing command line option -check-suffix
(with alias -check-suffixes) to accept multiple comma-separated FileCheck
prefixe
This revision was automatically updated to reflect the committed changes.
Closed by commit rL344015: [clang-tidy] The patch extends the existing command
line option -check-suffix (authored by zinovy.nis, committed by ).
Herald added a subscriber: llvm-commits.
Changed prior to commit:
https://
Author: zinovy.nis
Date: Mon Oct 8 22:48:57 2018
New Revision: 344016
URL: http://llvm.org/viewvc/llvm-project?rev=344016&view=rev
Log:
[clang-tidy][docs] Update docs for `--check-suffixes`
Differential Revision: https://reviews.llvm.org/D52971
Modified:
clang-tools-extra/trunk/docs/clang-t
101 - 107 of 107 matches
Mail list logo