diff options
author | Tomas Vanek <vanekt@fbl.cz> | 2019-01-08 00:26:39 +0100 |
---|---|---|
committer | Tomas Vanek <vanekt@fbl.cz> | 2019-01-27 20:58:10 +0000 |
commit | d0eb66f729ff50ffa21a3ea22bcff46167fe627a (patch) | |
tree | 3a1bec999ebc554ec61564e359ec280eaae05add /src/helper/command.c | |
parent | d9cb5593cd4855fba7ab76a7db2f2db6e7978f67 (diff) |
command: Log the failed command by full name
Commit 44009186cfabe77fb260af221ebd6272d1e78f44 added logging
of failed cmd name but it used c->name only. It might be confusing:
Debug: 244 105 command.c:644 run_command(): Command 'init' failed with error
code -4
User : 245 106 command.c:711 command_run_line():
Debug: 246 107 command.c:644 run_command(): Command 'init' failed with error
code -4
The command on line 244 is 'dap init'
Use full name of cmd including parents.
Change-Id: Iff131ce6454ef70b353ce1bc6d0a480b92820545
Signed-off-by: Tomas Vanek <vanekt@fbl.cz>
Reviewed-on: http://openocd.zylin.com/4837
Reviewed-by: Antonio Borneo <borneo.antonio@gmail.com>
Tested-by: jenkins
Reviewed-by: Jean-Christian de Rivaz <jcamdr70@gmail.com>
Diffstat (limited to 'src/helper/command.c')
-rw-r--r-- | src/helper/command.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/src/helper/command.c b/src/helper/command.c index 48d998f6..1ff4e01a 100644 --- a/src/helper/command.c +++ b/src/helper/command.c @@ -642,7 +642,10 @@ static int run_command(struct command_context *context, /* we do not print out an error message because the command *should* * have printed out an error */ - LOG_DEBUG("Command '%s' failed with error code %d", c->name, retval); + char *full_name = command_name(c, ' '); + LOG_DEBUG("Command '%s' failed with error code %d", + full_name ? full_name : c->name, retval); + free(full_name); } return retval; |