r205931 - Switching back from a static namespace-scoped function to a static class-scope function. No functional change, but resolves a warning.

Aaron Ballman aaron at aaronballman.com
Wed Apr 9 14:40:14 PDT 2014


Author: aaronballman
Date: Wed Apr  9 16:40:14 2014
New Revision: 205931

URL: http://llvm.org/viewvc/llvm-project?rev=205931&view=rev
Log:
Switching back from a static namespace-scoped function to a static class-scope function. No functional change, but resolves a warning.

Modified:
    cfe/trunk/include/clang/Analysis/Analyses/ThreadSafetyTIL.h

Modified: cfe/trunk/include/clang/Analysis/Analyses/ThreadSafetyTIL.h
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/Analysis/Analyses/ThreadSafetyTIL.h?rev=205931&r1=205930&r2=205931&view=diff
==============================================================================
--- cfe/trunk/include/clang/Analysis/Analyses/ThreadSafetyTIL.h (original)
+++ cfe/trunk/include/clang/Analysis/Analyses/ThreadSafetyTIL.h Wed Apr  9 16:40:14 2014
@@ -264,12 +264,12 @@ private:
 
 
 // Contains various helper functions for SExprs.
-namespace ThreadSafetyTIL {
-static bool isTrivial(SExpr *E) {
-  unsigned Op = E->opcode();
-  return Op == COP_Variable || Op == COP_Literal || Op == COP_LiteralPtr;
-}
-}
+struct ThreadSafetyTIL {
+  static bool isTrivial(SExpr *E) {
+    unsigned Op = E->opcode();
+    return Op == COP_Variable || Op == COP_Literal || Op == COP_LiteralPtr;
+  }
+};
 
 class Function;
 class SFunction;





More information about the cfe-commits mailing list