]> git.ipfire.org Git - thirdparty/Python/cpython.git/commitdiff
Merged revisions 61038,61042-61045,61047,61050,61053,61055-61056,61061-61064,61066...
authorChristian Heimes <christian@cheimes.de>
Tue, 26 Feb 2008 08:18:30 +0000 (08:18 +0000)
committerChristian Heimes <christian@cheimes.de>
Tue, 26 Feb 2008 08:18:30 +0000 (08:18 +0000)
svn+ssh://pythondev@svn.python.org/python/trunk

........
  r61063 | andrew.kuchling | 2008-02-25 17:29:19 +0100 (Mon, 25 Feb 2008) | 1 line

  Move .setupterm() output so that we don't try to call endwin() if it fails
........
  r61064 | andrew.kuchling | 2008-02-25 17:29:58 +0100 (Mon, 25 Feb 2008) | 1 line

  Use file descriptor for real stdout
........
  r61067 | facundo.batista | 2008-02-25 19:06:00 +0100 (Mon, 25 Feb 2008) | 4 lines

  Issue 2117.  Update compiler module to handle class decorators.
  Thanks Thomas Herve
........
  r61069 | georg.brandl | 2008-02-25 21:17:56 +0100 (Mon, 25 Feb 2008) | 2 lines

  Rename sphinx.addons to sphinx.ext.
........
  r61071 | georg.brandl | 2008-02-25 21:20:45 +0100 (Mon, 25 Feb 2008) | 2 lines

  Revert r61029.
........
  r61072 | facundo.batista | 2008-02-25 23:33:55 +0100 (Mon, 25 Feb 2008) | 4 lines

  Issue 2168. gdbm and dbm needs to be iterable; this fixes a
  failure in the shelve module.  Thanks Thomas Herve.
........
  r61073 | raymond.hettinger | 2008-02-25 23:42:32 +0100 (Mon, 25 Feb 2008) | 1 line

  Make sure the itertools filter functions give the same performance for func=bool as func=None.
........
  r61074 | raymond.hettinger | 2008-02-26 00:17:41 +0100 (Tue, 26 Feb 2008) | 1 line

  Revert part of r60927 which made invalid assumptions about the API offered by db modules.
........
  r61075 | facundo.batista | 2008-02-26 00:46:02 +0100 (Tue, 26 Feb 2008) | 3 lines

  Coerced PyBool_Type to be able to compare it.
........
  r61076 | raymond.hettinger | 2008-02-26 03:46:54 +0100 (Tue, 26 Feb 2008) | 1 line

  Docs for itertools.combinations().  Implementation in forthcoming checkin.
........
  r61077 | neal.norwitz | 2008-02-26 05:50:37 +0100 (Tue, 26 Feb 2008) | 3 lines

  Don't use a hard coded port.  This test could hang/fail if the port is in use.
  Speed this test up by avoiding a sleep and using the event.
........
  r61078 | neal.norwitz | 2008-02-26 06:12:50 +0100 (Tue, 26 Feb 2008) | 1 line

  Whitespace normalization
........
  r61079 | neal.norwitz | 2008-02-26 06:23:51 +0100 (Tue, 26 Feb 2008) | 1 line

  Whitespace normalization
........
  r61080 | georg.brandl | 2008-02-26 07:40:10 +0100 (Tue, 26 Feb 2008) | 2 lines

  Banish tab.
........

13 files changed:
Doc/conf.py
Doc/library/itertools.rst
Doc/library/thread.rst
Doc/library/threading.rst
Lib/curses/__init__.py
Lib/test/test_curses.py
Lib/test/test_ftplib.py
Lib/test/test_itertools.py
Lib/token.py
Modules/dbmmodule.c
Modules/gdbmmodule.c
Modules/itertoolsmodule.c
Python/getargs.c

index 5b3b42e63016a162bed6c9fb7794c85c2b485ce4..b7903beefcfd55a6f789301369cbdc6eaba71158 100644 (file)
@@ -13,7 +13,7 @@ sys.path.append('tools/sphinxext')
 # General configuration
 # ---------------------
 
-extensions = ['sphinx.addons.refcounting', 'sphinx.addons.coverage']
+extensions = ['sphinx.ext.refcounting', 'sphinx.ext.coverage']
 
 # General substitutions.
 project = 'Python'
index af73b570ad445f03a3e466be8293eaadd6ea15ca..d04f33b2e7b1548ecb3cb6c0ce224934daf423f2 100644 (file)
@@ -74,6 +74,45 @@ loops that truncate the stream.
                   yield element
 
 
+.. function:: combinations(iterable, r)
+
+   Return successive *r* length combinations of elements in the *iterable*.
+
+   Combinations are emitted in a lexicographic sort order.  So, if the 
+   input *iterable* is sorted, the combination tuples will be produced
+   in sorted order.  
+
+   Elements are treated as unique based on their position, not on their
+   value.  So if the input elements are unique, there will be no repeat
+   values within a single combination.
+
+   Each result tuple is ordered to match the input order.  So, every
+   combination is a subsequence of the input *iterable*.
+
+   Example:  ``combinations(range(4), 3) --> (0,1,2), (0,1,3), (0,2,3), (1,2,3)``
+
+   Equivalent to::
+
+        def combinations(iterable, r):
+            pool = tuple(iterable)
+            if pool:
+                n = len(pool)
+                vec = range(r)
+                yield tuple(pool[i] for i in vec)
+                while 1:
+                    for i in reversed(range(r)):
+                        if vec[i] == i + n-r:
+                            continue
+                        vec[i] += 1
+                        for j in range(i+1, r):
+                            vec[j] = vec[j-1] + 1
+                        yield tuple(pool[i] for i in vec)
+                        break
+                    else:
+                        return
+
+   .. versionadded:: 2.6
+
 .. function:: count([n])
 
    Make an iterator that returns consecutive integers starting with *n*. If not
@@ -298,9 +337,12 @@ loops that truncate the stream.
 
    The leftmost iterators are in the outermost for-loop, so the output tuples
    cycle in a manner similar to an odometer (with the rightmost element
-   changing on every iteration).
+   changing on every iteration).  This results in a lexicographic ordering
+   so that if the inputs iterables are sorted, the product tuples are emitted
+   in sorted order.
 
-   Equivalent to (but without building the entire result in memory)::
+   Equivalent to the following except that the actual implementation does not
+   build-up intermediate results in memory::
 
        def product(*args):
            pools = map(tuple, args)
index d7d140eba64a62fd38a67ce6a606af69cd42c469..31d58e784a3ef6fc9de703ae1808d5142c3ea5c3 100644 (file)
@@ -147,11 +147,6 @@ In addition to these methods, lock objects can also be used via the
   exception will be received by an arbitrary thread.  (When the :mod:`signal`
   module is available, interrupts always go to the main thread.)
 
-* The import machinery is not thread safe.  In general, an import may not
-  have the side effect of importing a module, and only the main thread
-  should import modules.  Imports within or caused by a thread other than
-  the main thread isn't safe.
-
 * Calling :func:`sys.exit` or raising the :exc:`SystemExit` exception is
   equivalent to calling :func:`exit`.
 
@@ -172,3 +167,4 @@ In addition to these methods, lock objects can also be used via the
 * When the main thread exits, it does not do any of its usual cleanup (except
   that :keyword:`try` ... :keyword:`finally` clauses are honored), and the
   standard I/O files are not flushed.
+
index 56b2b9906f7f38abf3a471bb45670459a993b05b..6f3e95b4c01d6ba2589718aeb6275277a1b2a5ba 100644 (file)
@@ -555,13 +555,6 @@ the :meth:`setDaemon` method and retrieved with the :meth:`isDaemon` method.
 There is a "main thread" object; this corresponds to the initial thread of
 control in the Python program.  It is not a daemon thread.
 
-.. warning::
-   
-   The import machinery is not thread safe.  In general, an import may not
-   have the side effect of importing a module, and only the main thread
-   should import modules.  Imports within or caused by a thread other than
-   the main thread isn't safe.
-
 There is the possibility that "dummy thread objects" are created. These are
 thread objects corresponding to "alien threads", which are threads of control
 started outside the threading module, such as directly from C code.  Dummy
index c3f2f25c30657fb07ea8cf7855b9818e16252c22..bd7d5f61cf4512c946a0c4e224cde36b03b24a9f 100644 (file)
@@ -15,6 +15,7 @@ __revision__ = "$Id$"
 from _curses import *
 from curses.wrapper import wrapper
 import os as _os
+import sys as _sys
 
 # Some constants, most notably the ACS_* ones, are only added to the C
 # _curses module's dictionary after initscr() is called.  (Some
@@ -28,7 +29,8 @@ def initscr():
     import _curses, curses
     # we call setupterm() here because it raises an error
     # instead of calling exit() in error cases.
-    setupterm(term=_os.environ.get("TERM", "unknown"))
+    setupterm(term=_os.environ.get("TERM", "unknown"),
+              fd=_sys.__stdout__.fileno())
     stdscr = _curses.initscr()
     for key, value in _curses.__dict__.items():
         if key[0:4] == 'ACS_' or key in ('LINES', 'COLS'):
index 64d6536ed03ea2f4b44af857e381a6ceb7b46146..d50d81dfe4843035d190352305180dec4034d45e 100644 (file)
@@ -269,13 +269,12 @@ if __name__ == '__main__':
     curses.wrapper(main)
     unit_tests()
 else:
+    # testing setupterm() inside initscr/endwin
+    # causes terminal breakage
+    curses.setupterm(fd=sys.__stdout__.fileno())
     try:
-        # testing setupterm() inside initscr/endwin
-        # causes terminal breakage
-        curses.setupterm(fd=sys.__stdout__.fileno())
         stdscr = curses.initscr()
         main(stdscr)
     finally:
         curses.endwin()
-
     unit_tests()
index d782c530f57b6f1b666ae563c645f599e67ee6dc..1ff8d088d127a4f44a4cc7e85d4c79211e5eb3ed 100644 (file)
@@ -6,34 +6,48 @@ import time
 from unittest import TestCase
 from test import test_support
 
-def server(evt, ready):
+server_port = None
+
+# This function sets the evt 3 times:
+#  1) when the connection is ready to be accepted.
+#  2) when it is safe for the caller to close the connection
+#  3) when we have closed the socket
+def server(evt):
+    global server_port
     serv = socket.socket(socket.AF_INET, socket.SOCK_STREAM)
     serv.settimeout(3)
     serv.setsockopt(socket.SOL_SOCKET, socket.SO_REUSEADDR, 1)
-    serv.bind(("", 9091))
+    server_port = test_support.bind_port(serv, "", 9091)
     serv.listen(5)
-    ready.set()
+
+    # (1) Signal the caller that we are ready to accept the connection.
+    evt.set()
     try:
         conn, addr = serv.accept()
     except socket.timeout:
         pass
     else:
         conn.send(b"1 Hola mundo\n")
+        # (2) Signal the caller that it is safe to close the socket.
+        evt.set()
         conn.close()
     finally:
         serv.close()
+        # (3) Signal the caller that we are done.
         evt.set()
 
 class GeneralTests(TestCase):
 
     def setUp(self):
-        ftplib.FTP.port = 9091
         self.evt = threading.Event()
-        ready = threading.Event()
-        threading.Thread(target=server, args=(self.evt, ready)).start()
-        ready.wait()
+        threading.Thread(target=server, args=(self.evt,)).start()
+        # Wait for the server to be ready.
+        self.evt.wait()
+        self.evt.clear()
+        ftplib.FTP.port = server_port
 
     def tearDown(self):
+        # Wait on the closing of the socket (this shouldn't be necessary).
         self.evt.wait()
 
     def testBasic(self):
@@ -42,30 +56,35 @@ class GeneralTests(TestCase):
 
         # connects
         ftp = ftplib.FTP("localhost")
+        self.evt.wait()
         ftp.sock.close()
 
     def testTimeoutDefault(self):
         # default
         ftp = ftplib.FTP("localhost")
         self.assertTrue(ftp.sock.gettimeout() is None)
+        self.evt.wait()
         ftp.sock.close()
 
     def testTimeoutValue(self):
         # a value
         ftp = ftplib.FTP("localhost", timeout=30)
         self.assertEqual(ftp.sock.gettimeout(), 30)
+        self.evt.wait()
         ftp.sock.close()
 
     def testTimeoutConnect(self):
         ftp = ftplib.FTP()
         ftp.connect("localhost", timeout=30)
         self.assertEqual(ftp.sock.gettimeout(), 30)
+        self.evt.wait()
         ftp.sock.close()
 
     def testTimeoutDifferentOrder(self):
         ftp = ftplib.FTP(timeout=30)
         ftp.connect("localhost")
         self.assertEqual(ftp.sock.gettimeout(), 30)
+        self.evt.wait()
         ftp.sock.close()
 
     def testTimeoutDirectAccess(self):
@@ -73,6 +92,7 @@ class GeneralTests(TestCase):
         ftp.timeout = 30
         ftp.connect("localhost")
         self.assertEqual(ftp.sock.gettimeout(), 30)
+        self.evt.wait()
         ftp.sock.close()
 
     def testTimeoutNone(self):
@@ -84,10 +104,10 @@ class GeneralTests(TestCase):
         finally:
             socket.setdefaulttimeout(previous)
         self.assertEqual(ftp.sock.gettimeout(), 30)
+        self.evt.wait()
         ftp.close()
 
 
-
 def test_main(verbose=None):
     test_support.run_unittest(GeneralTests)
 
index 744b344a81747cd1f4a0af0fc38f668344624e9f..63ca17b6e4f1dc7138e40e747de3cf93ab95b612 100644 (file)
@@ -177,6 +177,7 @@ class TestBasicOps(unittest.TestCase):
     def test_ifilter(self):
         self.assertEqual(list(ifilter(isEven, range(6))), [0,2,4])
         self.assertEqual(list(ifilter(None, [0,1,0,2,0])), [1,2])
+        self.assertEqual(list(ifilter(bool, [0,1,0,2,0])), [1,2])
         self.assertEqual(take(4, ifilter(isEven, count())), [0,2,4,6])
         self.assertRaises(TypeError, ifilter)
         self.assertRaises(TypeError, ifilter, lambda x:x)
@@ -187,6 +188,7 @@ class TestBasicOps(unittest.TestCase):
     def test_ifilterfalse(self):
         self.assertEqual(list(ifilterfalse(isEven, range(6))), [1,3,5])
         self.assertEqual(list(ifilterfalse(None, [0,1,0,2,0])), [0,0,0])
+        self.assertEqual(list(ifilterfalse(bool, [0,1,0,2,0])), [0,0,0])
         self.assertEqual(take(4, ifilterfalse(isEven, count())), [1,3,5,7])
         self.assertRaises(TypeError, ifilterfalse)
         self.assertRaises(TypeError, ifilterfalse, lambda x:x)
index eb48e76db81d224fb2273ae320b4ad17379de4c4..da4d29b3263d63cf95dc100ec22c8f788e1f7b49 100755 (executable)
@@ -72,6 +72,7 @@ tok_name = {}
 for _name, _value in list(globals().items()):
     if type(_value) is type(0):
         tok_name[_value] = _name
+del _name, _value
 
 
 def ISTERMINAL(x):
index 7129c8e9f369320215323403185af01a049d7f62..875f0e76a79553fc4a3faf0b146edce047287dea 100644 (file)
@@ -344,6 +344,13 @@ static PyTypeObject Dbmtype = {
        0,                        /*tp_as_number*/
        &dbm_as_sequence,         /*tp_as_sequence*/
        &dbm_as_mapping,          /*tp_as_mapping*/
+       0,                    /*tp_hash*/
+       0,                    /*tp_call*/
+       0,                    /*tp_str*/
+       0,                    /*tp_getattro*/
+       0,                    /*tp_setattro*/
+       0,                    /*tp_as_buffer*/
+       Py_TPFLAGS_DEFAULT,   /*tp_xxx4*/
 };
 
 /* ----------------------------------------------------------------- */
index 9e1b01d89389fc913a700b2305ff8fee1aedbdc8..cf197f57dd8f155ce26a3963c680738430b0bcd0 100644 (file)
@@ -407,7 +407,7 @@ static PyTypeObject Dbmtype = {
     0,                                  /*tp_getattro*/
     0,                                  /*tp_setattro*/
     0,                                  /*tp_as_buffer*/
-    0,                                  /*tp_xxx4*/
+    Py_TPFLAGS_DEFAULT,                 /*tp_xxx4*/
     gdbm_object__doc__,                 /*tp_doc*/
 };
 
index 5b6aec35c308be983256cbbd150124937ef85a4a..5c8e86fd5a5fa6b606af0b801b884be282b32ce7 100644 (file)
@@ -2025,7 +2025,7 @@ ifilter_next(ifilterobject *lz)
                if (item == NULL)
                        return NULL;
 
-               if (lz->func == Py_None) {
+               if (lz->func == Py_None || lz->func == (PyObject *)&PyBool_Type) {
                        ok = PyObject_IsTrue(item);
                } else {
                        PyObject *good;
@@ -2169,7 +2169,7 @@ ifilterfalse_next(ifilterfalseobject *lz)
                if (item == NULL)
                        return NULL;
 
-               if (lz->func == Py_None) {
+               if (lz->func == Py_None || lz->func == (PyObject *)&PyBool_Type) {
                        ok = PyObject_IsTrue(item);
                } else {
                        PyObject *good;
index aeeb04cee36a71412999fa8b1ae045c874671c18..c0b3624b3b0e954a6c3cc082914fb55535ffc9af 100644 (file)
@@ -154,7 +154,7 @@ addcleanup(void *ptr, PyObject **freelist)
                PyMem_FREE(ptr);
                return -1;
        }
-       if(PyList_Append(*freelist, cobj)) {
+       if (PyList_Append(*freelist, cobj)) {
                 PyMem_FREE(ptr);
                Py_DECREF(cobj);
                return -1;
@@ -166,8 +166,8 @@ addcleanup(void *ptr, PyObject **freelist)
 static int
 cleanreturn(int retval, PyObject *freelist)
 {
-       if(freelist) {
-               if((retval) == 0) {
+       if (freelist) {
+               if (retval == 0) {
                        Py_ssize_t len = PyList_GET_SIZE(freelist), i;
                        for (i = 0; i < len; i++)
                                 PyMem_FREE(PyCObject_AsVoidPtr(
@@ -708,7 +708,7 @@ convertsimple(PyObject *arg, const char **p_format, va_list *p_va, int flags,
        case 'L': {/* PY_LONG_LONG */
                PY_LONG_LONG *p = va_arg( *p_va, PY_LONG_LONG * );
                PY_LONG_LONG ival = PyLong_AsLongLong( arg );
-               ifival == (PY_LONG_LONG)-1 && PyErr_Occurred() ) {
+               if (ival == (PY_LONG_LONG)-1 && PyErr_Occurred() ) {
                        return converterr("long<L>", arg, msgbuf, bufsize);
                } else {
                        *p = ival;
@@ -1045,7 +1045,7 @@ convertsimple(PyObject *arg, const char **p_format, va_list *p_va, int flags,
                                                "(memory error)",
                                                arg, msgbuf, bufsize);
                                }
-                               if(addcleanup(*buffer, freelist)) {
+                               if (addcleanup(*buffer, freelist)) {
                                        Py_DECREF(s);
                                        return converterr(
                                                "(cleanup problem)",
@@ -1087,7 +1087,7 @@ convertsimple(PyObject *arg, const char **p_format, va_list *p_va, int flags,
                                return converterr("(memory error)",
                                                  arg, msgbuf, bufsize);
                        }
-                       if(addcleanup(*buffer, freelist)) {
+                       if (addcleanup(*buffer, freelist)) {
                                Py_DECREF(s);
                                return converterr("(cleanup problem)",
                                                arg, msgbuf, bufsize);