cli/cli-script.c: Remove some dead NULL checks

gdb/ChangeLog:
2016-11-08  Pedro Alves  <palves@redhat.com>

	* cli/cli-script.c (execute_control_command): Assume insert_args
	never returns NULL.
	(insert_args): Assume xmalloc never returns NULL.

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index 1b647bb38ed42449e3c0f79b4b8e41323eaf0552..bc422e54290aa0607e24e8518815a179bd81c679 100644 (file)
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,9 @@
+2016-11-08  Pedro Alves  <palves@redhat.com>
+
+       * cli/cli-script.c (execute_control_command): Assume insert_args
+       never returns NULL.
+       (insert_args): Assume xmalloc never returns NULL.
+
 2016-11-08  Pedro Alves  <palves@redhat.com>
 
        * Makefile.in (COMMON_OBS): Add utils-selftests.o.

diff --git a/gdb/cli/cli-script.c b/gdb/cli/cli-script.c
index 12a976a4f9bf2a4bd71aa597e7ce00f4c6c17d46..e5cf20b83cb0f4b957dc4132984083d6ba05f79f 100644 (file)
--- a/gdb/cli/cli-script.c
+++ b/gdb/cli/cli-script.c
@@ -458,8 +458,6 @@ execute_control_command (struct command_line *cmd)
     case simple_control:
       /* A simple command, execute it and return.  */
       new_line = insert_args (cmd->line);
-      if (!new_line)
-       break;
       make_cleanup (free_current_contents, &new_line);
       execute_command (new_line, 0);
       ret = cmd->control_type;
@@ -491,8 +489,6 @@ execute_control_command (struct command_line *cmd)
 
        /* Parse the loop control expression for the while statement.  */
        new_line = insert_args (cmd->line);
-       if (!new_line)
-         break;
        make_cleanup (free_current_contents, &new_line);
        expr = parse_expression (new_line);
        make_cleanup (free_current_contents, &expr);
@@ -559,8 +555,6 @@ execute_control_command (struct command_line *cmd)
        print_command_trace (buffer);
 
        new_line = insert_args (cmd->line);
-       if (!new_line)
-         break;
        make_cleanup (free_current_contents, &new_line);
        /* Parse the conditional for the if statement.  */
        expr = parse_expression (new_line);
@@ -604,8 +598,6 @@ execute_control_command (struct command_line *cmd)
        /* Breakpoint commands list, record the commands in the
           breakpoint's command list and return.  */
        new_line = insert_args (cmd->line);
-       if (!new_line)
-         break;
        make_cleanup (free_current_contents, &new_line);
        ret = commands_from_control_command (new_line, cmd);
        break;
@@ -831,7 +823,6 @@ insert_args (char *line)
       else if (i >= user_args->count)
        {
          error (_("Missing argument %d in user function."), i);
-         return NULL;
        }
       else
        {
@@ -845,8 +836,6 @@ insert_args (char *line)
 
   /* Allocate space for the new line and fill it in.  */
   new_line = (char *) xmalloc (len + 1);
-  if (new_line == NULL)
-    return NULL;
 
   /* Restore pointer to beginning of old line.  */
   line = save_line;