The filter() function does support a None argument in Py3.0.

This commit is contained in:
Raymond Hettinger 2008-03-19 17:58:59 +00:00
parent 05387861ea
commit 10dca6e282
3 changed files with 0 additions and 27 deletions

View File

@ -48,17 +48,6 @@ class TestPy3KWarnings(unittest.TestCase):
with catch_warning() as w: with catch_warning() as w:
self.assertWarning(cell0 < cell1, w, expected) self.assertWarning(cell0 < cell1, w, expected)
def test_filter(self):
from itertools import ifilter
from future_builtins import filter
expected = 'ifilter with None as a first argument is not supported '\
'in 3.x. Use a list comprehension instead.'
with catch_warning() as w:
self.assertWarning(ifilter(None, []), w, expected)
with catch_warning() as w:
self.assertWarning(filter(None, []), w, expected)
def test_code_inequality_comparisons(self): def test_code_inequality_comparisons(self):
expected = 'code inequality comparisons not supported in 3.x.' expected = 'code inequality comparisons not supported in 3.x.'
def f(x): def f(x):

View File

@ -2552,15 +2552,6 @@ ifilter_new(PyTypeObject *type, PyObject *args, PyObject *kwds)
if (!PyArg_UnpackTuple(args, "ifilter", 2, 2, &func, &seq)) if (!PyArg_UnpackTuple(args, "ifilter", 2, 2, &func, &seq))
return NULL; return NULL;
if (func == Py_None) {
if (Py_Py3kWarningFlag &&
PyErr_Warn(PyExc_DeprecationWarning,
"ifilter with None as a first argument "
"is not supported in 3.x. Use a list "
"comprehension instead.") < 0)
return NULL;
}
/* Get iterator. */ /* Get iterator. */
it = PyObject_GetIter(seq); it = PyObject_GetIter(seq);
if (it == NULL) if (it == NULL)

View File

@ -296,13 +296,6 @@ builtin_filter(PyObject *self, PyObject *args)
} }
if (func == (PyObject *)&PyBool_Type || func == Py_None) { if (func == (PyObject *)&PyBool_Type || func == Py_None) {
if (Py_Py3kWarningFlag &&
PyErr_Warn(PyExc_DeprecationWarning,
"filter with None as a first argument "
"is not supported in 3.x. Use a list "
"comprehension instead.") < 0)
return NULL;
ok = PyObject_IsTrue(item); ok = PyObject_IsTrue(item);
} }
else { else {