Error out immediatly when using if command without args in command list
authorSimon Marchi <simon.marchi@ericsson.com>
Mon, 4 Sep 2017 17:13:08 +0000 (19:13 +0200)
committerSimon Marchi <simon.marchi@ericsson.com>
Mon, 4 Sep 2017 17:13:48 +0000 (19:13 +0200)
commit80a65e9b8fbc93d4a7548ac17b8094ced23f66a7
tree9fee8bb1c13b929f564e73068c2ef4800e56989a
parent6b66338c70422d670637623cea8dc3b62e05e7de
Error out immediatly when using if command without args in command list

When using "if" (or while) without args directly on gdb's command line,
you get this:

  (gdb) if
  if/while commands require arguments

When doing the same when entering a command list, you only get an error
when the command is executed, when parse_exp_in_context_1 fails to
evaluate the expression.

  (gdb) define foo
  Type commands for definition of "foo".
  End with a line saying just "end".
  >if
   >end
  >end
  (gdb) foo
  Argument required (expression to compute).

I think it would make more sense to error out when inputting the command
list directly:

  (gdb) define foo
  Type commands for definition of "foo".
  End with a line saying just "end".
  >if
  if/while commands require arguments.

The only required change is to check whether args is an empty string in
build_command_line.

gdb/ChangeLog:

* cli/cli-script.c (build_command_line): For if/while commands,
check whether args is empty.

gdb/testsuite/ChangeLog:

* gdb.base/commands.exp: Call new procedure.
(define_if_without_arg_test): New procedure.
gdb/ChangeLog
gdb/cli/cli-script.c
gdb/testsuite/ChangeLog
gdb/testsuite/gdb.base/commands.exp