[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