ltog / osmi-addresses

Calculates the Address view of the OSM Inspector
Boost Software License 1.0
11 stars 4 forks source link

Show more compiler warnings #62

Open ltog opened 8 years ago

ltog commented 8 years ago

In #61 we have a case of an uninitialized variable. This was not reported by compilers. Maybe we can adjust the Makefile to get more warnings?

ltog commented 8 years ago

First of all: g++ (and clang++) has the option -Wall but that doesn't enable all warnings...

clang++ has the option

-Weverything

which is supposed to really warn about everything. See http://clang.llvm.org/docs/UsersManual.html#diagnostics-enable-everything

To filter out some irrelevant warnings after using -Weverything, we can append:

-Wno-c++98-compat -Wno-c++98-compat-pedantic
ltog commented 8 years ago

-Weverything is ok, but provokes warnings from the GDAL library.

Some ideas to suppress those:

ltog commented 8 years ago

More warnings from g++: