diff options
author | Zachary T Welch <zw@superlucidity.net> | 2009-11-28 16:29:41 -0800 |
---|---|---|
committer | Zachary T Welch <zw@superlucidity.net> | 2009-11-30 20:54:40 -0800 |
commit | 330733eadf76ea87ee8714ed7958f174b1be21db (patch) | |
tree | c40405503a582cb09d4035af8bbe3e0188c351d9 /src/helper/command.c | |
parent | 892604bc7e6b332cf3b0bf6c2586cbd0f54ec8ef (diff) | |
download | riscv-openocd-330733eadf76ea87ee8714ed7958f174b1be21db.zip riscv-openocd-330733eadf76ea87ee8714ed7958f174b1be21db.tar.gz riscv-openocd-330733eadf76ea87ee8714ed7958f174b1be21db.tar.bz2 |
improve command prohibition error report
Ensures that the correct information gets displayed, depending on the
mode of the command being denied. Fixes misreporting all commands as
needing to run "before 'init'".
Diffstat (limited to 'src/helper/command.c')
-rw-r--r-- | src/helper/command.c | 12 |
1 files changed, 10 insertions, 2 deletions
diff --git a/src/helper/command.c b/src/helper/command.c index 9b9c5ec..dcad6a1 100644 --- a/src/helper/command.c +++ b/src/helper/command.c @@ -565,8 +565,16 @@ static int run_command(struct command_context *context, { if (!command_can_run(context, c)) { - /* Config commands can not run after the config stage */ - LOG_ERROR("The '%s' command must be used before 'init'.", c->name); + /* Many commands may be run only before/after 'init' */ + const char *when; + switch (c->mode) { + case COMMAND_CONFIG: when = "before"; break; + case COMMAND_EXEC: when = "after"; break; + // handle the impossible with humor; it guarantees a bug report! + default: when = "if Cthulhu is summoned by"; break; + } + LOG_ERROR("The '%s' command must be used %s 'init'.", + c->name, when); return ERROR_FAIL; } |