long_mul(): Simplified exit code. In particular, k_mul() returns a

normalized result, so no point to normalizing it again.  The number
of test+branches was also excessive.

1 files changed, 3 insertions, 9 deletions

diff --git a/Objects/longobject.c b/Objects/longobject.c
index 348dcc4..2343db8 100644
--- a/Objects/longobject.c
+++ b/Objects/longobject.c
@@ -1878,18 +1878,12 @@ long_mul(PyLongObject *v, PyLongObject *w)
 	}
 
 	z = k_mul(a, b);
-	if(z == NULL) {
-		Py_DECREF(a);
-		Py_DECREF(b);
-		return NULL;
-	}
-	if (a->ob_size < 0)
-		z->ob_size = -(z->ob_size);
-	if (b->ob_size < 0)
+	/* Negate if exactly one of the inputs is negative. */
+	if (((a->ob_size ^ b->ob_size) < 0) && z)
 		z->ob_size = -(z->ob_size);
 	Py_DECREF(a);
 	Py_DECREF(b);
-	return (PyObject *) long_normalize(z);
+	return (PyObject *)z;
 }
 
 /* The / and % operators are now defined in terms of divmod().