public inbox for gdb-patches@sourceware.org
 help / color / mirror / Atom feed
* [PATCH 00/66] Clean up the TUI
@ 2019-06-23 22:43 Tom Tromey
  2019-06-23 22:43 ` [PATCH 12/66] Don't use TUI_DISASM_WIN in tui_disasm_window method Tom Tromey
                   ` (50 more replies)
  0 siblings, 51 replies; 85+ messages in thread
From: Tom Tromey @ 2019-06-23 22:43 UTC (permalink / raw)
  To: gdb-patches

My first real encounter with the TUI code was last year, when I added
syntax styling to it.

On the one hand, this experience convinced me that this code was some
of the ugliest in gdb.  It uses two different unions at different
spots in the conceptual class hierarchy, leading to very obscure code.
It also hard-codes both the possible window types but also the
qpossible layout types in a brittle and unextensible way.

On the other hand, working on the TUI changed my mind about the TUI
generally.  I've become a fan.  I think it would be good to improve
the TUI for users, but to do that I think we first have to improve its
code.

This series is a start at this.  It's already quite long,
unfortunately, and yet is still incomplete.  Still, I think it is
progress.  This series:

* Splits tui_win_info into subclasses, one per window type.

* Adds virtual methods, particularly in an attempt to remove all the
  code that switches based on the type of the window.  This is
  important to make it simpler to add new window types.

* Changes tui_gen_win_info into a real base class for tui_win_info.

* Removes the uses of unions from the window class hierarchy.

* Simplifies and C++-ifies various other minor things along the way.

There is still a ways to go, I only stopped here in the interest of
review sanity.  In particular:

* Window layouts should be first-class entities that do not involve
  hard-coding the possible layouts in the C++.

* tui_win_list should be changed to allow multiple types of windows.

* The few remaining checks of the various *_WIN constants should be
  removed.

I think once these things are done it should be possible to move on to
making useful user-visible changes, like allowing custom layouts, or
allowing custom windows to be written in Python.

I tested this by trying the various TUI features by hand.  However,
I'm not sure I managed to test them all.  I've also run the gdb.tui
part of the test suite, though that does not really test very much.

This series fixes one latent bug that I found with valgrind.  I found
two other existing bugs as well, but those I simply filed.

Tom


^ permalink raw reply	[flat|nested] 85+ messages in thread

end of thread, other threads:[~2019-06-25 13:51 UTC | newest]

Thread overview: 85+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-06-23 22:43 [PATCH 00/66] Clean up the TUI Tom Tromey
2019-06-23 22:43 ` [PATCH 12/66] Don't use TUI_DISASM_WIN in tui_disasm_window method Tom Tromey
2019-06-23 22:43 ` [PATCH 01/66] Use new and delete for TUI windows Tom Tromey
2019-06-23 22:43 ` [PATCH 13/66] Remove tui_clear_win_detail Tom Tromey
2019-06-23 22:43 ` [PATCH 05/66] Simplify command window creation Tom Tromey
2019-06-23 22:43 ` [PATCH 08/66] Remove tui_list Tom Tromey
2019-06-24 14:12   ` Pedro Alves
2019-06-24 15:12     ` Ruslan Kabatsayev
2019-06-24 16:05       ` Pedro Alves
2019-06-24 20:47     ` Tom Tromey
2019-06-23 22:43 ` [PATCH 10/66] Create tui_disasm_window Tom Tromey
2019-06-23 22:44 ` [PATCH 21/66] Introduce make_visible method Tom Tromey
2019-06-23 22:44 ` [PATCH 16/66] Remove struct tui_command_info Tom Tromey
2019-06-23 22:44 ` [PATCH 15/66] Remove struct tui_source_info Tom Tromey
2019-06-23 22:44 ` [PATCH 26/66] Introduce set_new_height method Tom Tromey
2019-06-23 22:44 ` [PATCH 25/66] Introduce the refresh_all method Tom Tromey
2019-06-23 22:44 ` [PATCH 17/66] Remove struct tui_data_info Tom Tromey
2019-06-23 22:44 ` [PATCH 27/66] Introduce max_height method Tom Tromey
2019-06-23 22:44 ` [PATCH 20/66] Remove an unneeded NULL check Tom Tromey
2019-06-23 22:44 ` [PATCH 28/66] Remove redundant check from make_visible Tom Tromey
2019-06-23 22:44 ` [PATCH 24/66] Introduce two TUI source window methods Tom Tromey
2019-06-23 22:44 ` [PATCH 06/66] Simplify source and disassembly window creation Tom Tromey
2019-06-23 22:44 ` [PATCH 14/66] Introduce has_locator method Tom Tromey
2019-06-24 14:13   ` Pedro Alves
2019-06-24 20:50     ` Tom Tromey
2019-06-23 22:44 ` [PATCH 04/66] Remove an unnecessary NULL check from the TUI Tom Tromey
2019-06-23 22:44 ` [PATCH 02/66] Add destructor to tui_win_info Tom Tromey
2019-06-23 22:44 ` [PATCH 07/66] Introduce tui_win_info::clear_detail method Tom Tromey
2019-06-23 22:44 ` [PATCH 11/66] Introduce methods for scrolling Tom Tromey
2019-06-23 22:44 ` [PATCH 09/66] Split the tui_win_info destructor Tom Tromey
2019-06-23 22:44 ` [PATCH 03/66] Create subclasses for different window types Tom Tromey
2019-06-24 22:21   ` Pedro Alves
2019-06-25 13:51     ` Tom Tromey
2019-06-23 22:44 ` [PATCH 29/66] Introduce set_highlight method Tom Tromey
2019-06-23 22:44 ` [PATCH 22/66] Use bool for visibility Tom Tromey
2019-06-23 22:44 ` [PATCH 23/66] Introduce the refresh method Tom Tromey
2019-06-23 22:44 ` [PATCH 18/66] Change tui_data_window::display_regs to bool Tom Tromey
2019-06-23 22:44 ` [PATCH 19/66] Inline constructors and initializers Tom Tromey
2019-06-23 23:25 ` [PATCH 41/66] Move make_visible method to tui_gen_win_info Tom Tromey
2019-06-23 23:25 ` [PATCH 40/66] Remove tui_scroll_direction enum Tom Tromey
2019-06-24 14:13   ` Pedro Alves
2019-06-24 20:51     ` Tom Tromey
2019-06-23 23:25 ` [PATCH 45/66] Introduce tui_win_info::update_tab_width Tom Tromey
2019-06-23 23:25 ` [PATCH 42/66] Introduce tui_gen_win_info::reset method Tom Tromey
2019-06-23 23:25 ` [PATCH 35/66] Introduce tui_gen_win_info::name method Tom Tromey
2019-06-23 23:25 ` [PATCH 34/66] Change tui_update_source_window for better type safety Tom Tromey
2019-06-23 23:26 ` [PATCH 48/66] Remove tui_alloc_win_info Tom Tromey
2019-06-23 23:26 ` [PATCH 37/66] Remove NULL check from tui_set_exec_info_content Tom Tromey
2019-06-23 23:26 ` [PATCH 43/66] Remove some TUI static allocations Tom Tromey
2019-06-23 23:26 ` [PATCH 39/66] Change more TUI functions to take a tui_source_window_base Tom Tromey
2019-06-23 23:26 ` [PATCH 38/66] Change tui_set_exec_info_content to return void Tom Tromey
2019-06-23 23:26 ` [PATCH 30/66] Change tui_which_element::data_window to be a pointer Tom Tromey
2019-06-23 23:26 ` [PATCH 46/66] Introduce tui_win_info::make_visible_with_new_height Tom Tromey
2019-06-23 23:26 ` [PATCH 31/66] Use new and delete for tui_gen_win_info Tom Tromey
2019-06-23 23:26 ` [PATCH 33/66] Introduce refresh_window method Tom Tromey
2019-06-23 23:26 ` [PATCH 47/66] Don't check window type in tui_set_win_focus_to Tom Tromey
2019-06-23 23:26 ` [PATCH 32/66] Derive tui_win_info from tui_gen_win_info Tom Tromey
2019-06-23 23:26 ` [PATCH 36/66] Change tui_alloc_source_buffer return type to void Tom Tromey
2019-06-23 23:26 ` [PATCH 49/66] Separate out execution-info window Tom Tromey
2019-06-24 14:13   ` Pedro Alves
2019-06-24 20:52     ` Tom Tromey
2019-06-23 23:26 ` [PATCH 44/66] Introduce enum tui_box Tom Tromey
2019-06-24 14:23 ` [PATCH 00/66] Clean up the TUI Pedro Alves
2019-06-24 16:47   ` Tom Tromey
2019-06-24 17:46     ` Pedro Alves
2019-06-24 18:54     ` Tom Tromey
2019-06-24 22:23       ` Pedro Alves
2019-06-25 13:51         ` Tom Tromey
2019-06-24 18:49 ` [PATCH 50/66] Separate out locator window Tom Tromey
2019-06-24 18:48   ` [PATCH 53/66] Remove two unused enum constants from tui_win_type Tom Tromey
2019-06-24 18:48   ` [PATCH 55/66] Remove tui_init_generic_part Tom Tromey
2019-06-24 18:49   ` [PATCH 54/66] Separate out data item window Tom Tromey
2019-06-24 18:49   ` [PATCH 64/66] Fix latent bug in set_is_exec_point_at Tom Tromey
2019-06-24 18:49   ` [PATCH 60/66] Use bool for is_exec_point Tom Tromey
2019-06-24 18:49   ` [PATCH 57/66] Remove "data_content" and "data_content_count" from TUI data window Tom Tromey
2019-06-24 18:49   ` [PATCH 66/66] Tidy tui_delete_win Tom Tromey
2019-06-24 18:49   ` [PATCH 59/66] Fix "auxiliary" typo Tom Tromey
2019-06-24 18:49   ` [PATCH 61/66] More type safety for TUI source window functions Tom Tromey
2019-06-24 18:49   ` [PATCH 65/66] Make tui_gen_win_info constructor protected Tom Tromey
2019-06-24 18:49   ` [PATCH 62/66] Remove union tui_which_element Tom Tromey
2019-06-24 18:49   ` [PATCH 58/66] Separate out data window Tom Tromey
2019-06-24 18:49   ` [PATCH 56/66] Turn tui_first_data_item_displayed into a method Tom Tromey
2019-06-24 18:49   ` [PATCH 51/66] Remove layout_def::split Tom Tromey
2019-06-24 18:49   ` [PATCH 52/66] Remove command from tui_which_element Tom Tromey
2019-06-24 18:49   ` [PATCH 63/66] Remove NULL checks before xfree Tom Tromey

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).