summaryrefslogtreecommitdiffstats
path: root/Objects/stringlib/formatter.h
diff options
context:
space:
mode:
authorEric Smith <eric@trueblade.com>2009-02-20 14:25:03 (GMT)
committerEric Smith <eric@trueblade.com>2009-02-20 14:25:03 (GMT)
commit5e5c0db5d71039549c71fe766aa01ce65af350bc (patch)
tree65989c3dae76e4f3758fd15e291d58ca4ddd49a5 /Objects/stringlib/formatter.h
parent247a9b87bd7ad5836b00e72212f9140f0faa5c9c (diff)
downloadcpython-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.h48
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;
}