Mailman 3 August 2006 - Python-checkins

r51565 - in python/trunk: Lib/test/test_array.py Misc/ACKS Modules/arraymodule.c
by thomas.wouters 24 Aug '06

24 Aug '06

Author: thomas.wouters Date: Thu Aug 24 20:40:20 2006 New Revision: 51565 Modified: python/trunk/Lib/test/test_array.py python/trunk/Misc/ACKS python/trunk/Modules/arraymodule.c Log: Fix SF bug #1545837: array.array borks on deepcopy. array.__deepcopy__() needs to take an argument, even if it doesn't actually use it. Will backport to 2.5 and 2.4 (if applicable.) Modified: python/trunk/Lib/test/test_array.py ============================================================================== --- python/trunk/Lib/test/test_array.py (original) +++ python/trunk/Lib/test/test_array.py Thu Aug 24 20:40:20 2006 @@ -85,6 +85,13 @@ self.assertNotEqual(id(a), id(b)) self.assertEqual(a, b) + def test_deepcopy(self): + import copy + a = array.array(self.typecode, self.example) + b = copy.deepcopy(a) + self.assertNotEqual(id(a), id(b)) + self.assertEqual(a, b) + def test_pickle(self): for protocol in (0, 1, 2): a = array.array(self.typecode, self.example) Modified: python/trunk/Misc/ACKS ============================================================================== --- python/trunk/Misc/ACKS (original) +++ python/trunk/Misc/ACKS Thu Aug 24 20:40:20 2006 @@ -242,6 +242,7 @@ Michael Guravage Lars Gust�bel Barry Haddow +V�clav Haisman Paul ten Hagen Rasmus Hahn Peter Haight Modified: python/trunk/Modules/arraymodule.c ============================================================================== --- python/trunk/Modules/arraymodule.c (original) +++ python/trunk/Modules/arraymodule.c Thu Aug 24 20:40:20 2006 @@ -1495,7 +1495,7 @@ copy_doc}, {"count", (PyCFunction)array_count, METH_O, count_doc}, - {"__deepcopy__",(PyCFunction)array_copy, METH_NOARGS, + {"__deepcopy__",(PyCFunction)array_copy, METH_O, copy_doc}, {"extend", (PyCFunction)array_extend, METH_O, extend_doc},

1 0

r51563 - python/branches/int_unification/INTBENCH
by martin.v.loewis 24 Aug '06

24 Aug '06

Author: martin.v.loewis Date: Thu Aug 24 19:44:49 2006 New Revision: 51563 Modified: python/branches/int_unification/INTBENCH Log: Measure 51562. Modified: python/branches/int_unification/INTBENCH ============================================================================== --- python/branches/int_unification/INTBENCH (original) +++ python/branches/int_unification/INTBENCH Thu Aug 24 19:44:49 2006 @@ -31,3 +31,11 @@ Test 6: 3.775941s Test 7: 4.001805s +r51562 (special-case one-digit operations) +Test 1: 3.527860s +Test 2: 3.975953s +Test 3: 4.226751s +Test 4: 10.605721s +Test 5: 5.233576s +Test 6: 2.161525s +Test 7: 3.421624s

1 0

r51562 - in python/branches/int_unification: Objects/longobject.c Objects/stringobject.c Python/ceval.c
by martin.v.loewis 24 Aug '06

24 Aug '06

Author: martin.v.loewis Date: Thu Aug 24 19:42:10 2006 New Revision: 51562 Modified: python/branches/int_unification/Objects/longobject.c python/branches/int_unification/Objects/stringobject.c python/branches/int_unification/Python/ceval.c Log: Special-case more single-digit operations. Avoid modifying singleton values. Fix index computation. Modified: python/branches/int_unification/Objects/longobject.c ============================================================================== --- python/branches/int_unification/Objects/longobject.c (original) +++ python/branches/int_unification/Objects/longobject.c Thu Aug 24 19:42:10 2006 @@ -48,6 +48,14 @@ #define CHECK_SMALL_INT(ival) #endif +#define MEDIUM_VALUE(x) ((x)->ob_size < 0 ? -(x)->ob_digit[0] : (x)->ob_digit[0]) +/* If a freshly-allocated long is already shared, it must + be a small integer, so negating it must go to PyLong_FromLong */ +#define NEGATE(x) \ + do if ((x)->ob_refcnt == 1) (x)->ob_size = -(x)->ob_size; \ + else { PyObject* tmp=PyInt_FromLong(-MEDIUM_VALUE(x)); \ + Py_DECREF(x); (x) = (PyLongObject*)tmp; } \ + while(0) /* For long multiplication, use the O(N**2) school algorithm unless * both operands contain more than KARATSUBA_CUTOFF digits (this * being an internal Python long digit, in base BASE). @@ -105,11 +113,18 @@ PyLongObject * _PyLong_New(Py_ssize_t size) { + PyLongObject *result; if (size > PY_SSIZE_T_MAX) { PyErr_NoMemory(); return NULL; } - return PyObject_NEW_VAR(PyLongObject, &PyLong_Type, size); + result = PyObject_MALLOC(sizeof(PyLongObject) + + (size-1)*sizeof(digit)); + if (!result) { + PyErr_NoMemory(); + return NULL; + } + return (PyLongObject*)PyObject_INIT_VAR(result, &PyLong_Type, size); } PyObject * @@ -155,7 +170,7 @@ if (ival < BASE) { /* Fast path for single-digits ints */ - v = PyObject_NEW_VAR(PyLongObject, &PyLong_Type, 1); + v = _PyLong_New(1); if (v) { v->ob_size = negative ? -1 : 1; v->ob_digit[0] = ival; @@ -291,6 +306,11 @@ v = (PyLongObject *)vv; i = v->ob_size; + switch (i) { + case -1: return -v->ob_digit[0]; + case 0: return 0; + case 1: return v->ob_digit[0]; + } sign = 1; x = 0; if (i < 0) { @@ -353,6 +373,11 @@ } v = (PyLongObject *)vv; i = v->ob_size; + switch (i) { + case -1: return -v->ob_digit[0]; + case 0: return 0; + case 1: return v->ob_digit[0]; + } sign = 1; x = 0; if (i < 0) { @@ -402,6 +427,10 @@ "can't convert negative value to unsigned long"); return (unsigned long) -1; } + switch (i) { + case 0: return 0; + case 1: return v->ob_digit[0]; + } while (--i >= 0) { prev = x; x = (x << SHIFT) + v->ob_digit[i]; @@ -436,6 +465,10 @@ "can't convert negative value to size_t"); return (size_t) -1; } + switch (i) { + case 0: return 0; + case 1: return v->ob_digit[0]; + } while (--i >= 0) { prev = x; x = (x << SHIFT) + v->ob_digit[i]; @@ -465,6 +498,10 @@ } v = (PyLongObject *)vv; i = v->ob_size; + switch (i) { + case 0: return 0; + case 1: return v->ob_digit[0]; + } sign = 1; x = 0; if (i < 0) { @@ -1014,6 +1051,7 @@ PY_LONG_LONG PyLong_AsLongLong(PyObject *vv) { + PyLongObject *v; PY_LONG_LONG bytes; int one = 1; int res; @@ -1043,6 +1081,12 @@ return -1; } + v = (PyLongObject*)vv; + switch(v->ob_size) { + case -1: return -v->ob_digit[0]; + case 0: return 0; + case 1: return v->ob_digit[0]; + } res = _PyLong_AsByteArray( (PyLongObject *)vv, (unsigned char *)&bytes, SIZEOF_LONG_LONG, IS_LITTLE_ENDIAN, 1); @@ -1060,6 +1104,7 @@ unsigned PY_LONG_LONG PyLong_AsUnsignedLongLong(PyObject *vv) { + PyLongObject *v; unsigned PY_LONG_LONG bytes; int one = 1; int res; @@ -1069,6 +1114,12 @@ return (unsigned PY_LONG_LONG)-1; } + v = (PyLongObject*)vv; + switch(v->ob_size) { + case 0: return 0; + case 1: return v->ob_digit[0]; + } + res = _PyLong_AsByteArray( (PyLongObject *)vv, (unsigned char *)&bytes, SIZEOF_LONG_LONG, IS_LITTLE_ENDIAN, 0); @@ -1096,6 +1147,10 @@ return (unsigned long) -1; } v = (PyLongObject *)vv; + switch(v->ob_size) { + case 0: return 0; + case 1: return v->ob_digit[0]; + } i = v->ob_size; sign = 1; x = 0; @@ -1816,7 +1871,7 @@ (size_a == size_b && a->ob_digit[size_a-1] < b->ob_digit[size_b-1])) { /* |a| < |b|. */ - *pdiv = _PyLong_New(0); + *pdiv = (PyLongObject*)PyLong_FromLong(0); Py_INCREF(a); *prem = (PyLongObject *) a; return 0; @@ -1838,9 +1893,9 @@ the remainder r has the sign of a, so a = b*z + r. */ if ((a->ob_size < 0) != (b->ob_size < 0)) - z->ob_size = -(z->ob_size); + NEGATE(z); if (a->ob_size < 0 && (*prem)->ob_size != 0) - (*prem)->ob_size = -((*prem)->ob_size); + NEGATE(*prem); *pdiv = z; return 0; } @@ -1996,6 +2051,11 @@ same value hash to the same value, otherwise comparisons of mapping keys will turn out weird */ i = v->ob_size; + switch(i) { + case -1: return v->ob_digit[0]==1 ? -2 : -v->ob_digit[0]; + case 0: return 0; + case 1: return v->ob_digit[1]; + } sign = 1; x = 0; if (i < 0) { @@ -2101,7 +2161,7 @@ } assert(borrow == 0); if (sign < 0) - z->ob_size = -(z->ob_size); + NEGATE(z); return long_normalize(z); } @@ -2112,6 +2172,9 @@ CONVERT_BINOP((PyObject *)v, (PyObject *)w, &a, &b); + if (ABS(a->ob_size) <= 1 && ABS(b->ob_size) <= 1) + return PyInt_FromLong(MEDIUM_VALUE(a) + + MEDIUM_VALUE(b)); if (a->ob_size < 0) { if (b->ob_size < 0) { z = x_add(a, b); @@ -2139,6 +2202,8 @@ CONVERT_BINOP((PyObject *)v, (PyObject *)w, &a, &b); + if (ABS(a->ob_size) <= 1 && ABS(b->ob_size) <= 1) + return PyLong_FromLong(MEDIUM_VALUE(a)-MEDIUM_VALUE(b)); if (a->ob_size < 0) { if (b->ob_size < 0) z = x_sub(a, b); @@ -2568,10 +2633,13 @@ return Py_NotImplemented; } + if (ABS(v->ob_size) <= 1 && ABS(w->ob_size) <= 1) + return PyLong_FromLong(MEDIUM_VALUE(v)*MEDIUM_VALUE(w)); + z = k_mul(a, b); /* Negate if exactly one of the inputs is negative. */ if (((a->ob_size ^ b->ob_size) < 0) && z) - z->ob_size = -(z->ob_size); + NEGATE(z); Py_DECREF(a); Py_DECREF(b); return (PyObject *)z; @@ -2810,7 +2878,7 @@ Py_DECREF(c); c = temp; temp = NULL; - c->ob_size = - c->ob_size; + NEGATE(c); } /* if modulus == 1: @@ -2931,6 +2999,8 @@ /* Implement ~x as -(x+1) */ PyLongObject *x; PyLongObject *w; + if (ABS(v->ob_size) <=1) + return PyLong_FromLong(-(MEDIUM_VALUE(v)+1)); w = (PyLongObject *)PyLong_FromLong(1L); if (w == NULL) return NULL; @@ -2957,11 +3027,8 @@ long_neg(PyLongObject *v) { PyLongObject *z; - if (v->ob_size == 0 && PyLong_CheckExact(v)) { - /* -0 == 0 */ - Py_INCREF(v); - return (PyObject *) v; - } + if (ABS(v->ob_size) <= 1) + return PyLong_FromLong(-MEDIUM_VALUE(v)); z = (PyLongObject *)_PyLong_Copy(v); if (z != NULL) z->ob_size = -(v->ob_size); @@ -3085,7 +3152,7 @@ if (z == NULL) goto lshift_error; if (a->ob_size < 0) - z->ob_size = -(z->ob_size); + NEGATE(z); for (i = 0; i < wordshift; i++) z->ob_digit[i] = 0; accum = 0; Modified: python/branches/int_unification/Objects/stringobject.c ============================================================================== --- python/branches/int_unification/Objects/stringobject.c (original) +++ python/branches/int_unification/Objects/stringobject.c Thu Aug 24 19:42:10 2006 @@ -4206,6 +4206,14 @@ int numdigits; /* len == numnondigits + numdigits */ int numnondigits = 0; + /* Avoid exceeding SSIZE_T_MAX */ + if (prec > PY_SSIZE_T_MAX-3) { + PyErr_SetString(PyExc_OverflowError, + "precision too large"); + return NULL; + } + + switch (type) { case 'd': case 'u': Modified: python/branches/int_unification/Python/ceval.c ============================================================================== --- python/branches/int_unification/Python/ceval.c (original) +++ python/branches/int_unification/Python/ceval.c Thu Aug 24 19:42:10 2006 @@ -3819,7 +3819,7 @@ { if (v != NULL) { Py_ssize_t x; - if (PyInt_Check(v)) { + if (PyInt_CheckExact(v)) { /* XXX(nnorwitz): I think PyInt_AS_LONG is correct, however, it looks like it should be AsSsize_t. There should be a comment here explaining why.

1 0

r51556 - python/branches/int_unification/INTBENCH
by martin.v.loewis 24 Aug '06

24 Aug '06

Author: martin.v.loewis Date: Thu Aug 24 17:03:21 2006 New Revision: 51556 Modified: python/branches/int_unification/INTBENCH Log: Record timings for medium-int allocations. Modified: python/branches/int_unification/INTBENCH ============================================================================== --- python/branches/int_unification/INTBENCH (original) +++ python/branches/int_unification/INTBENCH Thu Aug 24 17:03:21 2006 @@ -22,3 +22,12 @@ Test 3: 4.999010s Test 4: 10.818277s +r51452 (special-casing medium int allocation) +Test 1: 3.258219s +Test 2: 4.255007s +Test 3: 4.547923s +Test 4: 10.615123s +Test 5: 5.255545s +Test 6: 3.775941s +Test 7: 4.001805s +

1 0

r51534 - peps/trunk/pep-3100.txt
by brett.cannon 24 Aug '06

24 Aug '06

Author: brett.cannon Date: Thu Aug 24 03:13:18 2006 New Revision: 51534 Modified: peps/trunk/pep-3100.txt Log: Clarify why operator.isCallable() and operator.sequenceIncludes() are going away. Modified: peps/trunk/pep-3100.txt ============================================================================== --- peps/trunk/pep-3100.txt (original) +++ peps/trunk/pep-3100.txt Thu Aug 24 03:13:18 2006 @@ -209,7 +209,10 @@ not thread-safe; use ``sys.exc_info()`` or an attribute of the exception [2]_ [13]_ [#sys-module]_ * ``array.read``, ``array.write`` [#array-module]_ -* ``operator.isCallable``, ``operator.sequenceIncludes`` [#operator-module]_ +* ``operator.isCallable`` : ``callable()`` built-in is being removed + [#operator-module]_ +* ``operator.sequenceIncludes`` : redundant thanks to + ``operator.contains`` [#operator-module]_ * In the thread module, the aquire_lock() and release_lock() aliases for the acquire() and release() methods on lock objects. (Probably also just remove the thread module as a public API,

1 0

r51532 - peps/trunk/pep-0362.txt
by brett.cannon 24 Aug '06

24 Aug '06

Author: brett.cannon Date: Thu Aug 24 02:31:25 2006 New Revision: 51532 Modified: peps/trunk/pep-0362.txt Log: Swap some words to get the proper wording for Signature.bind(). Modified: peps/trunk/pep-0362.txt ============================================================================== --- peps/trunk/pep-0362.txt (original) +++ peps/trunk/pep-0362.txt Thu Aug 24 02:31:25 2006 @@ -61,7 +61,7 @@ Return the string representation of the signature as it might appear in source code. * bind(\*args, \*\*kwargs) -> dict - Create a mapping from argument to parameter for the signature (see + Create a mapping from parameter to argument for the signature (see `Open Issues`_ for question of how to handle tuples). The Signature object is stored in the ``__signature__`` attribute of

1 0

buildbot warnings in alpha Tru64 5.1 trunk
by buildbot＠python.org 23 Aug '06

23 Aug '06

The Buildbot has detected a new failure of alpha Tru64 5.1 trunk. Full details are available at: http://www.python.org/dev/buildbot/all/alpha%2520Tru64%25205.1%2520trunk/bu… Buildbot URL: http://www.python.org/dev/buildbot/all/ Build Reason: Build Source Stamp: [branch trunk] HEAD Blamelist: jeremy.hylton Build Had Warnings: warnings test sincerely, -The Buildbot

1 0

buildbot warnings in S-390 Debian trunk
by buildbot＠python.org 23 Aug '06

23 Aug '06

The Buildbot has detected a new failure of S-390 Debian trunk. Full details are available at: http://www.python.org/dev/buildbot/all/S-390%2520Debian%2520trunk/builds/345 Buildbot URL: http://www.python.org/dev/buildbot/all/ Build Reason: Build Source Stamp: [branch trunk] HEAD Blamelist: jeremy.hylton Build Had Warnings: warnings test sincerely, -The Buildbot

1 0

r51530 - in python/trunk: Lib/test/test_float.py Objects/floatobject.c
by alex.martelli 23 Aug '06

23 Aug '06

Author: alex.martelli Date: Thu Aug 24 00:17:59 2006 New Revision: 51530 Modified: python/trunk/Lib/test/test_float.py python/trunk/Objects/floatobject.c Log: Reverting the patch that tried to fix the issue whereby x**2 raises OverflowError while x*x succeeds and produces infinity; apparently these inconsistencies cannot be fixed across ``all'' platforms and there's a widespread feeling that therefore ``every'' platform should keep suffering forevermore. Ah well. Modified: python/trunk/Lib/test/test_float.py ============================================================================== --- python/trunk/Lib/test/test_float.py (original) +++ python/trunk/Lib/test/test_float.py Thu Aug 24 00:17:59 2006 @@ -99,25 +99,12 @@ ('<f', LE_FLOAT_NAN)]: struct.unpack(fmt, data) -# on an IEEE platform, "overflowing" operations produce infinity - -class IEEEOperationsTestCase(unittest.TestCase): - if float.__getformat__("double").startswith("IEEE"): - def test_double_infinity(self): - big = 4.8e159 - pro = big*big - self.assertEquals(repr(pro), 'inf') - sqr = big**2 - self.assertEquals(repr(sqr), 'inf') - def test_main(): test_support.run_unittest( FormatFunctionsTestCase, UnknownFormatTestCase, - IEEEFormatTestCase, - IEEEOperationsTestCase, - ) + IEEEFormatTestCase) if __name__ == '__main__': test_main() Modified: python/trunk/Objects/floatobject.c ============================================================================== --- python/trunk/Objects/floatobject.c (original) +++ python/trunk/Objects/floatobject.c Thu Aug 24 00:17:59 2006 @@ -821,12 +821,12 @@ ix = pow(iv, iw); PyFPE_END_PROTECT(ix) Py_ADJUST_ERANGE1(ix); - /* we need to ignore ERANGE here and just return inf */ - if (errno != 0 && errno != ERANGE) { + if (errno != 0) { /* We don't expect any errno value other than ERANGE, but * the range of libm bugs appears unbounded. */ - PyErr_SetFromErrno(PyExc_ValueError); + PyErr_SetFromErrno(errno == ERANGE ? PyExc_OverflowError : + PyExc_ValueError); return NULL; } return PyFloat_FromDouble(ix);

1 0