Overpass-API icon indicating copy to clipboard operation
Overpass-API copied to clipboard

Fix compile warning (and possible bug?)

Open puffetto opened this issue 4 years ago • 1 comments

Hi,

I noticed this compiler warning:

overpass_api/core/geometry.cc:1371:13: warning: implicit conversion turns floating-point number into bool: 'double' to 'bool' [-Wimplicit-conversion-floating-point-to-bool] if (::lon(uint32(value)<<16 > -180.)) ~~ ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ Shouldn't that line be:

if (::lon(uint32(value)<<16) > -180.) ?

I am sorry but I do not know the code of overpass well enough to understand and check what the issue causes and then ask a pull.

Cheers,

A.

puffetto avatar Nov 23 '21 09:11 puffetto

Fixed in fad5578

mmd-osm avatar Jan 16 '22 12:01 mmd-osm