Search code examples
c++gcctype-conversiongcc-warning

gcc warning flags for implicit conversions


I recently had a bug in a similar context to next one:

double getSomeValue()
{
    return 4.0;
}
...
std::string str;
str = getSomeValue();

As you can see here is easy to spot the problem, but in a large code base where getSomeValue() is not in the same file with the calling code it might be difficult to spot this double to std::string silent conversion. GCC compiles this code fine with -Wall -Wextra -Werror (sample output here, I don't know what warning flags were used: http://ideone.com/BTXBFk).

How may I force GCC to emit warnings for these dangerous implicit conversions? I tried -Wconversion, but it is very strict and it causes errors in most included headers for common cases like unsigned - 1. Is there a weaker version of -Wconversion?


Solution

  • You can use the -Wfloat-conversion flag, or the broader -Wconversion.

    However, note that with C++11 uniform initialization brace syntax, you get a warning "out of the box", without the -Wconversion flag; e.g.:

    #include <string>
    
    double getSomeValue() {
        return 4.0;
    }
    
    int main() {   
        std::string str{ getSomeValue() }; // C++11 brace-init
    }
    
    C:\Temp\CppTests>g++ -std=c++11 test.cpp
    test.cpp: In function 'int main()':
    test.cpp:8:35: warning: narrowing conversion of 'getSomeValue()' from 'double' t
    o 'char' inside { } [-Wnarrowing]
         std::string str{ getSomeValue() };
                                       ^