From 2bf405ad559a346577cc6b922fd45377c935fafb Mon Sep 17 00:00:00 2001 From: Tim Peters Date: Thu, 12 Oct 2000 19:42:00 +0000 Subject: [PATCH] Repaired a comment and asserted a precondition. --- Modules/mathmodule.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/Modules/mathmodule.c b/Modules/mathmodule.c index 09f7135077e..8c86c386f34 100644 --- a/Modules/mathmodule.c +++ b/Modules/mathmodule.c @@ -18,7 +18,7 @@ extern double modf (double, double *); #endif /* RED_FLAG 12-Oct-2000 Tim - * What CHECK does if errno != 0 and x is a NaN is a platform-dependent crap + * What CHECK does if errno == 0 and x is a NaN is a platform-dependent crap * shoot. Most (but not all!) platforms will end up setting errno to ERANGE * then, but EDOM is probably better. */ @@ -38,6 +38,7 @@ static int is_error(double x) { int result = 1; /* presumption of guilt */ + assert(errno); /* non-zero errno is a precondition for calling */ if (errno == EDOM) PyErr_SetString(PyExc_ValueError, "math domain error"); else if (errno == ERANGE) {