From: John H. <jo...@gl...> - 2017-06-11 17:09:54
|
Here is the Debian crash fix patch without any 0x7FFFFFFF changes and an new No32bit patch that uses std::numeric_limits<int>::max()/min(). -- john On Sun, Jun 11, 2017 at 9:39 AM John Horigan <jo...@gl...> wrote: > Hi Jerry. I avoided std::numeric_limits<> because I thought that it was a > C++11 feature. But I checked and it isn't. I will create new patches > replacing the macros with std:: namespace constants. > > -- john > > On Sun, Jun 11, 2017 at 6:23 AM Jerry Evans <je...@no...> wrote: > >> Hello John >> >> Good to see someone grabbing the horse. Thank you. >> >> >> -- agg24No32bitIntAssumption.diff I noticed that AGG is littered with >> the constant 0x7FFFFFFF, INT_MAX for 32-bit integers. This patch replaces >> all of them with the INT_MAX/INT_MIN macros. >> >> Suggest these might be better replaced with the C++ standards in <limits> >> ? >> >> BR. >> >> >> >> >> ------------------------------------------------------------------------------ >> Check out the vibrant tech community on one of the world's most >> engaging tech sites, Slashdot.org! http://sdm.link/slashdot >> _______________________________________________ >> Vector-agg-general mailing list >> Vec...@li... >> https://lists.sourceforge.net/lists/listinfo/vector-agg-general >> > |