Introduce enum_flag type for ui_out flags

This patch changes the ui_out flags to be an enum flag.

gdb/ChangeLog:

	* ui-out.h: Include "common/enum-flags.h".
	(enum ui_flags): Rename to ...
	(enum ui_out_flag): ... this.
	(ui_out_flags): Define enum flag type.
	(ui_out_test_flags): Change type of parameter to ui_out_flags.
	(ui_out_new): Likewise.
	* ui-out.c (ui_out_test_flags): Likewise.
	(ui_out_new): Likewise.
	* cli-out.c (cli_out_new): Update variable type.
	* mi/mi-out.c (mi_out_new): Likewise.
	* tui/tui-out.c (tui_out_new): Likewise.

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index 777abc7c30ad0d5934e51dabf64503711e222c4b..4a64fca8d2daa3dd3edaa9772d04d5aab41f0018 100644 (file)
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,17 @@
+2016-12-02  Simon Marchi  <simon.marchi@polymtl.ca>
+
+       * ui-out.h: Include "common/enum-flags.h".
+       (enum ui_flags): Rename to ...
+       (enum ui_out_flag): ... this.
+       (ui_out_flags): Define enum flag type.
+       (ui_out_test_flags): Change type of parameter to ui_out_flags.
+       (ui_out_new): Likewise.
+       * ui-out.c (ui_out_test_flags): Likewise.
+       (ui_out_new): Likewise.
+       * cli-out.c (cli_out_new): Update variable type.
+       * mi/mi-out.c (mi_out_new): Likewise.
+       * tui/tui-out.c (tui_out_new): Likewise.
+
 2016-12-02  Pedro Alves  <palves@redhat.com>
 
        * NEWS: Mention that user commands now accept an unlimited number

diff --git a/gdb/cli-out.c b/gdb/cli-out.c
index ac19e38c3d32002176adde4dc671dde8557201ee..f5b5072e557a27dc709a234f3d9100d3d993d9f0 100644 (file)
--- a/gdb/cli-out.c
+++ b/gdb/cli-out.c
@@ -388,7 +388,7 @@ cli_out_data_ctor (cli_out_data *self, struct ui_file *stream)
 struct ui_out *
 cli_out_new (struct ui_file *stream)
 {
-  int flags = ui_source_list;
+  ui_out_flags flags = ui_source_list;
   cli_out_data *data = new cli_out_data ();
 
   cli_out_data_ctor (data, stream);

diff --git a/gdb/mi/mi-out.c b/gdb/mi/mi-out.c
index 6e69d7cb348bf8758685f89b912e0af415e9dd47..19fcf87fbf34fd1a3445256679c8dfca4a893b6f 100644 (file)
--- a/gdb/mi/mi-out.c
+++ b/gdb/mi/mi-out.c
@@ -393,7 +393,7 @@ mi_out_data_dtor (struct ui_out *ui_out)
 struct ui_out *
 mi_out_new (int mi_version)
 {
-  int flags = 0;
+  ui_out_flags flags = 0;
   mi_out_data *data = new mi_out_data ();
   struct ui_file *stream = mem_fileopen ();
 

diff --git a/gdb/tui/tui-out.c b/gdb/tui/tui-out.c
index 48565623436c409026b36460b039df304053d50b..a5f0541037a109ce7c1a6214668e16b6a693885e 100644 (file)
--- a/gdb/tui/tui-out.c
+++ b/gdb/tui/tui-out.c
@@ -145,7 +145,7 @@ tui_text (struct ui_out *uiout, const char *string)
 struct ui_out *
 tui_out_new (struct ui_file *stream)
 {
-  int flags = 0;
+  ui_out_flags flags = 0;
 
   tui_out_data *data = new tui_out_data ();
 

diff --git a/gdb/ui-out.c b/gdb/ui-out.c
index 8f745dadac025deba7c9fc0b7b8bd1ef8fcef617..774be2cb715063884f0e048fee22018a0513eb2d 100644 (file)
--- a/gdb/ui-out.c
+++ b/gdb/ui-out.c
@@ -702,7 +702,7 @@ ui_out_redirect (struct ui_out *uiout, struct ui_file *outstream)
 
 /* Test the flags against the mask given.  */
 int
-ui_out_test_flags (struct ui_out *uiout, int mask)
+ui_out_test_flags (struct ui_out *uiout, ui_out_flags mask)
 {
   return (uiout->flags & mask);
 }
@@ -923,7 +923,7 @@ ui_out_query_field (struct ui_out *uiout, int colno,
 
 struct ui_out *
 ui_out_new (const struct ui_out_impl *impl, void *data,
-           int flags)
+           ui_out_flags flags)
 {
   struct ui_out *uiout = new ui_out ();
 

diff --git a/gdb/ui-out.h b/gdb/ui-out.h
index 06c05e2f3af8566697faacb16c471f1c7d239ec7..cdf567148fdfa2a9bf008b8df8ece3b28f9f292b 100644 (file)
--- a/gdb/ui-out.h
+++ b/gdb/ui-out.h
@@ -23,6 +23,8 @@
 #ifndef UI_OUT_H
 #define UI_OUT_H 1
 
+#include "common/enum-flags.h"
+
 /* The ui_out structure */
 
 struct ui_out;
@@ -45,12 +47,12 @@ enum ui_align
   };
 
 /* flags enum */
-enum ui_flags
+enum ui_out_flag
   {
-    ui_from_tty = 1,
-    ui_source_list = 2
+    ui_source_list = (1 << 0),
   };
 
+DEF_ENUM_FLAGS_TYPE (ui_out_flag, ui_out_flags);
 
 /* Prototypes for ui-out API.  */
 
@@ -127,7 +129,7 @@ extern void ui_out_wrap_hint (struct ui_out *uiout, const char *identstring);
 
 extern void ui_out_flush (struct ui_out *uiout);
 
-extern int ui_out_test_flags (struct ui_out *uiout, int mask);
+extern int ui_out_test_flags (struct ui_out *uiout, ui_out_flags mask);
 
 extern int ui_out_query_field (struct ui_out *uiout, int colno,
                               int *width, int *alignment,
@@ -222,9 +224,8 @@ 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 (const struct ui_out_impl *impl,
-                                 void *data,
-                                 int flags);
+extern struct ui_out *ui_out_new (const struct ui_out_impl *impl, void *data,
+                                 ui_out_flags flags);
 
 /* Redirect the ouptut of a ui_out object temporarily.  */