[clang-tools-extra] [clang-tidy] Add option to keep virtual in 'modernize-use-override' (PR #144916)
Baranov Victor via cfe-commits
cfe-commits at lists.llvm.org
Thu Jun 19 09:51:49 PDT 2025
================
@@ -0,0 +1,25 @@
+// RUN: %check_clang_tidy %s modernize-use-override %t -- \
+// RUN: -config="{CheckOptions: {modernize-use-override.AllowVirtual: true}}"
+
+struct Base {
+ virtual ~Base();
+ virtual void a();
+ virtual void b();
+ virtual void c();
+};
+
+struct Derived : public Base {
+ virtual ~Derived() override;
+
+ virtual void a() override;
+ // CHECK-MESSAGES-NOT: warning:
+ // CHECK-FIXES: {{^}} virtual void a() override;
+
+ virtual void b();
+ // CHECK-MESSAGES: :[[@LINE-1]]:16: warning: add 'override'
+ // CHECK-FIXES: {{^}} virtual void b() override;
----------------
vbvictor wrote:
don't use `{{^}}`, just match string as a whole
https://github.com/llvm/llvm-project/pull/144916
More information about the cfe-commits
mailing list