PR python/17364

gdb/ChangeLog:

	* python/lib/gdb/__init__.py (packages): Add "printer".
	* python/lib/gdb/command/bound_registers.py: Moved to ...
	* python/lib/gdb/printer/bound_registers.py: ... here.
	Add printer to global set of builtin printers.  Rename printer from
	"bound" to "mpx_bound128".
	* python/lib/gdb/printing.py (_builtin_pretty_printers): New global,
	registered as global "builtin" printer.
	(add_builtin_pretty_printer): New function.
	* data-directory/Makefile.in (PYTHON_FILE_LIST): Update, and add
	gdb/printer/__init__.py.

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index fd9cb7d22cec182549a2fe2e16ebf634298ba234..47a4d42b860c9494cd153813daa07801b14f2c45 100644 (file)
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,18 @@
+2014-10-15  Doug Evans  <dje@google.com>
+           Walfred Tedeschi  <walfred.tedeschi@intel.com>
+
+       PR python/17364
+       * python/lib/gdb/__init__.py (packages): Add "printer".
+       * python/lib/gdb/command/bound_registers.py: Moved to ...
+       * python/lib/gdb/printer/bound_registers.py: ... here.
+       Add printer to global set of builtin printers.  Rename printer from
+       "bound" to "mpx_bound128".
+       * python/lib/gdb/printing.py (_builtin_pretty_printers): New global,
+       registered as global "builtin" printer.
+       (add_builtin_pretty_printer): New function.
+       * data-directory/Makefile.in (PYTHON_FILE_LIST): Update, and add
+       gdb/printer/__init__.py.
+
 2014-10-15  Iain Buclaw  <ibuclaw@gdcproject.org>
 
        * Makefile.in (SFILES): Remove d-support.c.

diff --git a/gdb/data-directory/Makefile.in b/gdb/data-directory/Makefile.in
index 1e8cd4b666a2f4c521fd03c945ba9398cb7c0ba0..00c70bb2800aaa26fe74f1698043c6ef33e7ba14 100644 (file)
--- a/gdb/data-directory/Makefile.in
+++ b/gdb/data-directory/Makefile.in
@@ -64,7 +64,6 @@ PYTHON_FILE_LIST = \
        gdb/printing.py \
        gdb/prompt.py \
        gdb/xmethod.py \
-       gdb/command/bound_registers.py \
        gdb/command/__init__.py \
        gdb/command/xmethods.py \
        gdb/command/frame_filters.py \
@@ -74,7 +73,9 @@ PYTHON_FILE_LIST = \
        gdb/command/explore.py \
        gdb/function/__init__.py \
        gdb/function/caller_is.py \
-       gdb/function/strfns.py
+       gdb/function/strfns.py \
+       gdb/printer/__init__.py \
+       gdb/printer/bound_registers.py
 
 @HAVE_PYTHON_TRUE@PYTHON_FILES = $(PYTHON_FILE_LIST)
 @HAVE_PYTHON_FALSE@PYTHON_FILES =

diff --git a/gdb/python/lib/gdb/__init__.py b/gdb/python/lib/gdb/__init__.py
index 557e1681c627186f17e767d634e99407b5b6b3a0..8c6eee2170ce4ea03b658fe94147f99df799e4b8 100644 (file)
--- a/gdb/python/lib/gdb/__init__.py
+++ b/gdb/python/lib/gdb/__init__.py
@@ -81,7 +81,8 @@ PYTHONDIR = os.path.dirname(os.path.dirname(__file__))
 
 packages = [
     'function',
-    'command'
+    'command',
+    'printer'
 ]
 
 # pkgutil.iter_modules is not available prior to Python 2.6.  Instead,

diff --git a/gdb/python/lib/gdb/printer/__init__.py b/gdb/python/lib/gdb/printer/__init__.py
new file mode 100644 (file)
index 0000000..04c0c7d
--- /dev/null
+++ b/gdb/python/lib/gdb/printer/__init__.py
@@ -0,0 +1,14 @@
+# Copyright (C) 2014 Free Software Foundation, Inc.
+
+# This program is free software; you can redistribute it and/or modify
+# it under the terms of the GNU General Public License as published by
+# the Free Software Foundation; either version 3 of the License, or
+# (at your option) any later version.
+#
+# This program is distributed in the hope that it will be useful,
+# but WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+# GNU General Public License for more details.
+#
+# You should have received a copy of the GNU General Public License
+# along with this program.  If not, see <http://www.gnu.org/licenses/>.

diff --git a/gdb/python/lib/gdb/command/bound_registers.py b/gdb/python/lib/gdb/printer/bound_registers.py
similarity index 62%
rename from gdb/python/lib/gdb/command/bound_registers.py
rename to gdb/python/lib/gdb/printer/bound_registers.py
index 24d4c45a1717bf6c0cdc7cd3a066ab1da11ecc2c..25e6e8060ffffa0ca7bbe1c61b02087b21744513 100644 (file)
--- a/gdb/python/lib/gdb/command/bound_registers.py
+++ b/gdb/python/lib/gdb/printer/bound_registers.py
@@ -1,4 +1,4 @@
-# Pretty-printer utilities.
+# Pretty-printers for bounds registers.
 # Copyright (C) 2013-2014 Free Software Foundation, Inc.
 
 # This program is free software; you can redistribute it and/or modify
@@ -16,8 +16,8 @@
 
 import gdb.printing
 
-class BoundPrinter:
-    """Adds size field to a _rawbound128 type."""
+class MpxBound128Printer:
+    """Adds size field to a mpx __gdb_builtin_type_bound128 type."""
 
     def __init__ (self, val):
         self.val = val
@@ -31,15 +31,6 @@ class BoundPrinter:
         result = '{lbound = %s, ubound = %s} : size %s' % (lower, upper, size)
         return result
 
-# There are two pattern matching used: first one is related to a library
-# second is related to the type. Since we are displaying a register all
-# libraries are accepted. Type to be processed is the same present
-# in the xml file.
-
-def build_pretty_printer ():
-    pp = gdb.printing.RegexpCollectionPrettyPrinter (".*")
-    pp.add_printer ('bound', '^__gdb_builtin_type_bound128', BoundPrinter)
-    return pp
-
-gdb.printing.register_pretty_printer (gdb.current_objfile (),
-                                      build_pretty_printer ())
+gdb.printing.add_builtin_pretty_printer ('mpx_bound128',
+                                         '^__gdb_builtin_type_bound128',
+                                         MpxBound128Printer)

diff --git a/gdb/python/lib/gdb/printing.py b/gdb/python/lib/gdb/printing.py
index 2940b939056e956e9629d13614255079286870e9..ff5250a8093608e37b56c4733323b42aa8bc5011 100644 (file)
--- a/gdb/python/lib/gdb/printing.py
+++ b/gdb/python/lib/gdb/printing.py
@@ -263,3 +263,17 @@ class FlagEnumerationPrinter(PrettyPrinter):
             return _EnumInstance(self.enumerators, val)
         else:
             return None
+
+
+# Builtin pretty-printers.
+# The set is defined as empty, and files in printing/*.py add their printers
+# to this with add_builtin_pretty_printer.
+
+_builtin_pretty_printers = RegexpCollectionPrettyPrinter("builtin")
+
+register_pretty_printer(None, _builtin_pretty_printers)
+
+# Add a builtin pretty-printer.
+
+def add_builtin_pretty_printer(name, regexp, printer):
+    _builtin_pretty_printers.add_printer(name, regexp, printer)