diff options
author | Eric Smith <eric@trueblade.com> | 2009-02-20 14:25:03 (GMT) |
---|---|---|
committer | Eric Smith <eric@trueblade.com> | 2009-02-20 14:25:03 (GMT) |
commit | 5e5c0db5d71039549c71fe766aa01ce65af350bc (patch) | |
tree | 65989c3dae76e4f3758fd15e291d58ca4ddd49a5 /Objects/stringlib/formatter.h | |
parent | 247a9b87bd7ad5836b00e72212f9140f0faa5c9c (diff) | |
download | cpython-5e5c0db5d71039549c71fe766aa01ce65af350bc.zip cpython-5e5c0db5d71039549c71fe766aa01ce65af350bc.tar.gz cpython-5e5c0db5d71039549c71fe766aa01ce65af350bc.tar.bz2 |
Merged revisions 69806 via svnmerge from
svn+ssh://pythondev@svn.python.org/python/trunk
........
r69806 | eric.smith | 2009-02-20 09:02:36 -0500 (Fri, 20 Feb 2009) | 1 line
Issue #5247: Improve error message when unknown format codes are used when using str.format() with str, int, and float arguments.
........
Diffstat (limited to 'Objects/stringlib/formatter.h')
-rw-r--r-- | Objects/stringlib/formatter.h | 48 |
1 files changed, 31 insertions, 17 deletions
diff --git a/Objects/stringlib/formatter.h b/Objects/stringlib/formatter.h index aa99123..2e3e5a6 100644 --- a/Objects/stringlib/formatter.h +++ b/Objects/stringlib/formatter.h @@ -15,6 +15,34 @@ #define ALLOW_PARENS_FOR_SIGN 0 +/* Raises an exception about an unknown presentation type for this + * type. */ + +static void +unknown_presentation_type(STRINGLIB_CHAR presentation_type, + const char* type_name) +{ +#if STRINGLIB_IS_UNICODE + /* If STRINGLIB_CHAR is Py_UNICODE, %c might be out-of-range, + hence the two cases. If it is char, gcc complains that the + condition below is always true, hence the ifdef. */ + if (presentation_type > 32 && presentation_type < 128) +#endif + PyErr_Format(PyExc_ValueError, + "Unknown format code '%c' " + "for object of type '%.200s'", + presentation_type, + type_name); +#if STRINGLIB_IS_UNICODE + else + PyErr_Format(PyExc_ValueError, + "Unknown format code '\\x%x' " + "for object of type '%.200s'", + (unsigned int)presentation_type, + type_name); +#endif +} + /* get_integer consumes 0 or more decimal digit characters from an input string, updates *result with the corresponding positive @@ -854,19 +882,7 @@ FORMAT_STRING(PyObject *obj, break; default: /* unknown */ - #if STRINGLIB_IS_UNICODE - /* If STRINGLIB_CHAR is Py_UNICODE, %c might be out-of-range, - hence the two cases. If it is char, gcc complains that the - condition below is always true, hence the ifdef. */ - if (format.type > 32 && format.type <128) - #endif - PyErr_Format(PyExc_ValueError, "Unknown conversion type %c", - (char)format.type); - #if STRINGLIB_IS_UNICODE - else - PyErr_Format(PyExc_ValueError, "Unknown conversion type '\\x%x'", - (unsigned int)format.type); - #endif + unknown_presentation_type(format.type, obj->ob_type->tp_name); goto done; } @@ -928,8 +944,7 @@ format_int_or_long(PyObject* obj, default: /* unknown */ - PyErr_Format(PyExc_ValueError, "Unknown conversion type %c", - format.type); + unknown_presentation_type(format.type, obj->ob_type->tp_name); goto done; } @@ -1031,8 +1046,7 @@ FORMAT_FLOAT(PyObject *obj, default: /* unknown */ - PyErr_Format(PyExc_ValueError, "Unknown conversion type %c", - format.type); + unknown_presentation_type(format.type, obj->ob_type->tp_name); goto done; } |