diff options
author | Simon Marchi <simon.marchi@ericsson.com> | 2017-09-04 19:13:08 +0200 |
---|---|---|
committer | Simon Marchi <simon.marchi@ericsson.com> | 2017-09-04 19:13:48 +0200 |
commit | 80a65e9b8fbc93d4a7548ac17b8094ced23f66a7 (patch) | |
tree | 9fee8bb1c13b929f564e73068c2ef4800e56989a /gdb/target-debug.h | |
parent | 6b66338c70422d670637623cea8dc3b62e05e7de (diff) | |
download | binutils-80a65e9b8fbc93d4a7548ac17b8094ced23f66a7.zip binutils-80a65e9b8fbc93d4a7548ac17b8094ced23f66a7.tar.gz binutils-80a65e9b8fbc93d4a7548ac17b8094ced23f66a7.tar.bz2 |
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.
Diffstat (limited to 'gdb/target-debug.h')
0 files changed, 0 insertions, 0 deletions