[PATCH] D59963: [clang-tidy] Add a module for the Linux kernel.
Eugene Zelenko via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri Jul 26 09:54:58 PDT 2019
Eugene.Zelenko added inline comments.
================
Comment at: clang-tools-extra/docs/ReleaseNotes.rst:77
+
+ This is important in the Linux kernel because ``ERR_PTR``, ``PTR_ERR``,
+ ``IS_ERR``, ``IS_ERR_OR_NULL``, ``ERR_CAST``, and ``PTR_ERR_OR_ZERO`` return
----------------
tmroeder wrote:
> Eugene.Zelenko wrote:
> > Release Notes should include short description. One sentence is enough, but it'll good idea to keep it same as first statement of documentation.
> Would you like me to submit a patch that removes the second paragraph of this description, then?
Yes. Documentation is created for details. But you still need to make its first sentence same as in Release Notes. See other checks and their Release Notes as example (in previous release branches).
Repository:
rL LLVM
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D59963/new/
https://reviews.llvm.org/D59963
More information about the cfe-commits
mailing list