[PATCH] D18424: [Clang] Fix Clang-tidy modernize-deprecated-headers warnings; other minor fixes
Eugene Zelenko via cfe-commits
cfe-commits at lists.llvm.org
Tue Apr 5 13:32:35 PDT 2016
Eugene.Zelenko added inline comments.
================
Comment at: include/clang-c/Index.h:19
@@ -18,1 +18,3 @@
+#ifdef __cplusplus
+#include <ctime>
----------------
aaron.ballman wrote:
> Is this produced by the deprecated headers check? If not, what value does ctime add over time.h?
Yes, since file is include from C and C++.
================
Comment at: lib/Lex/ModuleMap.cpp:1286
@@ -1284,3 +1285,3 @@
};
-}
+} // end anonymous namespaces
----------------
aaron.ballman wrote:
> namespace instead of namespaces
Will fix in commit.
Repository:
rL LLVM
http://reviews.llvm.org/D18424
More information about the cfe-commits
mailing list