diff options
| author | Simon Marchi <simon.marchi@polymtl.ca> | 2021-11-05 15:29:20 -0400 |
|---|---|---|
| committer | Simon Marchi <simon.marchi@polymtl.ca> | 2021-11-08 16:39:14 -0500 |
| commit | 313f3b21cbcd3475c2fc6030abb26e6c8346d03b (patch) | |
| tree | be75de561691f60bc061a6711805a57db50ecbe0 /gdb/tui | |
| parent | 9be90c6894a208b32ed7237d4b31ecf9afb1ec8a (diff) | |
| download | fsf-binutils-gdb-313f3b21cbcd3475c2fc6030abb26e6c8346d03b.zip fsf-binutils-gdb-313f3b21cbcd3475c2fc6030abb26e6c8346d03b.tar.gz fsf-binutils-gdb-313f3b21cbcd3475c2fc6030abb26e6c8346d03b.tar.bz2 | |
gdb: remove bpstat typedef, rename bpstats to bpstat
I don't find that the bpstat typedef, which hides a pointer, is
particularly useful. In fact, it confused me many times, and I just see
it as something to remember that adds cognitive load. Also, with C++,
we might want to be able to pass bpstats objects by const-reference, not
necessarily by pointer.
So, remove the bpstat typedef and rename struct bpstats to bpstat (since
it represents one bpstat, it makes sense that it is singular).
Change-Id: I52e763b6e54ee666a9e045785f686d37b4f5f849
Diffstat (limited to 'gdb/tui')
| -rw-r--r-- | gdb/tui/tui-hooks.c | 2 | ||||
| -rw-r--r-- | gdb/tui/tui-interp.c | 2 |
2 files changed, 2 insertions, 2 deletions
diff --git a/gdb/tui/tui-hooks.c b/gdb/tui/tui-hooks.c index 52519ae..9eb10a6 100644 --- a/gdb/tui/tui-hooks.c +++ b/gdb/tui/tui-hooks.c @@ -190,7 +190,7 @@ tui_before_prompt (const char *current_gdb_prompt) /* Observer for the normal_stop notification. */ static void -tui_normal_stop (struct bpstats *bs, int print_frame) +tui_normal_stop (struct bpstat *bs, int print_frame) { from_stack = true; } diff --git a/gdb/tui/tui-interp.c b/gdb/tui/tui-interp.c index aaa353c..0cd5b12 100644 --- a/gdb/tui/tui-interp.c +++ b/gdb/tui/tui-interp.c @@ -79,7 +79,7 @@ tui_exit (void) /* Observer for the normal_stop notification. */ static void -tui_on_normal_stop (struct bpstats *bs, int print_frame) +tui_on_normal_stop (struct bpstat *bs, int print_frame) { if (!print_frame) return; |