Message157779
I was going to add a test for this to Lib/test/test_math.py, but found this comment:
# copysign(INF, NAN) may be INF or it may be NINF, since
# we don't know whether the sign bit of NAN is set on any
# given platform.
I would try to claim this is fixable, by this patch:
--- Include\pymath.h.orig Sun Apr 08 10:02:37 2012
+++ Include\pymath.h Sun Apr 08 10:02:41 2012
@@ -150,7 +150,7 @@
* doesn't support NaNs.
*/
#if !defined(Py_NAN) && !defined(Py_NO_NAN)
-#define Py_NAN (Py_HUGE_VAL * 0.)
+#define Py_NAN abs(Py_HUGE_VAL * 0.)
#endif
/* Py_OVERFLOWED(X)
Should I rework the tests to reflect this and submit a patch? |
|
Date |
User |
Action |
Args |
2012-04-08 07:10:28 | mattip | set | recipients:
+ mattip, loewis |
2012-04-08 07:10:28 | mattip | set | messageid: <1333869028.12.0.0177458850406.issue14521@psf.upfronthosting.co.za> |
2012-04-08 07:10:27 | mattip | link | issue14521 messages |
2012-04-08 07:10:27 | mattip | create | |
|