search

ARPA-SIMC / meteosatlib

OpenMTP/HRI/HRIT C++ access libraries, gdal drivers and other tools
GNU General Public License v2.0
8 stars 2 forks source link

[Fedora 24] misleading indentation in grib/grib.cpp #5

Closed brancomat closed 7 years ago

brancomat commented 7 years ago 
libtool: compile:  g++ -DHAVE_CONFIG_H -I.. -I.. -Werror -I/usr/include/gdal -I/usr/include/gdal -I/usr/include/gdal -O2 -g -pipe -Wall -Werror=format-security -Wp,-D_FORTIFY_SOURCE=2 -fexceptions -fstack-protector-strong --param=ssp-buffer-size=4 -grecord-gcc-switches -specs=/usr/lib/rpm/redhat/redhat-hardened-cc1 -m64 -mtune=generic -Wall -O2 -g -pipe -Wall -Werror=format-security -Wp,-D_FORTIFY_SOURCE=2 -fexceptions -fstack-protector-strong --param=ssp-buffer-size=4 -grecord-gcc-switches -specs=/usr/lib/rpm/redhat/redhat-hardened-cc1 -m64 -mtune=generic -c grib/grib.cpp  -fPIC -DPIC -o grib/.libs/libmsatdrv_la-grib.o
/bin/sh ../libtool  --tag=CXX   --mode=compile g++ -DHAVE_CONFIG_H   -I.. -I.. -Werror -I/usr/include/gdal   -I/usr/include/gdal   -I/usr/include/gdal  -O2 -g -pipe -Wall -Werror=format-security -Wp,-D_FORTIFY_SOURCE=2 -fexceptions -fstack-protector-strong --param=ssp-buffer-size=4 -grecord-gcc-switches -specs=/usr/lib/rpm/redhat/redhat-hardened-cc1 -m64 -mtune=generic  -Wall -O2 -g -pipe -Wall -Werror=format-security -Wp,-D_FORTIFY_SOURCE=2 -fexceptions -fstack-protector-strong --param=ssp-buffer-size=4 -grecord-gcc-switches -specs=/usr/lib/rpm/redhat/redhat-hardened-cc1 -m64 -mtune=generic -c -o netcdf/libmsatdrv_la-utils.lo `test -f 'netcdf/utils.cpp' || echo './'`netcdf/utils.cpp
grib/grib.cpp: In constructor 'msat::grib::GRIBRasterBand::GRIBRasterBand(msat::grib::GRIBDataset*, int)':
grib/grib.cpp:162:5: error: this 'if' clause does not guard... [-Werror=misleading-indentation]
     if (!grib.get_long_ifexists("channelNumber", &channel_id))
     ^~
grib/grib.cpp:172:2: note: ...this statement, but the latter is misleadingly indented as if it is guarded by the 'if'
  char buf[25];
  ^~~~
libtool: compile:  g++ -DHAVE_CONFIG_H -I.. -I.. -Werror -I/usr/include/gdal -I/usr/include/gdal -I/usr/include/gdal -O2 -g -pipe -Wall -Werror=format-security -Wp,-D_FORTIFY_SOURCE=2 -fexceptions -fstack-protector-strong --param=ssp-buffer-size=4 -grecord-gcc-switches -specs=/usr/lib/rpm/redhat/redhat-hardened-cc1 -m64 -mtune=generic -Wall -O2 -g -pipe -Wall -Werror=format-security -Wp,-D_FORTIFY_SOURCE=2 -fexceptions -fstack-protector-strong --param=ssp-buffer-size=4 -grecord-gcc-switches -specs=/usr/lib/rpm/redhat/redhat-hardened-cc1 -m64 -mtune=generic -c netcdf/utils.cpp  -fPIC -DPIC -o netcdf/.libs/libmsatdrv_la-utils.o
libtool: compile:  g++ -DHAVE_CONFIG_H -I.. -I.. -Werror -I/usr/include/gdal -I/usr/include/gdal -I/usr/include/gdal -O2 -g -pipe -Wall -Werror=format-security -Wp,-D_FORTIFY_SOURCE=2 -fexceptions -fstack-protector-strong --param=ssp-buffer-size=4 -grecord-gcc-switches -specs=/usr/lib/rpm/redhat/redhat-hardened-cc1 -m64 -mtune=generic -Wall -O2 -g -pipe -Wall -Werror=format-security -Wp,-D_FORTIFY_SOURCE=2 -fexceptions -fstack-protector-strong --param=ssp-buffer-size=4 -grecord-gcc-switches -specs=/usr/lib/rpm/redhat/redhat-hardened-cc1 -m64 -mtune=generic -c netcdf/utils.cpp -o netcdf/libmsatdrv_la-utils.o >/dev/null 2>&1
cc1plus: all warnings being treated as errors
Makefile:669: recipe for target 'grib/libmsatdrv_la-grib.lo' failed
make[2]: *** [grib/libmsatdrv_la-grib.lo] Error 1
brancomat commented 7 years ago

fixed in dfb33a757eaee010d0ba3d5eb5a975cb046a4e57