[RFA 13/23] Replace tui_restore_gdbout with scoped_restore
Pedro Alves
palves@redhat.com
Fri Jun 2 18:34:00 GMT 2017
On 05/03/2017 11:46 PM, Tom Tromey wrote:
> This patch replaces tui_restore_gdbout (a cleaup function) with a use
> of scoped_restore. This one is broken out into its own patch because
> it might slightly change the behavior of gdb: it saves and restores
> pagination_enabled, whereas the tui_restore_gdbout unconditionally set
> pagination_enabled to 1; and I think this warrants closer review.
>
> 2017-05-02 Tom Tromey <tom@tromey.com>
>
> * tui/tui-regs.c (tui_restore_gdbout): Remove.
> (tui_register_format): Use scoped_restore.
I think I ran into this in last year's console work (in an
earlier prototype.). It just looks bogus to me to
unconditionally enable pagination.
Patch is OK.
Thanks,
Pedro Alves
More information about the Gdb-patches
mailing list