[Bug guile/17177] New: Replace *pretty-printers* global with access functions

[Bug guile/17177] New: Replace *pretty-printers* global with access functions

[dje at google dot com]

https://sourceware.org/bugzilla/show_bug.cgi?id=17177

            Bug ID: 17177
           Summary: Replace *pretty-printers* global with access functions
           Product: gdb
           Version: HEAD
            Status: NEW
          Severity: normal
          Priority: P2
         Component: guile
          Assignee: unassigned at sourceware dot org
          Reporter: dje at google dot com

Instead of exporting the list of global pretty-printers as a global variable,
we should export functions to get/set the value.

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Bug guile/17177] Replace *pretty-printers* global with access functions

[cvs-commit at gcc dot gnu.org]

https://sourceware.org/bugzilla/show_bug.cgi?id=17177

--- Comment #1 from cvs-commit at gcc dot gnu.org <cvs-commit at gcc dot gnu.org> ---
This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "gdb and binutils".

The branch, master has been updated
       via  ee7333ae06f46f3a6909a0059c2f0bd441a54bfe (commit)
      from  74edf51613b507d1f27d66360cd8fdd8a253e88a (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=ee7333ae06f46f3a6909a0059c2f0bd441a54bfe

commit ee7333ae06f46f3a6909a0059c2f0bd441a54bfe
Author: Doug Evans <xdje42@gmail.com>
Date:   Sat Jul 26 14:58:58 2014 -0700

    PR guile/17177

        * guile/lib/gdb.scm (pretty-printers): Export.
        (set-pretty-printers!): Export.
        * guile/lib/gdb/printing.scm (gdb module): Update.
        (prepend-pretty-printer!, append-pretty-printer!): Update.
        * guile/scm-pretty-print.c (pretty_printer_list_name): Delete.
        (pretty_printer_list_var): Delete.
        (pretty_printer_list): New static global.
        (gdbscm_pretty_printers): New function.
        (gdbscm_set_pretty_printers_x): New function.
        (ppscm_find_pretty_printer_from_gdb): Update.
        (pretty_printer_functions): Add pretty-printers, set-pretty-printers!.
        (gdbscm_initialize_pretty_printers): Update.

        doc/
        * guile.texi (Guile Pretty Printing API): Fix typo.
        Document set-pretty-printers!, pretty-printers.
        (Selecting Guile Pretty-Printers): Update.
        Mention program-space based pretty-printers.

-----------------------------------------------------------------------

Summary of changes:
 gdb/ChangeLog                  |   16 ++++++++++++
 gdb/doc/ChangeLog              |    8 ++++++
 gdb/doc/guile.texi             |   40 ++++++++++++++++++++++--------
 gdb/guile/lib/gdb.scm          |    2 +
 gdb/guile/lib/gdb/printing.scm |    7 +++--
 gdb/guile/scm-pretty-print.c   |   51 ++++++++++++++++++++++++++++-----------
 6 files changed, 95 insertions(+), 29 deletions(-)

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Bug guile/17177] Replace *pretty-printers* global with access functions

[cvs-commit at gcc dot gnu.org]

https://sourceware.org/bugzilla/show_bug.cgi?id=17177

--- Comment #2 from cvs-commit at gcc dot gnu.org <cvs-commit at gcc dot gnu.org> ---
This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "gdb and binutils".

The branch, gdb-7.8-branch has been updated
       via  68455456ca7b1f35ce97f62c50b466e2fc9cce8f (commit)
      from  b13f24b87c227ef532cac27fc81f85b60fba2832 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=68455456ca7b1f35ce97f62c50b466e2fc9cce8f

commit 68455456ca7b1f35ce97f62c50b466e2fc9cce8f
Author: Doug Evans <xdje42@gmail.com>
Date:   Sat Jul 26 20:39:33 2014 -0700

    PR guile/17177

        * guile/lib/gdb.scm (pretty-printers): Export.
        (set-pretty-printers!): Export.
        * guile/lib/gdb/printing.scm (gdb module): Update.
        (prepend-pretty-printer!, append-pretty-printer!): Update.
        * guile/scm-pretty-print.c (pretty_printer_list_name): Delete.
        (pretty_printer_list_var): Delete.
        (pretty_printer_list): New static global.
        (gdbscm_pretty_printers): New function.
        (gdbscm_set_pretty_printers_x): New function.
        (ppscm_find_pretty_printer_from_gdb): Update.
        (pretty_printer_functions): Add pretty-printers, set-pretty-printers!.
        (gdbscm_initialize_pretty_printers): Update.

        doc/
        * guile.texi (Guile Pretty Printing API): Fix typo.
        Document set-pretty-printers!, pretty-printers.
        (Selecting Guile Pretty-Printers): Update.
        Mention program-space based pretty-printers.

-----------------------------------------------------------------------

Summary of changes:
 gdb/ChangeLog                  |   16 ++++++++++++
 gdb/doc/ChangeLog              |    8 ++++++
 gdb/doc/guile.texi             |   40 ++++++++++++++++++++++--------
 gdb/guile/lib/gdb.scm          |    2 +
 gdb/guile/lib/gdb/printing.scm |    7 +++--
 gdb/guile/scm-pretty-print.c   |   51 ++++++++++++++++++++++++++++-----------
 6 files changed, 95 insertions(+), 29 deletions(-)

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Bug guile/17177] Replace *pretty-printers* global with access functions

[xdje42 at gmail dot com]

https://sourceware.org/bugzilla/show_bug.cgi?id=17177

Doug Evans <xdje42 at gmail dot com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
                 CC|                            |xdje42 at gmail dot com
         Resolution|---                         |FIXED

--- Comment #3 from Doug Evans <xdje42 at gmail dot com> ---
patch applied

-- 
You are receiving this mail because:
You are on the CC list for the bug.