Revert "bpo-29941: Assert fixes (#886)"

Yhg1s · web-flow · commit 27058ede21f6 · 2017-03-31T09:37:40.000-07:00
This reverts commit a00c3fd.
diff --git a/Include/unicodeobject.h b/Include/unicodeobject.h
@@ -2313,10 +2313,6 @@ PyAPI_FUNC(Py_UNICODE*) PyUnicode_AsUnicodeCopy(
 PyAPI_FUNC(int) _PyUnicode_CheckConsistency(
     PyObject *op,
     int check_content);
-#elif !defined(NDEBUG)
-/* For asserts that call _PyUnicode_CheckConsistency(), which would
- * otherwise be a problem when building with asserts but without Py_DEBUG. */
-#define _PyUnicode_CheckConsistency(op, check_content) PyUnicode_Check(op)
 #endif
 
 #ifndef Py_LIMITED_API
diff --git a/Objects/dictobject.c b/Objects/dictobject.c
@@ -437,7 +437,7 @@ static PyObject *empty_values[1] = { NULL };
 /* #define DEBUG_PYDICT */
 
 
-#ifndef NDEBUG
+#ifdef Py_DEBUG
 static int
 _PyDict_CheckConsistency(PyDictObject *mp)
 {
diff --git a/Objects/obmalloc.c b/Objects/obmalloc.c
@@ -1227,9 +1227,6 @@ _PyObject_Alloc(int use_calloc, void *ctx, size_t nelem, size_t elsize)
 
     _Py_AllocatedBlocks++;
 
-    if (nelem == 0 || elsize == 0)
-        goto redirect;
-
     assert(nelem <= PY_SSIZE_T_MAX / elsize);
     nbytes = nelem * elsize;
 
@@ -1240,6 +1237,9 @@ _PyObject_Alloc(int use_calloc, void *ctx, size_t nelem, size_t elsize)
         goto redirect;
 #endif
 
+    if (nelem == 0 || elsize == 0)
+        goto redirect;
+
     if ((nbytes - 1) < SMALL_REQUEST_THRESHOLD) {
         LOCK();
         /*
diff --git a/Objects/typeobject.c b/Objects/typeobject.c
@@ -128,7 +128,7 @@ skip_signature(const char *doc)
     return NULL;
 }
 
-#ifndef NDEBUG
+#ifdef Py_DEBUG
 static int
 _PyType_CheckConsistency(PyTypeObject *type)
 {

Original file line number	Diff line number	Diff line change
`@@ -437,7 +437,7 @@ static PyObject *empty_values[1] = { NULL };`
`437`	`437`	`/* #define DEBUG_PYDICT */`
`438`	`438`
`439`	`439`
`440`		`-#ifndef NDEBUG`
	`440`	`+#ifdef Py_DEBUG`
`441`	`441`	`static int`
`442`	`442`	`_PyDict_CheckConsistency(PyDictObject *mp)`
`443`	`443`	`{`
Original file line number	Diff line number	Diff line change
`@@ -128,7 +128,7 @@ skip_signature(const char *doc)`
`128`	`128`	`return NULL;`
`129`	`129`	`}`
`130`	`130`
`131`		`-#ifndef NDEBUG`
	`131`	`+#ifdef Py_DEBUG`
`132`	`132`	`static int`
`133`	`133`	`_PyType_CheckConsistency(PyTypeObject *type)`
`134`	`134`	`{`