This is an automated email from Gerrit.

"Antonio Borneo <borneo.anto...@gmail.com>" just uploaded a new patch set to 
Gerrit, which you can find at https://review.openocd.org/c/openocd/+/8802

-- gerrit

commit 0e0c80d20b0617fdd5d38bb44a164f7bcf42bfb8
Author: Antonio Borneo <borneo.anto...@gmail.com>
Date:   Sat Nov 23 18:15:40 2024 +0100

    helper: command: return correct error on command 'echo'
    
    In case of incorrect syntax, return ERROR_COMMAND_SYNTAX_ERROR so
    the command framework will print the usage string.
    
    Change-Id: I348debc77f470551d54fa77b4da780a48ff539c0
    Signed-off-by: Antonio Borneo <borneo.anto...@gmail.com>

diff --git a/src/helper/command.c b/src/helper/command.c
index d90d34141b..38fb4f83b7 100644
--- a/src/helper/command.c
+++ b/src/helper/command.c
@@ -673,7 +673,7 @@ COMMAND_HANDLER(handle_echo)
        }
 
        if (CMD_ARGC != 1)
-               return ERROR_FAIL;
+               return ERROR_COMMAND_SYNTAX_ERROR;
 
        LOG_USER("%s", CMD_ARGV[0]);
        return ERROR_OK;

-- 

Reply via email to