[PATCH]: 631dc14cab jtag: tcl: make wrong # args error more meaningful for drscan, pathmove

[gerrit]

This is an automated email from Gerrit.

"Paul Fertser <fercer...@gmail.com>" just uploaded a new patch set to Gerrit, 
which you can find at https://review.openocd.org/c/openocd/+/7652

-- gerrit

commit 631dc14cab321898363237cc1bd8f4f24111a850
Author: Paul Fertser <fercer...@gmail.com>
Date:   Tue May 2 14:55:41 2023 +0300

    jtag: tcl: make wrong # args error more meaningful for drscan, pathmove
    
    Suggest using "usage" command instead of just repeating that the args are
    wrong.
    
    It would be better to automatically call "usage" to show it there but
    jim_handlers do not have command context so it's unclear how to properly do
    it.
    
    Signed-off-by: Paul Fertser <fercer...@gmail.com>
    Change-Id: I4d4c6be7bd7f356fb03ad43d5951c0759b9b20a6

diff --git a/src/jtag/tcl.c b/src/jtag/tcl.c
index b3cbc48b49..7d92dacd97 100644
--- a/src/jtag/tcl.c
+++ b/src/jtag/tcl.c
@@ -92,7 +92,7 @@ static int jim_command_drscan(Jim_Interp *interp, int argc, 
Jim_Obj * const *arg
         *     args[N-1] = statename
         */
        if ((argc < 4) || ((argc % 2) != 0)) {
-               Jim_WrongNumArgs(interp, 1, args, "wrong arguments");
+               Jim_WrongNumArgs(interp, 1, args, "(see usage)");
                return JIM_ERR;
        }
 
@@ -218,7 +218,7 @@ static int jim_command_pathmove(Jim_Interp *interp, int 
argc, Jim_Obj * const *a
        tap_state_t states[8];
 
        if ((argc < 2) || ((size_t)argc > (ARRAY_SIZE(states) + 1))) {
-               Jim_WrongNumArgs(interp, 1, args, "wrong arguments");
+               Jim_WrongNumArgs(interp, 1, args, "(see usage)");
                return JIM_ERR;
        }
 

--