[PATCH] D18649: [clang-tidy] cppcoreguidelines-interfaces-global-init

Clement Courbet via cfe-commits cfe-commits at lists.llvm.org
Fri Apr 1 06:54:55 PDT 2016


courbet added a subscriber: etienneb.
courbet added a comment.

Looks like there's a race condition in phabricator.


================
Comment at: test/clang-tidy/cppcoreguidelines-interfaces-global-init.cpp:37
@@ +36,3 @@
+static int FromClassBadInit = takesInt(A::ClassScope);
+// CHECK-MESSAGES: [[@LINE-1]]:12: warning: initializing static variable with
+// non-const expression depending on static variable 'ClassScope'
----------------
alexfh wrote:
> These should be on a single line. I guess, your clang-format doesn't use -style=file by default and doesn't understand the local style configured for the test/ directory. Otherwise it wouldn't split the lines.
Thanks for the hint. I was using -style=LLVM indeed.


http://reviews.llvm.org/D18649





More information about the cfe-commits mailing list