On 07/28/2014 07:13 PM, Brian Paul wrote: > On 07/22/2014 03:07 PM, Alon Levy wrote: >> Under VS2013 compiler that is distinguished by _MSC_VER of 1800 those two >> are already defined, use the SDK definition. >> >> Another option would have been to undef and use the definitions here - >> I'm >> not sure which is better. >> --- >> This removes warnings, this patch is not actually required to build. >> >> src/gallium/auxiliary/util/u_math.h | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/src/gallium/auxiliary/util/u_math.h >> b/src/gallium/auxiliary/util/u_math.h >> index ec03e4e..60995d7 100644 >> --- a/src/gallium/auxiliary/util/u_math.h >> +++ b/src/gallium/auxiliary/util/u_math.h >> @@ -136,10 +136,10 @@ roundf(float x) >> { >> return x >= 0.0f ? floorf(x + 0.5f) : ceilf(x - 0.5f); >> } >> -#endif >> >> #define INFINITY (DBL_MAX + DBL_MAX) >> #define NAN (INFINITY - INFINITY) >> +#endif >> >> #endif /* _MSC_VER */ >> >> > > This does not apply to the master branch. Neither does patch 3.
My bad, I was looking at an old branch. This is already fixed in master by 7ebdc9e48c99a92475b48668284695663e871f7d > > -Brian > > _______________________________________________ > mesa-dev mailing list > mesa-dev@lists.freedesktop.org > http://lists.freedesktop.org/mailman/listinfo/mesa-dev _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev