gribozavr added a comment.

> This suggestion would result another strange behavior: if the user disables 
> cert-err09-cpp because he or she doesn't want to see its reports, the other 
> one (cert-err61-cpp) will still report the issue. So he or she has to disable 
> both (or as many aliases it has).

That seems to be the case regardless of the implementation strategy in this 
patch.

> This could be another advantage of this patch, since it would be annoying to 
> see a message in which a GCC-style checker warns about an LLVM-style 
> violation.

I don't understand the scenario. I think people wouldn't enable both checkers 
in the first place.


Repository:
  rCTE Clang Tools Extra

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

https://reviews.llvm.org/D65065



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

Reply via email to