Re: [PATCH] D19323: Fix cast compiler warning message in include-fixer.

[Haojian Wu via cfe-commits]

This revision was automatically updated to reflect the committed changes.
Closed by commit rL266970: Fix cast compiler warning message in include-fixer. 
(authored by hokein).

Changed prior to commit:
  http://reviews.llvm.org/D19323?vs=54361=54473#toc

Repository:
  rL LLVM

http://reviews.llvm.org/D19323

Files:
  clang-tools-extra/trunk/include-fixer/tool/ClangIncludeFixer.cpp

Index: clang-tools-extra/trunk/include-fixer/tool/ClangIncludeFixer.cpp
===
--- clang-tools-extra/trunk/include-fixer/tool/ClangIncludeFixer.cpp
+++ clang-tools-extra/trunk/include-fixer/tool/ClangIncludeFixer.cpp
@@ -18,9 +18,8 @@
 
 static llvm::cl::OptionCategory tool_options("Tool options");
 
-int main(int argc, char **argv) {
-  clang::tooling::CommonOptionsParser options(argc, (const char **)argv,
-  tool_options);
+int main(int argc, const char **argv) {
+  clang::tooling::CommonOptionsParser options(argc, argv, tool_options);
   clang::tooling::ClangTool tool(options.getCompilations(),
  options.getSourcePathList());
   // Set up the data source.


Index: clang-tools-extra/trunk/include-fixer/tool/ClangIncludeFixer.cpp
===
--- clang-tools-extra/trunk/include-fixer/tool/ClangIncludeFixer.cpp
+++ clang-tools-extra/trunk/include-fixer/tool/ClangIncludeFixer.cpp
@@ -18,9 +18,8 @@
 
 static llvm::cl::OptionCategory tool_options("Tool options");
 
-int main(int argc, char **argv) {
-  clang::tooling::CommonOptionsParser options(argc, (const char **)argv,
-  tool_options);
+int main(int argc, const char **argv) {
+  clang::tooling::CommonOptionsParser options(argc, argv, tool_options);
   clang::tooling::ClangTool tool(options.getCompilations(),
  options.getSourcePathList());
   // Set up the data source.
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: [PATCH] D19323: Fix cast compiler warning message in include-fixer.

[Benjamin Kramer via cfe-commits]

bkramer accepted this revision.
bkramer added a comment.
This revision is now accepted and ready to land.

Looks good, thanks!


Repository:
  rL LLVM

http://reviews.llvm.org/D19323



___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits