From: Tim Peters <tim.peters@gmail.com>
Date: Thu, 12 Oct 2000 19:42:00 +0000 (+0000)
Subject: Repaired a comment and asserted a precondition.
X-Git-Tag: v2.0~74
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=2bf405ad559a346577cc6b922fd45377c935fafb;p=python

Repaired a comment and asserted a precondition.
---

diff --git a/Modules/mathmodule.c b/Modules/mathmodule.c
index 09f7135077..8c86c386f3 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) {