On 21 March 2017 at 15:50, Kurt Schwehr <[email protected]> wrote: > Hey Even, > > Probably not a big deal, but I'm curious why you didn't use !isinf(MinX) or > isfinite(MinX) for your recent OGREnvelope patch: > > int IsInit() const { return MinX != > std::numeric_limits<double>::infinity(); }
Portability wank: it's also good idea to check for std::numeric_limits<number<T> >::has_infinity eg. something like std::numeric_limits<number<double>::has_infinity ? std::numeric_limits<number<double>::infinity() : (std::numeric_limits<number<double>::max)() Best regards, -- Mateusz Loskot, http://mateusz.loskot.net _______________________________________________ gdal-dev mailing list [email protected] https://lists.osgeo.org/mailman/listinfo/gdal-dev
