================
Comment at: test/support/platform_support.h:55
@@ +54,3 @@
+
+class TempFileName {
+public:
----------------
It's actually a TempFile, not a TempFileName, since you leave the fd open. I 
think what you actually wanted was to mkstemp(), stash the name and then close 
the fd (before leaving the constructor).

================
Comment at: test/support/platform_support.h:76
@@ -58,2 +75,3 @@
 #endif
-   return s;
+   std::remove(Name.c_str());
+  }
----------------
I must be missing something here... std::remove() takes three arguments.

http://reviews.llvm.org/D4962



_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to