* python/py-type.c (typy_make_iter): Add forward declaration.

(typy_fields_items):  Use the gdb.Type iterator.

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index 6c92b9ed1a9a71127c9750e208dae85cf0e4b422..29a61fe1c1090813db9a854a96146dd53bef35a8 100644 (file)
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,8 @@
+2011-10-04  Paul Koning  <paul_koning@dell.com>
+
+       * python/py-type.c (typy_make_iter): Add forward declaration.
+       (typy_fields_items):  Use the gdb.Type iterator.
+
 2011-10-04  Paul Koning  <paul_koning@dell.com>
 
        * NEWS: Add entry for Python gdb.Type mapping methods.

diff --git a/gdb/python/py-type.c b/gdb/python/py-type.c
index 76415b3019e663766bf687639cd86b0762b4d226..c7fd25b8cb861abc07ca8c46ef17be28c2c748b3 100644 (file)
--- a/gdb/python/py-type.c
+++ b/gdb/python/py-type.c
@@ -77,6 +77,9 @@ struct pyty_code
   const char *name;
 };
 
+/* Forward declarations.  */
+static PyObject *typy_make_iter (PyObject *self, enum gdbpy_iter_kind kind);
+
 #define ENTRY(X) { X, #X }
 
 static struct pyty_code pyty_codes[] =
@@ -290,40 +293,15 @@ make_fielditem (struct type *type, int i, enum gdbpy_iter_kind kind)
 static PyObject *
 typy_fields_items (PyObject *self, enum gdbpy_iter_kind kind)
 {
-  PyObject *result = NULL, *item = NULL;
-  int i;
-  struct type *type = ((type_object *) self)->type;
-  volatile struct gdb_exception except;
-
-  TRY_CATCH (except, RETURN_MASK_ALL)
-    {
-      CHECK_TYPEDEF (type);
-    }
-  GDB_PY_HANDLE_EXCEPTION (except);
-
-  /* We would like to make a tuple here, make fields immutable, and
-     then memoize the result (and perhaps make Field.type() lazy).
-     However, that can lead to cycles.  */
-  result = PyList_New (0);
-  if (result == NULL)
-    return NULL;
+  PyObject *result = NULL, *iter = NULL;
   
-  for (i = 0; i < TYPE_NFIELDS (type); ++i)
-    {
-      item = make_fielditem (type, i, kind);
-      if (!item)
-       goto fail;
-      if (PyList_Append (result, item))
-       goto fail;
-      Py_DECREF (item);
-    }
-
+  iter = typy_make_iter (self, kind);
+  if (iter == NULL)
+    return NULL;
+    
+  result = PySequence_List (iter);
+  Py_DECREF (iter);
   return result;
-
- fail:
-  Py_XDECREF (item);
-  Py_XDECREF (result);
-  return NULL;
 }
 
 /* Return a sequence of all fields.  Each field is a gdb.Field object.  */