diff options
author | Tom Tromey <tromey@redhat.com> | 2013-12-23 20:53:23 -0700 |
---|---|---|
committer | Tom Tromey <tromey@redhat.com> | 2014-02-28 08:51:15 -0700 |
commit | 89de4da46b3b3629c51e6e1a37177a6269dc8b05 (patch) | |
tree | 50603f2b5dfbdf72136500334974d7c511b3c291 | |
parent | 02fe846e43cb0190251a6fefbd322201b4bbe02d (diff) | |
download | gdb-89de4da46b3b3629c51e6e1a37177a6269dc8b05.zip gdb-89de4da46b3b3629c51e6e1a37177a6269dc8b05.tar.gz gdb-89de4da46b3b3629c51e6e1a37177a6269dc8b05.tar.bz2 |
constify ui_out_impl
This patch constifies ui_out_impl in struct ui_out, and various
instances of ui_out_impl.
This removes a couple of FIXME comments (near cli_ui_out_impl and
mi_ui_out_impl) that did not make sense to me.
Tested by rebuilding.
2014-02-28 Tom Tromey <tromey@redhat.com>
* cli-out.c (cli_ui_out_impl): Now const. Remove comment.
* cli-out.h (cli_ui_out_impl): Now const.
* mi/mi-out.c (mi_ui_out_impl): Now const. Remove comment.
* ui-out.c (struct ui_out) <impl>: Now const.
(default_ui_out_impl): Now const.
(ui_out_new): Make 'impl' parameter const.
* ui-out.h (ui_out_new): Update.
-rw-r--r-- | gdb/ChangeLog | 10 | ||||
-rw-r--r-- | gdb/cli-out.c | 5 | ||||
-rw-r--r-- | gdb/cli-out.h | 2 | ||||
-rw-r--r-- | gdb/mi/mi-out.c | 5 | ||||
-rw-r--r-- | gdb/ui-out.c | 6 | ||||
-rw-r--r-- | gdb/ui-out.h | 2 |
6 files changed, 17 insertions, 13 deletions
diff --git a/gdb/ChangeLog b/gdb/ChangeLog index c99510c..2c615be 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,13 @@ +2014-02-28 Tom Tromey <tromey@redhat.com> + + * cli-out.c (cli_ui_out_impl): Now const. Remove comment. + * cli-out.h (cli_ui_out_impl): Now const. + * mi/mi-out.c (mi_ui_out_impl): Now const. Remove comment. + * ui-out.c (struct ui_out) <impl>: Now const. + (default_ui_out_impl): Now const. + (ui_out_new): Make 'impl' parameter const. + * ui-out.h (ui_out_new): Update. + 2014-02-27 Mark Kettenis <kettenis@gnu.org> * solib-svr4.c (svr4_read_so_list): Initialize first_l_name to 0. diff --git a/gdb/cli-out.c b/gdb/cli-out.c index b936f30..5943fa7 100644 --- a/gdb/cli-out.c +++ b/gdb/cli-out.c @@ -349,10 +349,7 @@ field_separator (void) /* This is the CLI ui-out implementation functions vector */ -/* FIXME: This can be initialized dynamically after default is set to - handle initial output in main.c */ - -struct ui_out_impl cli_ui_out_impl = +const struct ui_out_impl cli_ui_out_impl = { cli_table_begin, cli_table_body, diff --git a/gdb/cli-out.h b/gdb/cli-out.h index db7824d..a592890 100644 --- a/gdb/cli-out.h +++ b/gdb/cli-out.h @@ -37,7 +37,7 @@ struct cli_ui_out_data int suppress_output; }; -extern struct ui_out_impl cli_ui_out_impl; +extern const struct ui_out_impl cli_ui_out_impl; extern struct ui_out *cli_out_new (struct ui_file *stream); diff --git a/gdb/mi/mi-out.c b/gdb/mi/mi-out.c index a8a3d8d..6ec41e6 100644 --- a/gdb/mi/mi-out.c +++ b/gdb/mi/mi-out.c @@ -67,10 +67,7 @@ static int mi_redirect (struct ui_out *uiout, struct ui_file *outstream); /* This is the MI ui-out implementation functions vector */ -/* FIXME: This can be initialized dynamically after default is set to - handle initial output in main.c */ - -struct ui_out_impl mi_ui_out_impl = +static const struct ui_out_impl mi_ui_out_impl = { mi_table_begin, mi_table_body, diff --git a/gdb/ui-out.c b/gdb/ui-out.c index a591352..2edb140 100644 --- a/gdb/ui-out.c +++ b/gdb/ui-out.c @@ -102,7 +102,7 @@ struct ui_out { int flags; /* Specific implementation of ui-out. */ - struct ui_out_impl *impl; + const struct ui_out_impl *impl; void *data; /* Current level. */ @@ -198,7 +198,7 @@ static void default_data_destroy (struct ui_out *uiout); /* This is the default ui-out implementation functions vector. */ -struct ui_out_impl default_ui_out_impl = +const struct ui_out_impl default_ui_out_impl = { default_table_begin, default_table_body, @@ -1095,7 +1095,7 @@ ui_out_query_field (struct ui_out *uiout, int colno, /* Initalize private members at startup. */ struct ui_out * -ui_out_new (struct ui_out_impl *impl, void *data, +ui_out_new (const struct ui_out_impl *impl, void *data, int flags) { struct ui_out *uiout = XNEW (struct ui_out); diff --git a/gdb/ui-out.h b/gdb/ui-out.h index 27e6801..cbd0b9b 100644 --- a/gdb/ui-out.h +++ b/gdb/ui-out.h @@ -234,7 +234,7 @@ extern void uo_field_string (struct ui_out *uiout, int fldno, int width, /* Create a ui_out object */ -extern struct ui_out *ui_out_new (struct ui_out_impl *impl, +extern struct ui_out *ui_out_new (const struct ui_out_impl *impl, void *data, int flags); |