]> granicus.if.org Git - python/commitdiff
Correct previous checkin, probably a svn merge issue.
authorAmaury Forgeot d'Arc <amauryfa@gmail.com>
Fri, 11 Apr 2008 00:33:07 +0000 (00:33 +0000)
committerAmaury Forgeot d'Arc <amauryfa@gmail.com>
Fri, 11 Apr 2008 00:33:07 +0000 (00:33 +0000)
Now the code is similar to the one in trunk/.

The behavior was funny:
   >>> print (), repr(())
   (), ()
   >>> print (), repr(())
   (), (...)

Objects/tupleobject.c

index 56a1024fa1d206f50b1f83afbf198b6a666690b6..b68c80e4a2cba5cc9c6f553f1f9a9e7f19f8891e 100644 (file)
@@ -208,6 +208,10 @@ tuplerepr(PyTupleObject *v)
        PyObject *s, *temp;
        PyObject *pieces, *result = NULL;
 
+       n = v->ob_size;
+       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
@@ -217,10 +221,6 @@ tuplerepr(PyTupleObject *v)
                return i > 0 ? PyString_FromString("(...)") : NULL;
        }
 
-       n = v->ob_size;
-       if (n == 0)
-               return PyString_FromString("()");
-
        pieces = PyTuple_New(n);
        if (pieces == NULL)
                return NULL;