This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
[PATCH, doc RFA] New attribute "progspace" for python gdb.Objfile objects
- From: Doug Evans <dje at google dot com>
- To: gdb-patches at sourceware dot org, eliz at gnu dot org
- Date: Mon, 13 Oct 2014 15:10:17 -0700
- Subject: [PATCH, doc RFA] New attribute "progspace" for python gdb.Objfile objects
- Authentication-results: sourceware.org; auth=none
Hi.
This patch adds a new attribute "progspace" for python gdb.Objfile objects.
While one could just always use gdb.current_progspace(), having to refer
to global state is generally a no-no in an API. The progspace is
recorded with the objfile, this patch exposes it.
Regression tested on amd64-linux.
2014-10-13 Doug Evans <dje@google.com>
* NEWS: Mention new gdb.Objfile.progspace attribute.
* python/py-objfile.c (objfpy_get_progspace): New function.
(objfile_getset): New entry for "progspace".
doc/
* python.texi (Objfiles In Python): Document new progspace attribute.
testsuite/
* gdb.python/py-objfile.exp: Test progspace attribute.
diff --git a/gdb/NEWS b/gdb/NEWS
index 5de0a33..9d056b9 100644
--- a/gdb/NEWS
+++ b/gdb/NEWS
@@ -9,6 +9,8 @@
** You can now access frame registers from Python scripts.
** New attribute 'producer' for gdb.Symtab objects.
+ ** gdb.Objfile objects have a new attribute "progspace",
+ which is the gdb.Progspace object of the containing program space.
* New Python-based convenience functions:
diff --git a/gdb/doc/python.texi b/gdb/doc/python.texi
index 81ec11b..84d6637 100644
--- a/gdb/doc/python.texi
+++ b/gdb/doc/python.texi
@@ -3392,6 +3392,11 @@ class.
The file name of the objfile as a string.
@end defvar
+@defvar Objfile.progspace
+The containing program space of the objfile as a @code{gdb.Progspace}
+object. @xref{Progspaces In Python}.
+@end defvar
+
@defvar Objfile.pretty_printers
The @code{pretty_printers} attribute is a list of functions. It is
used to look up pretty-printers. A @code{Value} is passed to each
diff --git a/gdb/python/py-objfile.c b/gdb/python/py-objfile.c
index df29691..cda6a48 100644
--- a/gdb/python/py-objfile.c
+++ b/gdb/python/py-objfile.c
@@ -62,6 +62,18 @@ objfpy_get_filename (PyObject *self, void *closure)
Py_RETURN_NONE;
}
+/* An Objfile method which returns the objfile's progspace, or None. */
+
+static PyObject *
+objfpy_get_progspace (PyObject *self, void *closure)
+{
+ objfile_object *obj = (objfile_object *) self;
+
+ if (obj->objfile)
+ return pspace_to_pspace_object (obj->objfile->pspace);
+ Py_RETURN_NONE;
+}
+
static void
objfpy_dealloc (PyObject *o)
{
@@ -338,6 +350,8 @@ static PyGetSetDef objfile_getset[] =
{
{ "filename", objfpy_get_filename, NULL,
"The objfile's filename, or None.", NULL },
+ { "progspace", objfpy_get_progspace, NULL,
+ "The objfile's progspace, or None.", NULL },
{ "pretty_printers", objfpy_get_printers, objfpy_set_printers,
"Pretty printers.", NULL },
{ "frame_filters", objfpy_get_frame_filters,
diff --git a/gdb/testsuite/gdb.python/py-objfile.exp b/gdb/testsuite/gdb.python/py-objfile.exp
index 1d2f550..8796170 100644
--- a/gdb/testsuite/gdb.python/py-objfile.exp
+++ b/gdb/testsuite/gdb.python/py-objfile.exp
@@ -39,6 +39,8 @@ gdb_py_test_silent_cmd "python objfile = sym\[0\].symtab.objfile" \
gdb_test "python print (objfile.filename)" ".*py-objfile.*" \
"Get objfile file name"
+gdb_test "python print (objfile.progspace)" "<gdb\.Progspace object at .*>" \
+ "Get objfile program space"
gdb_test "python print (objfile.is_valid())" "True" \
"Get objfile validity"
gdb_unload