[binutils-gdb] Move command lines types/declarations to cli-script.h

[Simon Marchi <simark at sourceware dot org>]

https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=6b66338c70422d670637623cea8dc3b62e05e7de

commit 6b66338c70422d670637623cea8dc3b62e05e7de
Author: Simon Marchi <simon.marchi@ericsson.com>
Date:   Mon Sep 4 19:09:12 2017 +0200

    Move command lines types/declarations to cli-script.h
    
    I think it would make more sense if the types and function declarations
    related to command lines were in cli-script.h rather than defs.h, since
    the related function definitions are in cli-script.c.
    
    I had to add a few includes here and there.  I also had to rename the
    "lines" parameter of command_lines_deleter::operator(), because ncurses
    has a "#define lines ..." that was interfering when cli-script.h is
    included by some TUI source files that also include ncurses header files.
    
    gdb/ChangeLog:
    
    	* cli/cli-script.h (enum misc_command_type): Move from defs.h.
    	(enum command_control_type): Likewise.
    	(struct command_line): Likewise.
    	(free_command_lines): Likewise.
    	(struct command_lines_deleter): Likewise.
    	(command_line_up): Likewise.
    	(read_command_lines): Likewise.
    	(read_command_lines_1): Likewise.
    	* defs.h (enum misc_command_type): Move to cli/cli-script.h.
    	(enum command_control_type): Likewise.
    	(struct command_line): Likewise.
    	(free_command_lines): Likewise.
    	(struct command_lines_deleter): Likewise.
    	(command_line_up): Likewise.
    	(read_command_lines): Likewise.
    	(read_command_lines_1): Likewise.
    	* breakpoint.h: Include cli/cli-script.h.
    	* extension-priv.h: Likewise.
    	* gdbcmd.h: Likewise.

Diff:
---
 gdb/ChangeLog        | 22 +++++++++++++++
 gdb/breakpoint.h     |  1 +
 gdb/cli/cli-script.h | 76 +++++++++++++++++++++++++++++++++++++++++++++++++++-
 gdb/defs.h           | 74 --------------------------------------------------
 gdb/extension-priv.h |  1 +
 gdb/gdbcmd.h         |  1 +
 6 files changed, 100 insertions(+), 75 deletions(-)

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index 6b4000b..0a1171a 100644
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,25 @@
+2017-09-04  Simon Marchi  <simon.marchi@ericsson.com>
+
+	* cli/cli-script.h (enum misc_command_type): Move from defs.h.
+	(enum command_control_type): Likewise.
+	(struct command_line): Likewise.
+	(free_command_lines): Likewise.
+	(struct command_lines_deleter): Likewise.
+	(command_line_up): Likewise.
+	(read_command_lines): Likewise.
+	(read_command_lines_1): Likewise.
+	* defs.h (enum misc_command_type): Move to cli/cli-script.h.
+	(enum command_control_type): Likewise.
+	(struct command_line): Likewise.
+	(free_command_lines): Likewise.
+	(struct command_lines_deleter): Likewise.
+	(command_line_up): Likewise.
+	(read_command_lines): Likewise.
+	(read_command_lines_1): Likewise.
+	* breakpoint.h: Include cli/cli-script.h.
+	* extension-priv.h: Likewise.
+	* gdbcmd.h: Likewise.
+
 2017-09-04  Pedro Alves  <palves@redhat.com>
 
 	* ada-lang.c (is_known_support_routine): Move sal declaration to
diff --git a/gdb/breakpoint.h b/gdb/breakpoint.h
index d0fa324..8b35eef 100644
--- a/gdb/breakpoint.h
+++ b/gdb/breakpoint.h
@@ -29,6 +29,7 @@
 #include "location.h"
 #include <vector>
 #include "common/array-view.h"
+#include "cli/cli-script.h"
 
 struct value;
 struct block;
diff --git a/gdb/cli/cli-script.h b/gdb/cli/cli-script.h
index 9514938..234faf7 100644
--- a/gdb/cli/cli-script.h
+++ b/gdb/cli/cli-script.h
@@ -18,9 +18,83 @@
 #define CLI_SCRIPT_H 1
 
 struct ui_file;
-struct command_line;
 struct cmd_list_element;
 
+/* * Control types for commands.  */
+
+enum misc_command_type
+{
+  ok_command,
+  end_command,
+  else_command,
+  nop_command
+};
+
+enum command_control_type
+{
+  simple_control,
+  break_control,
+  continue_control,
+  while_control,
+  if_control,
+  commands_control,
+  python_control,
+  compile_control,
+  guile_control,
+  while_stepping_control,
+  invalid_control
+};
+
+/* * Structure for saved commands lines (for breakpoints, defined
+   commands, etc).  */
+
+struct command_line
+{
+  struct command_line *next;
+  char *line;
+  enum command_control_type control_type;
+  union
+    {
+      struct
+	{
+	  enum compile_i_scope_types scope;
+	  void *scope_data;
+	}
+      compile;
+    }
+  control_u;
+  /* * The number of elements in body_list.  */
+  int body_count;
+  /* * For composite commands, the nested lists of commands.  For
+     example, for "if" command this will contain the then branch and
+     the else branch, if that is available.  */
+  struct command_line **body_list;
+};
+
+extern void free_command_lines (struct command_line **);
+
+/* A deleter for command_line that calls free_command_lines.  */
+
+struct command_lines_deleter
+{
+  void operator() (command_line *cmd_lines) const
+  {
+    free_command_lines (&cmd_lines);
+  }
+};
+
+/* A unique pointer to a command_line.  */
+
+typedef std::unique_ptr<command_line, command_lines_deleter> command_line_up;
+
+extern command_line_up read_command_lines (char *, int, int,
+					   void (*)(char *, void *),
+					   void *);
+extern command_line_up read_command_lines_1 (char * (*) (void), int,
+					     void (*)(char *, void *),
+					     void *);
+
+
 /* Exported to cli/cli-cmds.c */
 
 extern void script_from_file (FILE *stream, const char *file);
diff --git a/gdb/defs.h b/gdb/defs.h
index e180523..af9e32e 100644
--- a/gdb/defs.h
+++ b/gdb/defs.h
@@ -400,80 +400,6 @@ enum lval_type
     lval_computed
   };
 
-/* * Control types for commands.  */
-
-enum misc_command_type
-  {
-    ok_command,
-    end_command,
-    else_command,
-    nop_command
-  };
-
-enum command_control_type
-  {
-    simple_control,
-    break_control,
-    continue_control,
-    while_control,
-    if_control,
-    commands_control,
-    python_control,
-    compile_control,
-    guile_control,
-    while_stepping_control,
-    invalid_control
-  };
-
-/* * Structure for saved commands lines (for breakpoints, defined
-   commands, etc).  */
-
-struct command_line
-  {
-    struct command_line *next;
-    char *line;
-    enum command_control_type control_type;
-    union
-      {
-	struct
-	  {
-	    enum compile_i_scope_types scope;
-	    void *scope_data;
-	  }
-	compile;
-      }
-    control_u;
-    /* * The number of elements in body_list.  */
-    int body_count;
-    /* * For composite commands, the nested lists of commands.  For
-       example, for "if" command this will contain the then branch and
-       the else branch, if that is available.  */
-    struct command_line **body_list;
-  };
-
-extern void free_command_lines (struct command_line **);
-
-/* A deleter for command_line that calls free_command_lines.  */
-
-struct command_lines_deleter
-{
-  void operator() (command_line *lines) const
-  {
-    free_command_lines (&lines);
-  }
-};
-
-/* A unique pointer to a command_line.  */
-
-typedef std::unique_ptr<command_line, command_lines_deleter> command_line_up;
-
-extern command_line_up read_command_lines (char *, int, int,
-					   void (*)(char *, void *),
-					   void *);
-extern command_line_up read_command_lines_1 (char * (*) (void), int,
-					     void (*)(char *, void *),
-					     void *);
-
 /* * Parameters of the "info proc" command.  */
 
 enum info_proc_what
diff --git a/gdb/extension-priv.h b/gdb/extension-priv.h
index f77f088..4d16ac5 100644
--- a/gdb/extension-priv.h
+++ b/gdb/extension-priv.h
@@ -23,6 +23,7 @@
 
 #include "extension.h"
 #include <signal.h>
+#include "cli/cli-script.h"
 
 /* The return code for some API calls.  */
 
diff --git a/gdb/gdbcmd.h b/gdb/gdbcmd.h
index 14c7f4e..3191ecd 100644
--- a/gdb/gdbcmd.h
+++ b/gdb/gdbcmd.h
@@ -26,6 +26,7 @@
 
 #include "command.h"
 #include "ui-out.h"
+#include "cli/cli-script.h"
 
 /* Chain containing all defined commands.  */