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



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

Reply via email to