[Python-checkins] cpython (merge 3.2 -> 3.3): Issue #16761: Raise TypeError when int() called with base argument only.
serhiy.storchaka
python-checkins at python.org
Fri Dec 28 09:21:03 CET 2012
http://hg.python.org/cpython/rev/157ff02bcc16
changeset: 81125:157ff02bcc16
branch: 3.3
parent: 81119:e100d34070cb
parent: 81124:e4ea38a92c4d
user: Serhiy Storchaka <storchaka at gmail.com>
date: Fri Dec 28 10:02:42 2012 +0200
summary:
Issue #16761: Raise TypeError when int() called with base argument only.
files:
Lib/test/test_int.py | 12 ++----------
Misc/NEWS | 3 +++
Objects/longobject.c | 10 ++++++++--
3 files changed, 13 insertions(+), 12 deletions(-)
diff --git a/Lib/test/test_int.py b/Lib/test/test_int.py
--- a/Lib/test/test_int.py
+++ b/Lib/test/test_int.py
@@ -233,16 +233,8 @@
self.assertEqual(int(x=1.2), 1)
self.assertEqual(int('100', base=2), 4)
self.assertEqual(int(x='100', base=2), 4)
-
- # For example, PyPy 1.9.0 raised TypeError for these cases because it
- # expects x to be a string if base is given.
- @support.cpython_only
- def test_base_arg_with_no_x_arg(self):
- self.assertEqual(int(base=6), 0)
- # Even invalid bases don't raise an exception.
- self.assertEqual(int(base=1), 0)
- self.assertEqual(int(base=1000), 0)
- self.assertEqual(int(base='foo'), 0)
+ self.assertRaises(TypeError, int, base=10)
+ self.assertRaises(TypeError, int, base=0)
def test_non_numeric_input_types(self):
# Test possible non-numeric types for the argument x, including
diff --git a/Misc/NEWS b/Misc/NEWS
--- a/Misc/NEWS
+++ b/Misc/NEWS
@@ -12,6 +12,9 @@
Core and Builtins
-----------------
+- Issue #16761: Calling ``int()`` with *base* argument only now raises
+ TypeError.
+
- Issue #16759: Support the full DWORD (unsigned long) range in Reg2Py
when retreiving a REG_DWORD value. This corrects functions like
winreg.QueryValueEx that may have been returning truncated values.
diff --git a/Objects/longobject.c b/Objects/longobject.c
--- a/Objects/longobject.c
+++ b/Objects/longobject.c
@@ -4267,8 +4267,14 @@
if (!PyArg_ParseTupleAndKeywords(args, kwds, "|OO:int", kwlist,
&x, &obase))
return NULL;
- if (x == NULL)
+ if (x == NULL) {
+ if (obase != NULL) {
+ PyErr_SetString(PyExc_TypeError,
+ "int() missing string argument");
+ return NULL;
+ }
return PyLong_FromLong(0L);
+ }
if (obase == NULL)
return PyNumber_Long(x);
@@ -4277,7 +4283,7 @@
return NULL;
if (overflow || (base != 0 && base < 2) || base > 36) {
PyErr_SetString(PyExc_ValueError,
- "int() arg 2 must be >= 2 and <= 36");
+ "int() base must be >= 2 and <= 36");
return NULL;
}
--
Repository URL: http://hg.python.org/cpython
More information about the Python-checkins
mailing list