From: openocd-gerrit <ope...@us...> - 2023-11-18 11:32:53
|
This is an automated email from the git hooks/post-receive script. It was generated because a ref change was pushed to the repository containing the project "Main OpenOCD repository". The branch, master has been updated via 1df35d92fe59eac4603f28a2e1c6e5ee1d56e7da (commit) from dc0f79d45d4ea1fec9f31557478ef0d0cdba1d01 (commit) Those revisions listed above that are new to this repository have not appeared on any other notification email; so we list those revisions in full, below. - Log ----------------------------------------------------------------- commit 1df35d92fe59eac4603f28a2e1c6e5ee1d56e7da Author: Marc Schink <de...@za...> Date: Thu Nov 9 10:13:10 2023 +0100 jtag/drivers/rshim: Remove redundant error message The correct syntax is already suggested due to the return value used. Change-Id: I0f4a7f93fdf056e7517c754d6d4ecd7928f1d226 Signed-off-by: Marc Schink <de...@za...> Reviewed-on: https://review.openocd.org/c/openocd/+/7992 Reviewed-by: Antonio Borneo <bor...@gm...> Reviewed-by: Tomas Vanek <va...@fb...> Tested-by: jenkins diff --git a/src/jtag/drivers/rshim.c b/src/jtag/drivers/rshim.c index 6170e86bd..21fc7fd37 100644 --- a/src/jtag/drivers/rshim.c +++ b/src/jtag/drivers/rshim.c @@ -434,10 +434,8 @@ static void rshim_disconnect(struct adiv5_dap *dap) COMMAND_HANDLER(rshim_dap_device_command) { - if (CMD_ARGC != 1) { - command_print(CMD, "Too many arguments"); + if (CMD_ARGC != 1) return ERROR_COMMAND_SYNTAX_ERROR; - } free(rshim_dev_path); rshim_dev_path = strdup(CMD_ARGV[0]); ----------------------------------------------------------------------- Summary of changes: src/jtag/drivers/rshim.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) hooks/post-receive -- Main OpenOCD repository |