From 31ba8480d823f2c424d77ccd8c446b79ca48c1ce Mon Sep 17 00:00:00 2001 From: Brett Cannon Date: Sun, 30 Sep 2007 20:37:19 +0000 Subject: [PATCH] Fix error introduced by r58288; if a tuple is length 0 return its repr and don't worry about any self-referring tuples. --- Objects/tupleobject.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/Objects/tupleobject.c b/Objects/tupleobject.c index 38304dda18..b1704f7be4 100644 --- a/Objects/tupleobject.c +++ b/Objects/tupleobject.c @@ -216,6 +216,10 @@ tuplerepr(PyTupleObject *v) PyObject *s, *temp; PyObject *pieces, *result = NULL; + n = Py_Size(v); + if (n == 0) + return PyString_FromString("()"); + /* While not mutable, it is still possible to end up with a cycle in a tuple through an object that stores itself within a tuple (and thus infinitely asks for the repr of itself). This should only be @@ -225,10 +229,6 @@ tuplerepr(PyTupleObject *v) return i > 0 ? PyString_FromString("(...)") : NULL; } - n = Py_Size(v); - if (n == 0) - return PyString_FromString("()"); - pieces = PyTuple_New(n); if (pieces == NULL) return NULL; -- 2.50.0