[Python-checkins] r42405 - python/trunk/Python/exceptions.c

Thu Feb 16 08:11:33 CET 2006

Author: martin.v.loewis
Date: Thu Feb 16 08:11:33 2006
New Revision: 42405
Modified:
 python/trunk/Python/exceptions.c
Log:
Use PyString_FromFormat for formatting error messages.
Modified: python/trunk/Python/exceptions.c
==============================================================================

--- python/trunk/Python/exceptions.c	(original)
+++ python/trunk/Python/exceptions.c	Thu Feb 16 08:11:33 2006
@@ -1238,7 +1238,6 @@
 Py_ssize_t start;
 Py_ssize_t end;
 PyObject *reasonObj = NULL;
- char buffer[1000];
 PyObject *result = NULL;
 
 self = arg;
@@ -1260,32 +1259,30 @@
 
 if (end==start+1) {
 	int badchar = (int)PyUnicode_AS_UNICODE(objectObj)[start];
-	char *format;
+	char badchar_str[20];
 	if (badchar <= 0xff)
-	 format = "'%.400s' codec can't encode character u'\\x%02x' in position %d: %.400s";
+	 PyOS_snprintf(badchar_str, sizeof(badchar_str), "x%02x", badchar);
 	else if (badchar <= 0xffff)
-	 format = "'%.400s' codec can't encode character u'\\u%04x' in position %d: %.400s";
+	 PyOS_snprintf(badchar_str, sizeof(badchar_str), "u%04x", badchar);
 	else
-	 format = "'%.400s' codec can't encode character u'\\U%08x' in position %d: %.400s";
-	PyOS_snprintf(buffer, sizeof(buffer),
-	 format,
+	 PyOS_snprintf(badchar_str, sizeof(badchar_str), "U%08x", badchar);
+	result = PyString_FromFormat(
+	 "'%.400s' codec can't encode character u'\\%s' in position %zd: %.400s",
 	 PyString_AS_STRING(encodingObj),
-	 badchar,
+	 badchar_str,
 	 start,
 	 PyString_AS_STRING(reasonObj)
 	);
 }
 else {
-	/* XXX %zd? */
-	PyOS_snprintf(buffer, sizeof(buffer),
-	 "'%.400s' codec can't encode characters in position %d-%d: %.400s",
+	result = PyString_FromFormat(
+	 "'%.400s' codec can't encode characters in position %zd-%zd: %.400s",
 	 PyString_AS_STRING(encodingObj),
-	 (int)start,
-	 (int)(end-1),
+	 start,
+	 (end-1),
 	 PyString_AS_STRING(reasonObj)
 	);
 }
- result = PyString_FromString(buffer);
 
 error:
 Py_XDECREF(reasonObj);
@@ -1324,7 +1321,6 @@
 Py_ssize_t start;
 Py_ssize_t end;
 PyObject *reasonObj = NULL;
- char buffer[1000];
 PyObject *result = NULL;
 
 self = arg;
@@ -1345,26 +1341,28 @@
 	goto error;
 
 if (end==start+1) {
-	/* XXX %zd? */
-	PyOS_snprintf(buffer, sizeof(buffer),
-	 "'%.400s' codec can't decode byte 0x%02x in position %d: %.400s",
+	/* FromFormat does not support %02x, so format that separately */
+	char byte[4];
+	PyOS_snprintf(byte, sizeof(byte), "%02x", 
+		 ((int)PyString_AS_STRING(objectObj)[start])&0xff);
+	result = PyString_FromFormat(				 
+	 "'%.400s' codec can't decode byte 0x%s in position %zd: %.400s",
 	 PyString_AS_STRING(encodingObj),
-	 ((int)PyString_AS_STRING(objectObj)[start])&0xff,
-	 (int)start,
+	 byte,
+	 start,
 	 PyString_AS_STRING(reasonObj)
 	);
 }
 else {
-	/* XXX %zd? */
-	PyOS_snprintf(buffer, sizeof(buffer),
-	 "'%.400s' codec can't decode bytes in position %d-%d: %.400s",
+	result = PyString_FromFormat(
+	 "'%.400s' codec can't decode bytes in position %zd-%zd: %.400s",
 	 PyString_AS_STRING(encodingObj),
-	 (int)start,
-	 (int)(end-1),
+	 start,
+	 (end-1),
 	 PyString_AS_STRING(reasonObj)
 	);
 }
- result = PyString_FromString(buffer);
+
 
 error:
 Py_XDECREF(reasonObj);
@@ -1442,7 +1440,6 @@
 Py_ssize_t start;
 Py_ssize_t end;
 PyObject *reasonObj = NULL;
- char buffer[1000];
 PyObject *result = NULL;
 
 self = arg;
@@ -1461,31 +1458,28 @@
 
 if (end==start+1) {
 	int badchar = (int)PyUnicode_AS_UNICODE(objectObj)[start];
-	char *format;
-	/* XXX %zd? */
+	char badchar_str[20];
 	if (badchar <= 0xff)
-	 format = "can't translate character u'\\x%02x' in position %d: %.400s";
+	 PyOS_snprintf(badchar_str, sizeof(badchar_str), "x%02x", badchar);
 	else if (badchar <= 0xffff)
-	 format = "can't translate character u'\\u%04x' in position %d: %.400s";
+	 PyOS_snprintf(badchar_str, sizeof(badchar_str), "u%04x", badchar);
 	else
-	 format = "can't translate character u'\\U%08x' in position %d: %.400s";
-	PyOS_snprintf(buffer, sizeof(buffer),
-	 format,
-	 badchar,
-	 (int)start,
+	 PyOS_snprintf(badchar_str, sizeof(badchar_str), "U%08x", badchar);
+	result = PyString_FromFormat(
+ "can't translate character u'\\%s' in position %zd: %.400s",
+	 badchar_str,
+	 start,
 	 PyString_AS_STRING(reasonObj)
 	);
 }
 else {
-	/* XXX %zd? */
-	PyOS_snprintf(buffer, sizeof(buffer),
-	 "can't translate characters in position %d-%d: %.400s",
-	 (int)start,
-	 (int)(end-1),
+	result = PyString_FromFormat(
+	 "can't translate characters in position %zd-%zd: %.400s",
+	 start,
+	 (end-1),
 	 PyString_AS_STRING(reasonObj)
 	);
 }
- result = PyString_FromString(buffer);
 
 error:
 Py_XDECREF(reasonObj);
