This is an automated email from Gerrit. "zapb <d...@zapb.de>" just uploaded a new patch set to Gerrit, which you can find at https://review.openocd.org/c/openocd/+/7992
-- gerrit commit cf19d580f47ec74d1b54ec1651ddbf122aa830c0 Author: Marc Schink <d...@zapb.de> 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 <d...@zapb.de> diff --git a/src/jtag/drivers/rshim.c b/src/jtag/drivers/rshim.c index 6170e86bdb..21fc7fd378 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]); --