has_finalizer(): simplified "if (complicated_bool) 1 else 0" to

"complicated_bool".

has_finalizer(): simplified "if (complicated_bool) 1 else 0" to
"complicated_bool".
db865611 · Tim Peters · 49cc01e5 · db865611
Commit db865611 authored Nov 01, 2001 by Tim Peters
Hide whitespace changes
Inline Side-by-side

Showing with 3 additions and 8 deletions

Modules/gcmodule.c Modules/gcmodule.c +3 -8

No files found.
--- a/Modules/gcmodule.c
+++ b/Modules/gcmodule.c
@@ -236,14 +236,9 @@ has_finalizer(PyObject *op)
 		if (delstr == NULL)
 			Py_FatalError("PyGC: can't initialize __del__ string");
 	}
-	if ((PyInstance_Check(op) ||
-	     PyType_HasFeature(op->ob_type, Py_TPFLAGS_HEAPTYPE)) &&
-	    PyObject_HasAttr(op, delstr)) {
-		return 1;
-	}
-	else {
-		return 0;
-	}
+	return (PyInstance_Check(op) ||
+	        PyType_HasFeature(op->ob_type, Py_TPFLAGS_HEAPTYPE))
+	       && PyObject_HasAttr(op, delstr);
 }

 /* Move all objects with finalizers (instances with __del__) */