This is an automated email from Gerrit.

Antonio Borneo ([email protected]) just uploaded a new patch set to 
Gerrit, which you can find at http://openocd.zylin.com/5644

-- gerrit

commit 9785376b513d97ac35821f4eb711ea7018a2977b
Author: Antonio Borneo <[email protected]>
Date:   Fri May 1 18:30:00 2020 +0200

    helper/command: register all commands through register_commands()
    
    The commands "ocd_find" and "capture" are registered directly
    through the jim API, instead of the common way to describe them
    in a struct command_registration that is then passed to the helper
    register_commands().
    This cause the two commands above to not have either "help" nor
    "usage" string nor a properly identified "mode".
    
    Since the following line registers the commands listed in struct
    command_builtin_handlers, simply add the two commands above in the
    same struct.
    
    Change-Id: Id6ee11dac3b18364deeed65ee8e18ad80152750a
    Signed-off-by: Antonio Borneo <[email protected]>

diff --git a/src/helper/command.c b/src/helper/command.c
index ec07a5f..0882ecd 100644
--- a/src/helper/command.c
+++ b/src/helper/command.c
@@ -1230,6 +1230,21 @@ static const struct command_registration 
command_subcommand_handlers[] = {
 
 static const struct command_registration command_builtin_handlers[] = {
        {
+               .name = "ocd_find",
+               .mode = COMMAND_ANY,
+               .jim_handler = jim_find,
+               .help = "find full path to file",
+               .usage = "file",
+       },
+       {
+               .name = "capture",
+               .mode = COMMAND_ANY,
+               .jim_handler = jim_capture,
+               .help = "Capture progress output and return as tcl return 
value. If the "
+                               "progress output was empty, return tcl return 
value.",
+               .usage = "command",
+       },
+       {
                .name = "echo",
                .handler = jim_echo,
                .mode = COMMAND_ANY,
@@ -1339,9 +1354,6 @@ struct command_context *command_init(const char 
*startup_tcl, Jim_Interp *interp
        Jim_SetGlobalVariableStr(interp, "ocd_HOSTOS",
                Jim_NewStringObj(interp, HostOs, strlen(HostOs)));
 
-       Jim_CreateCommand(interp, "ocd_find", jim_find, NULL, NULL);
-       Jim_CreateCommand(interp, "capture", jim_capture, NULL, NULL);
-
        register_commands(context, NULL, command_builtin_handlers);
 
        Jim_SetAssocData(interp, "context", NULL, context);

-- 


_______________________________________________
OpenOCD-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/openocd-devel

Reply via email to