[PATCH] Move unit test into anonymous namespace as per convention.

Douglas Katzman dougk at google.com
Fri May 1 10:46:44 PDT 2015


Hi joerg,

Current placement seems to have been inadvertent.

http://reviews.llvm.org/D9438

Files:
  unittests/ADT/TripleTest.cpp

Index: unittests/ADT/TripleTest.cpp
===================================================================
--- unittests/ADT/TripleTest.cpp
+++ unittests/ADT/TripleTest.cpp
@@ -694,7 +694,6 @@
     EXPECT_STREQ("cortex-a8", Triple.getARMCPUForArch("arm"));
   }
 }
-}
 
 TEST(TripleTest, NormalizeARM) {
   EXPECT_EQ("armv6--netbsd-eabi", Triple::normalize("armv6-netbsd-eabi"));
@@ -712,3 +711,5 @@
   T = Triple("armv6eb--netbsd-eabi");
   EXPECT_EQ(Triple::armeb, T.getArch());
 }
+
+} // end anonymous namespace

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D9438.24819.patch
Type: text/x-patch
Size: 513 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20150501/a8471983/attachment.bin>


More information about the llvm-commits mailing list