Szelethus added inline comments.
================
Comment at:
clang/include/clang/StaticAnalyzer/Core/BugReporter/BugReporter.h:108
+
+ StringRef getDescription() const { return Description; }
+
----------------
gribozavr wrote:
> What's the difference between description and short description?
>
> What are the expected grammatical conventions? For example, Clang and
> ClangTidy warning are sentence fragments that start with a lowercase letter
> and don't end with a period.
I just did some work related to this! For 99% of the cases (and for one, I'm
pretty sure this estimate is quite accurate) there is no difference. AFAIK
CodeChecker doesnt even use the short description field emitted in the plist
output.
There is one checker however that uses it, RetainCount, though I dislike that
particular case (the short version is something like "the counter here isn't
zero", and the full version is "the counter here is +2"). Could we get rid of
this?
As for diagnostic messages, we start with uppercase but dont terminate the
sentence.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D66572/new/
https://reviews.llvm.org/D66572
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
