whisperity added a comment.

I am a little bit conflicted about the //Severity// column. While I know our 
people put a great deal of effort into keeping this classification sane, what 
was put into CodeChecker is, at the end of the day, a pretty arbitrary 
classification.

I think RSTs support comments, right? Maybe it should be indicated in the code 
(as a comment only) where the severity was taken from, so if someone comes up 
with a reason to change a checker's severity, they could also submit a patch to 
CodeChecker's knowledge-base.

Maybe not //just// in the code comment, but also in the markup that gets 
rendered for the users, too?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D36051/new/

https://reviews.llvm.org/D36051



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

Reply via email to