[Python-3000-checkins] r61083 - in python/branches/py3k: 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
christian.heimes
python-3000-checkins at python.org
Tue Feb 26 09:18:31 CET 2008
Author: christian.heimes
Date: Tue Feb 26 09:18:30 2008
New Revision: 61083
Modified:
python/branches/py3k/ (props changed)
python/branches/py3k/Doc/conf.py
python/branches/py3k/Doc/library/itertools.rst
python/branches/py3k/Doc/library/thread.rst
python/branches/py3k/Doc/library/threading.rst
python/branches/py3k/Lib/curses/__init__.py
python/branches/py3k/Lib/test/test_curses.py
python/branches/py3k/Lib/test/test_ftplib.py
python/branches/py3k/Lib/test/test_itertools.py
python/branches/py3k/Lib/token.py
python/branches/py3k/Modules/dbmmodule.c
python/branches/py3k/Modules/gdbmmodule.c
python/branches/py3k/Modules/itertoolsmodule.c
python/branches/py3k/Python/getargs.c
Log:
Merged revisions 61038,61042-61045,61047,61050,61053,61055-61056,61061-61064,61066-61080 via svnmerge from
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.
........
Modified: python/branches/py3k/Doc/conf.py
==============================================================================
--- python/branches/py3k/Doc/conf.py (original)
+++ python/branches/py3k/Doc/conf.py Tue Feb 26 09:18:30 2008
@@ -13,7 +13,7 @@
# General configuration
# ---------------------
-extensions = ['sphinx.addons.refcounting', 'sphinx.addons.coverage']
+extensions = ['sphinx.ext.refcounting', 'sphinx.ext.coverage']
# General substitutions.
project = 'Python'
Modified: python/branches/py3k/Doc/library/itertools.rst
==============================================================================
--- python/branches/py3k/Doc/library/itertools.rst (original)
+++ python/branches/py3k/Doc/library/itertools.rst Tue Feb 26 09:18:30 2008
@@ -74,6 +74,45 @@
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 @@
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)
Modified: python/branches/py3k/Doc/library/thread.rst
==============================================================================
--- python/branches/py3k/Doc/library/thread.rst (original)
+++ python/branches/py3k/Doc/library/thread.rst Tue Feb 26 09:18:30 2008
@@ -147,11 +147,6 @@
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 @@
* 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.
+
Modified: python/branches/py3k/Doc/library/threading.rst
==============================================================================
--- python/branches/py3k/Doc/library/threading.rst (original)
+++ python/branches/py3k/Doc/library/threading.rst Tue Feb 26 09:18:30 2008
@@ -555,13 +555,6 @@
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
Modified: python/branches/py3k/Lib/curses/__init__.py
==============================================================================
--- python/branches/py3k/Lib/curses/__init__.py (original)
+++ python/branches/py3k/Lib/curses/__init__.py Tue Feb 26 09:18:30 2008
@@ -15,6 +15,7 @@
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 @@
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'):
Modified: python/branches/py3k/Lib/test/test_curses.py
==============================================================================
--- python/branches/py3k/Lib/test/test_curses.py (original)
+++ python/branches/py3k/Lib/test/test_curses.py Tue Feb 26 09:18:30 2008
@@ -269,13 +269,12 @@
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()
Modified: python/branches/py3k/Lib/test/test_ftplib.py
==============================================================================
--- python/branches/py3k/Lib/test/test_ftplib.py (original)
+++ python/branches/py3k/Lib/test/test_ftplib.py Tue Feb 26 09:18:30 2008
@@ -6,34 +6,48 @@
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 @@
# 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 @@
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 @@
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)
Modified: python/branches/py3k/Lib/test/test_itertools.py
==============================================================================
--- python/branches/py3k/Lib/test/test_itertools.py (original)
+++ python/branches/py3k/Lib/test/test_itertools.py Tue Feb 26 09:18:30 2008
@@ -177,6 +177,7 @@
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 @@
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)
Modified: python/branches/py3k/Lib/token.py
==============================================================================
--- python/branches/py3k/Lib/token.py (original)
+++ python/branches/py3k/Lib/token.py Tue Feb 26 09:18:30 2008
@@ -72,6 +72,7 @@
for _name, _value in list(globals().items()):
if type(_value) is type(0):
tok_name[_value] = _name
+del _name, _value
def ISTERMINAL(x):
Modified: python/branches/py3k/Modules/dbmmodule.c
==============================================================================
--- python/branches/py3k/Modules/dbmmodule.c (original)
+++ python/branches/py3k/Modules/dbmmodule.c Tue Feb 26 09:18:30 2008
@@ -344,6 +344,13 @@
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*/
};
/* ----------------------------------------------------------------- */
Modified: python/branches/py3k/Modules/gdbmmodule.c
==============================================================================
--- python/branches/py3k/Modules/gdbmmodule.c (original)
+++ python/branches/py3k/Modules/gdbmmodule.c Tue Feb 26 09:18:30 2008
@@ -407,7 +407,7 @@
0, /*tp_getattro*/
0, /*tp_setattro*/
0, /*tp_as_buffer*/
- 0, /*tp_xxx4*/
+ Py_TPFLAGS_DEFAULT, /*tp_xxx4*/
gdbm_object__doc__, /*tp_doc*/
};
Modified: python/branches/py3k/Modules/itertoolsmodule.c
==============================================================================
--- python/branches/py3k/Modules/itertoolsmodule.c (original)
+++ python/branches/py3k/Modules/itertoolsmodule.c Tue Feb 26 09:18:30 2008
@@ -2025,7 +2025,7 @@
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 @@
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;
Modified: python/branches/py3k/Python/getargs.c
==============================================================================
--- python/branches/py3k/Python/getargs.c (original)
+++ python/branches/py3k/Python/getargs.c Tue Feb 26 09:18:30 2008
@@ -154,7 +154,7 @@
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 @@
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 @@
case 'L': {/* PY_LONG_LONG */
PY_LONG_LONG *p = va_arg( *p_va, PY_LONG_LONG * );
PY_LONG_LONG ival = PyLong_AsLongLong( arg );
- if( ival == (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 @@
"(memory error)",
arg, msgbuf, bufsize);
}
- if(addcleanup(*buffer, freelist)) {
+ if (addcleanup(*buffer, freelist)) {
Py_DECREF(s);
return converterr(
"(cleanup problem)",
@@ -1087,7 +1087,7 @@
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);
More information about the Python-3000-checkins
mailing list