From 4e04c971fb3d855f3e81231a7649f398ff021c4c Mon Sep 17 00:00:00 2001 From: Doug Evans Date: Mon, 29 Nov 2010 23:20:58 +0000 Subject: [PATCH] * python/lib/gdb/printing.py (register_pretty_printer): Change printer-name:subprinter-name to printer-name;subprinter-name. * python/lib/gdb/command/pretty_printers.py (parse_printer_regexps): Ditto. (InfoPrettyPrinter, EnablePrettyPrinter, DisablePrettyPrinter): Ditto. doc/ * gdb.texinfo (Pretty-Printer Introduction): Change printer-name:subprinter-name to printer-name;subprinter-name. testsuite/ * gdb.python/py-pp-maint.exp: Change printer-name:subprinter-name to printer-name;subprinter-name. --- gdb/ChangeLog | 8 ++++++++ gdb/doc/ChangeLog | 5 +++++ gdb/doc/gdb.texinfo | 2 +- gdb/python/lib/gdb/command/pretty_printers.py | 10 +++++----- gdb/python/lib/gdb/printing.py | 10 +++++----- gdb/testsuite/ChangeLog | 5 +++++ gdb/testsuite/gdb.python/py-pp-maint.exp | 18 ++++++++++++------ 7 files changed, 41 insertions(+), 17 deletions(-) diff --git a/gdb/ChangeLog b/gdb/ChangeLog index d5a283bae66..0286cac408e 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,11 @@ +2010-11-29 Doug Evans + + * python/lib/gdb/printing.py (register_pretty_printer): Change + printer-name:subprinter-name to printer-name;subprinter-name. + * python/lib/gdb/command/pretty_printers.py (parse_printer_regexps): + Ditto. + (InfoPrettyPrinter, EnablePrettyPrinter, DisablePrettyPrinter): Ditto. + 2010-11-29 Tom Tromey * opencl-lang.c (lval_func_check_synthetic_pointer): New diff --git a/gdb/doc/ChangeLog b/gdb/doc/ChangeLog index df2449a87f8..6b4f2bb396a 100644 --- a/gdb/doc/ChangeLog +++ b/gdb/doc/ChangeLog @@ -1,3 +1,8 @@ +2010-11-29 Doug Evans + + * gdb.texinfo (Pretty-Printer Introduction): Change + printer-name:subprinter-name to printer-name;subprinter-name. + 2010-11-29 Phil Muldoon PR python/12199 diff --git a/gdb/doc/gdb.texinfo b/gdb/doc/gdb.texinfo index 28ea55d6453..422812c323e 100644 --- a/gdb/doc/gdb.texinfo +++ b/gdb/doc/gdb.texinfo @@ -8160,7 +8160,7 @@ pretty-printers with their names. If a pretty-printer can handle multiple data types, then its @dfn{subprinters} are the printers for the individual data types. Each such subprinter has its own name. -The format of the name is @var{printer-name}:@var{subprinter-name}. +The format of the name is @var{printer-name};@var{subprinter-name}. Pretty-printers are installed by @dfn{registering} them with @value{GDBN}. Typically they are automatically loaded and registered when the corresponding diff --git a/gdb/python/lib/gdb/command/pretty_printers.py b/gdb/python/lib/gdb/command/pretty_printers.py index 58a639aabdc..39559a9784d 100644 --- a/gdb/python/lib/gdb/command/pretty_printers.py +++ b/gdb/python/lib/gdb/command/pretty_printers.py @@ -28,7 +28,7 @@ def parse_printer_regexps(arg): arg: The arguments to the command. The format is: [object-regexp [name-regexp]]. Individual printers in a collection are named as - printer-name:subprinter-name. + printer-name;subprinter-name. Returns: The result is a 3-tuple of compiled regular expressions, except that @@ -48,7 +48,7 @@ def parse_printer_regexps(arg): if argc >= 1: object_regexp = argv[0] if argc >= 2: - name_subname = argv[1].split(":", 1) + name_subname = argv[1].split(";", 1) name_regexp = name_subname[0] if len(name_subname) == 2: subname_regexp = name_subname[1] @@ -92,7 +92,7 @@ class InfoPrettyPrinter(gdb.Command): NAME-REGEXP matches the name of the pretty-printer. Individual printers in a collection are named as - printer-name:subprinter-name. + printer-name;subprinter-name. """ def __init__ (self): @@ -328,7 +328,7 @@ class EnablePrettyPrinter (gdb.Command): NAME-REGEXP matches the name of the pretty-printer. Individual printers in a collection are named as - printer-name:subprinter-name. + printer-name;subprinter-name. """ def __init__(self): @@ -351,7 +351,7 @@ class DisablePrettyPrinter (gdb.Command): NAME-REGEXP matches the name of the pretty-printer. Individual printers in a collection are named as - printer-name:subprinter-name. + printer-name;subprinter-name. """ def __init__(self): diff --git a/gdb/python/lib/gdb/printing.py b/gdb/python/lib/gdb/printing.py index 0971375d2df..80aa2cfed9e 100644 --- a/gdb/python/lib/gdb/printing.py +++ b/gdb/python/lib/gdb/printing.py @@ -85,7 +85,7 @@ def register_pretty_printer(obj, printer): Raises: TypeError: A problem with the type of the printer. - ValueError: The printer's name contains a colon ":". + ValueError: The printer's name contains a semicolon ";". If the caller wants the printer to be listable and disableable, it must follow the PrettyPrinter API. This applies to the old way (functions) too. @@ -116,11 +116,11 @@ def register_pretty_printer(obj, printer): if hasattr(printer, "name"): if not isinstance(printer.name, basestring): raise TypeError("printer name is not a string") - # If printer provides a name, make sure it doesn't contain ":". - # Colon is used by the info/enable/disable pretty-printer commands + # If printer provides a name, make sure it doesn't contain ";". + # Semicolon is used by the info/enable/disable pretty-printer commands # to delimit subprinters. - if printer.name.find(":") >= 0: - raise ValueError("colon ':' in printer name") + if printer.name.find(";") >= 0: + raise ValueError("semicolon ';' in printer name") # Also make sure the name is unique. # Alas, we can't do the same for functions and __name__, they could # all have a canonical name like "lookup_function". diff --git a/gdb/testsuite/ChangeLog b/gdb/testsuite/ChangeLog index 3564704c090..b5aea2eb785 100644 --- a/gdb/testsuite/ChangeLog +++ b/gdb/testsuite/ChangeLog @@ -1,3 +1,8 @@ +2010-11-29 Doug Evans + + * gdb.python/py-pp-maint.exp: Change printer-name:subprinter-name to + printer-name;subprinter-name. + 2010-11-29 Tom Tromey * gdb.dwarf2/implptr.exp: New file. diff --git a/gdb/testsuite/gdb.python/py-pp-maint.exp b/gdb/testsuite/gdb.python/py-pp-maint.exp index 0aa79568e89..747458463de 100644 --- a/gdb/testsuite/gdb.python/py-pp-maint.exp +++ b/gdb/testsuite/gdb.python/py-pp-maint.exp @@ -80,14 +80,20 @@ gdb_test "print ss" " = a= b=<$hex>> b= b=<$hex>>" \ gdb_test "disable pretty-printer" \ "5 printers disabled.*0 of 5 printers enabled" +gdb_test "enable pretty-printer" \ + "5 printers enabled.*5 of 5 printers enabled" + gdb_test "disable pretty-printer global" \ - "0 printers disabled.*0 of 5 printers enabled" + "5 printers disabled.*0 of 5 printers enabled" + +gdb_test "enable pretty-printer" \ + "5 printers enabled.*5 of 5 printers enabled" gdb_test "disable pretty-printer global lookup_function_lookup_test" \ - "0 printers disabled.*0 of 5 printers enabled" + "1 printer disabled.*4 of 5 printers enabled" -gdb_test "disable pretty-printer global pp-test:.*" \ - "0 printers disabled.*0 of 5 printers enabled" +gdb_test "disable pretty-printer global pp-test;.*" \ + "4 printers disabled.*0 of 5 printers enabled" gdb_test "info pretty-printer global .*function" \ {.*function_lookup_test \[disabled\].*} @@ -110,10 +116,10 @@ gdb_test "enable pretty-printer global lookup_function_lookup_test" \ gdb_test "enable pretty-printer global pp-test" \ "0 printers enabled.*1 of 5 printers enabled" -gdb_test "enable pretty-printer global pp-test:.*ss.*" \ +gdb_test "enable pretty-printer global pp-test;.*ss.*" \ "2 printers enabled.*3 of 5 printers enabled" -gdb_test "enable pretty-printer global pp-test:.*s.*" \ +gdb_test "enable pretty-printer global pp-test;.*s.*" \ "2 printers enabled.*5 of 5 printers enabled" gdb_test "info pretty-printer" \ -- 2.30.2