[gdal-dev] What warnings in r38007 require casting int const to double?

[Even Rouault]

On mercredi 17 mai 2017 07:50:23 CEST Kurt Schwehr wrote:
> Ok. I will try reverting just those casts today and if it break travis, I
> will put them back with a comment as to why

I don't think this will break Travis since that comes from changesets you introduced days/
weeks before, so we would have noticed. I likely noticed when compiling in some VM. I should 
have noted the compiler. I don't think this was a compilation error, but just a warning about 
loss of precision.

> 
> On May 17, 2017 7:34 AM, "Even Rouault" <even.rouault at spatialys.com> wrote:
> > On mercredi 17 mai 2017 07:13:39 CEST Kurt Schwehr wrote:
> > > +gdal-dev
> > > 
> > > On Wed, May 17, 2017 at 7:12 AM, Kurt Schwehr <schwehr at gmail.com> wrote:
> > > > Hi Even,
> > > > 
> > > > 
> > > > 
> > > > Do you remember what warning by which compiler with this changeset?
> > 
> > Hum of course not :-) It might be some older compiler/distribution I
> > tested before releasing 2.2.
> > 
> > 
> > 
> > I don't see indeed in the part you quote below why a compiler would
> > complain.
> > 
> > 
> > 
> > But in the changeset the change related to GUIntBig is legitimate since
> > big values of 64 bit integers cannot be exactly represented as a 64 bit
> > double.
> > 
> > 
> > 
> > Even
> > 
> > 
> > 
> > --
> > 
> > Spatialys - Geospatial professional services
> > 
> > http://www.spatialys.com


-- 
Spatialys - Geospatial professional services
http://www.spatialys.com
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osgeo.org/pipermail/gdal-dev/attachments/20170517/af71489c/attachment.html>