Alternative (to r216344) fix of gcc -Wpedantic.
authorPatrik Hagglund <patrik.h.hagglund@ericsson.com>
Thu, 18 Sep 2014 11:52:57 +0000 (11:52 +0000)
committerPatrik Hagglund <patrik.h.hagglund@ericsson.com>
Thu, 18 Sep 2014 11:52:57 +0000 (11:52 +0000)
As suggested by David Blaikie, this may be easier to read.

The original warning was:

../tools/llvm-cov/llvm-cov.cpp:53:49: error: ISO C++ forbids zero-size array 'argv' [-Werror=pedantic]
       std::string Invocation(std::string(argv[0]) + " " + argv[1]);

It seems to be the case that GCC's warning gets confused and thinks
'argv' is a declaration here. GCC bugzilla issue #61259.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@218048 91177308-0d34-0410-b5e6-96231b3b80d8

tools/llvm-cov/llvm-cov.cpp

index 0a540b7a929f3e8a9940868a960efc9f3946fb31..8c1ad8bedb3f37b5da698bbc37382c9cee4a60d5 100644 (file)
@@ -50,7 +50,7 @@ int main(int argc, const char **argv) {
       func = gcov_main;
 
     if (func) {
-      std::string Invocation(std::string() + argv[0] + " " + argv[1]);
+      std::string Invocation = std::string(argv[0]) + " " + argv[1];
       argv[1] = Invocation.c_str();
       return func(argc - 1, argv + 1);
     }