[PATCH] D18914: [clang-tidy] new readability-redundant-inline

Matthias Gehre via cfe-commits cfe-commits at lists.llvm.org
Mon Apr 11 14:58:28 PDT 2016


mgehre added inline comments.

================
Comment at: test/clang-tidy/readability-redundant-inline.cpp:6
@@ +5,3 @@
+// CHECK-MESSAGES: :[[@LINE-1]]:14: warning: 'inline' is redundant because method body is defined inside class [readability-redundant-inline]
+// CHECK-FIXES: {{^}}  int f1()
+    return 0;
----------------
Personally, I never use "inline" to mean anything else than "multiple definitions can appear". I didn't know that
compilers still respected this.

Does that mean that the whole checker is useless?


http://reviews.llvm.org/D18914





More information about the cfe-commits mailing list