[Python-3000-checkins] r57584 - in python/branches/py3k/Objects: stringlib/string_format.h unicodeobject.c

[eric.smith]

Author: eric.smith
Date: Tue Aug 28 01:30:47 2007
New Revision: 57584

Modified:
   python/branches/py3k/Objects/stringlib/string_format.h
   python/branches/py3k/Objects/unicodeobject.c
Log:
Cleanup in anticipation of moving formatteriterator and fieldnameiterator into stringlib/string_format.h.

Modified: python/branches/py3k/Objects/stringlib/string_format.h
==============================================================================
--- python/branches/py3k/Objects/stringlib/string_format.h	(original)
+++ python/branches/py3k/Objects/stringlib/string_format.h	Tue Aug 28 01:30:47 2007
@@ -55,24 +55,6 @@
 }
 
 /************************************************************************/
-/***********      Error handling and exception generation  **************/
-/************************************************************************/
-
-/*
-    Most of our errors are value errors, because to Python, the
-    format string is a "value".  Also, it's convenient to return
-    a NULL when we are erroring out.
-
-    XXX: need better error handling, per PEP 3101.
-*/
-static void *
-SetError(const char *s)
-{
-    /* PyErr_Format always returns NULL */
-    return PyErr_Format(PyExc_ValueError, "%s in format string", s);
-}
-
-/************************************************************************/
 /***********    Output string management functions       ****************/
 /************************************************************************/
 
@@ -187,7 +169,7 @@
 getattr(PyObject *obj, SubString *name)
 {
     PyObject *newobj;
-    PyObject *str = STRINGLIB_NEW(name->ptr, name->end - name->ptr);
+    PyObject *str = SubString_new_object(name);
     if (str == NULL)
         return NULL;
     newobj = PyObject_GetAttr(obj, str);
@@ -220,7 +202,7 @@
 getitem_str(PyObject *obj, SubString *name)
 {
     PyObject *newobj;
-    PyObject *str = STRINGLIB_NEW(name->ptr, name->end - name->ptr);
+    PyObject *str = SubString_new_object(name);
     if (str == NULL)
         return NULL;
     newobj = PyObject_GetItem(obj, str);
@@ -407,7 +389,7 @@
 
     if (index == -1) {
         /* look up in kwargs */
-        PyObject *key = STRINGLIB_NEW(first.ptr, first.end - first.ptr);
+        PyObject *key = SubString_new_object(&first);
         if (key == NULL)
             goto error;
         if ((kwargs == NULL) || (obj = PyDict_GetItem(kwargs, key)) == NULL) {
@@ -719,11 +701,13 @@
         len = self->str.ptr - start;
 
         if ((c == '}') && (at_end || (c != *self->str.ptr))) {
-            SetError("Single } encountered");
+            PyErr_SetString(PyExc_ValueError, "Single '}' encountered "
+                            "in format string");
             return 0;
         }
         if (at_end && c == '{') {
-            SetError("Single { encountered");
+            PyErr_SetString(PyExc_ValueError, "Single '{' encountered "
+                            "in format string");
             return 0;
         }
         if (!at_end) {

Modified: python/branches/py3k/Objects/unicodeobject.c
==============================================================================
--- python/branches/py3k/Objects/unicodeobject.c	(original)
+++ python/branches/py3k/Objects/unicodeobject.c	Tue Aug 28 01:30:47 2007
@@ -8209,7 +8209,7 @@
                 if (idx != -1)
                         obj = PyInt_FromSsize_t(idx);
                 else
-                        obj = STRINGLIB_NEW(name.ptr, name.end - name.ptr);
+                        obj = SubString_new_object(&name);
                 if (obj == NULL)
                         goto error;
 
@@ -8301,7 +8301,7 @@
                 first_obj = PyInt_FromSsize_t(first_idx);
         else
                 /* convert "first" into a string object */
-                first_obj = STRINGLIB_NEW(first.ptr, first.end - first.ptr);
+                first_obj = SubString_new_object(&first);
         if (first_obj == NULL)
                 goto error;