kiszk added inline comments.
================
Comment at:
clang-tools-extra/test/clang-tidy/checkers/fuchsia-multiple-inheritance.cpp:48
// Inherits from multiple concrete classes.
-// CHECK-MESSAGES: [[@LINE+2]]:1: warning: inheriting mulitple classes that
aren't pure virtual is discouraged [fuchsia-multiple-inheritance]
+// CHECK-MESSAGES: [[@LINE+2]]:1: warning: inheriting multiple classes that
aren't pure virtual is discouraged [fuchsia-multiple-inheritance]
// CHECK-NEXT: class Bad_Child1 : public Base_A, Base_B {};
----------------
sammccall wrote:
> Did you update the actual generated diagnostic in
> clang-tidy/fuchsia/MultipleInheritanceCheck.cpp? I don't see it in this patch.
Good catch. I also updated clang-tidy/fuchsia/MultipleInheritanceCheck.cpp.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D77458/new/
https://reviews.llvm.org/D77458
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
