mirrors/cpython
1
0
Fork 0
mirror of https://github.com/python/cpython.git synced 2025-11-03 03:22:27 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 8

Issue 6089: str.format raises SystemError.

Browse source
This commit is contained in:
Eric Smith 2009-05-23 13:56:13 +00:00
parent 8254d39840
commit 4b94b192ff
3 changed files with 11 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

4
Lib/test/test_str.py
View file

@ -351,6 +351,10 @@ class StrTest(
self.assertRaises(IndexError, "{:s}".format) self.assertRaises(IndexError, "{:s}".format)
self.assertRaises(IndexError, "{}".format) self.assertRaises(IndexError, "{}".format)
# issue 6089
self.assertRaises(ValueError, "{0[0]x}".format, [None])
self.assertRaises(ValueError, "{0[0](10)}".format, [None])
# can't have a replacement on the field name portion # can't have a replacement on the field name portion
self.assertRaises(TypeError, '{0[{1}]}'.format, 'abcdefg', 4) self.assertRaises(TypeError, '{0[{1}]}'.format, 'abcdefg', 4)

4
Lib/test/test_unicode.py
View file

@ -1100,6 +1100,10 @@ class UnicodeTest(
self.assertRaises(IndexError, u"{:s}".format) self.assertRaises(IndexError, u"{:s}".format)
self.assertRaises(IndexError, u"{}".format) self.assertRaises(IndexError, u"{}".format)
# issue 6089
self.assertRaises(ValueError, u"{0[0]x}".format, [None])
self.assertRaises(ValueError, u"{0[0](10)}".format, [None])
# can't have a replacement on the field name portion # can't have a replacement on the field name portion
self.assertRaises(TypeError, u'{0[{1}]}'.format, u'abcdefg', 4) self.assertRaises(TypeError, u'{0[{1}]}'.format, u'abcdefg', 4)

5
Objects/stringlib/string_format.h
View file

@ -375,8 +375,9 @@ FieldNameIterator_next(FieldNameIterator *self, int *is_attribute,
*name_idx = get_integer(name); *name_idx = get_integer(name);
break; break;
default: default:
/* interal error, can't get here */ /* Invalid character follows ']' */
assert(0); PyErr_SetString(PyExc_ValueError, "Only '.' or '[' may "
"follow ']' in format field specifier");
return 0; return 0;
} }