helper/command: remove unused functions and make static local ones
[openocd.git] / src / helper / command.h
index 611db8733eb806df4e185a9457dc2b268b1e3aa1..886bde86b6289c6505ae4d44da6e59711243fdb7 100644 (file)
  *   GNU General Public License for more details.                          *
  *                                                                         *
  *   You should have received a copy of the GNU General Public License     *
- *   along with this program; if not, write to the                         *
- *   Free Software Foundation, Inc.,                                       *
- *   59 Temple Place - Suite 330, Boston, MA  02111-1307, USA.             *
+ *   along with this program.  If not, see <http://www.gnu.org/licenses/>. *
  ***************************************************************************/
-#ifndef COMMAND_H
-#define COMMAND_H
 
-#include "types.h"
+#ifndef OPENOCD_HELPER_COMMAND_H
+#define OPENOCD_HELPER_COMMAND_H
 
-/* Integrate the JIM TCL interpretor into the command processing. */
-#if BUILD_ECOSBOARD
-#include <stdio.h>
-#include <stdarg.h>
-/* Jim is provied by eCos */
-#include <cyg/jimtcl/jim.h>
-#else
-#include "jim.h"
-#endif
+#include <stdint.h>
+#include <stdbool.h>
+#include <jim-nvp.h>
+
+#include <helper/types.h>
 
 /* To achieve C99 printf compatibility in MinGW, gnu_printf should be
  * used for __attribute__((format( ... ))), with GCC v4.4 or later
@@ -44,8 +37,7 @@
 #define PRINTF_ATTRIBUTE_FORMAT printf
 #endif
 
-enum command_mode
-{
+enum command_mode {
        COMMAND_EXEC,
        COMMAND_CONFIG,
        COMMAND_ANY,
@@ -53,16 +45,23 @@ enum command_mode
 
 struct command_context;
 
-/// The type signature for command context's output handler.
+/** The type signature for command context's output handler. */
 typedef int (*command_output_handler_t)(struct command_context *context,
-                               const char* line);
+               const char *line);
 
-struct command_context
-{
+struct command_context {
        Jim_Interp *interp;
        enum command_mode mode;
        struct command *commands;
-       int current_target;
+       struct target *current_target;
+               /* The target set by 'targets xx' command or the latest created */
+       struct target *current_target_override;
+               /* If set overrides current_target
+                * It happens during processing of
+                *      1) a target prefixed command
+                *      2) an event handler
+                * Pay attention to reentrancy when setting override.
+                */
        command_output_handler_t output_handler;
        void *output_handler_priv;
 };
@@ -80,6 +79,7 @@ struct command_invocation {
        const char *name;
        unsigned argc;
        const char **argv;
+       Jim_Obj *output;
 };
 
 /**
@@ -87,8 +87,8 @@ struct command_invocation {
  * set provided by command.c.  This macro uses C99 magic to allow
  * defining all such derivative types using this macro.
  */
-#define __COMMAND_HANDLER(name, extra...) \
-               int name(struct command_invocation *cmd, ##extra)
+#define __COMMAND_HANDLER(name, extra ...) \
+               int name(struct command_invocation *cmd, ## extra)
 
 /**
  * Use this to macro to call a command helper (or a nested handler).
@@ -103,8 +103,8 @@ struct command_invocation {
  * helper function, or care must be taken to avoid redefining the same
  * variables in intervening scope(s) by accident.
  */
-#define CALL_COMMAND_HANDLER(name, extra...) \
-               name(cmd, ##extra)
+#define CALL_COMMAND_HANDLER(name, extra ...) \
+               name(cmd, ## extra)
 
 /**
  * Always use this macro to define new command handler functions.
@@ -112,46 +112,51 @@ struct command_invocation {
  * they be can be used by other macros (e.g. COMMAND_PARSE_NUMBER).
  * All command handler functions must be defined as static in scope.
  */
-#define COMMAND_HANDLER(name) static __COMMAND_HANDLER(name)
+#define COMMAND_HANDLER(name) \
+               static __COMMAND_HANDLER(name)
 
 /**
  * Similar to COMMAND_HANDLER, except some parameters are expected.
  * A helper is globally-scoped because it may be shared between several
  * source files (e.g. the s3c24xx device command helper).
  */
-#define COMMAND_HELPER(name, extra...) __COMMAND_HANDLER(name, extra)
+#define COMMAND_HELPER(name, extra ...) __COMMAND_HANDLER(name, extra)
 
+/**
+ * Use this macro to access the command being handled,
+ * rather than accessing the variable directly.  It may be moved.
+ */
+#define CMD (cmd)
 /**
  * Use this macro to access the context of the command being handled,
  * rather than accessing the variable directly.  It may be moved.
  */
-#define CMD_CTX cmd->ctx
+#define CMD_CTX (cmd->ctx)
 /**
  * Use this macro to access the number of arguments for the command being
  * handled, rather than accessing the variable directly.  It may be moved.
  */
-#define CMD_ARGC cmd->argc
+#define CMD_ARGC (cmd->argc)
 /**
  * Use this macro to access the arguments for the command being handled,
  * rather than accessing the variable directly.  It may be moved.
  */
-#define CMD_ARGV cmd->argv
+#define CMD_ARGV (cmd->argv)
 /**
  * Use this macro to access the name of the command being handled,
  * rather than accessing the variable directly.  It may be moved.
  */
-#define CMD_NAME cmd->name
+#define CMD_NAME (cmd->name)
 /**
  * Use this macro to access the current command being handled,
  * rather than accessing the variable directly.  It may be moved.
  */
-#define CMD_CURRENT cmd->current
+#define CMD_CURRENT (cmd->current)
 /**
  * Use this macro to access the invoked command handler's data pointer,
  * rather than accessing the variable directly.  It may be moved.
  */
-#define CMD_DATA CMD_CURRENT->jim_handler_data
-
+#define CMD_DATA (CMD_CURRENT->jim_handler_data)
 
 /**
  * The type signature for command handling functions.  They are
@@ -172,33 +177,27 @@ struct command_invocation {
  */
 typedef __COMMAND_HANDLER((*command_handler_t));
 
-struct command
-{
+struct command {
        char *name;
-       const char *help;
-       const char *usage;
+       char *help;
+       char *usage;
        struct command *parent;
        struct command *children;
        command_handler_t handler;
-       Jim_CmdProc jim_handler;
+       Jim_CmdProc *jim_handler;
        void *jim_handler_data;
+               /* Currently used only for target of target-prefixed cmd.
+                * Native OpenOCD commands use jim_handler_data exclusively
+                * as a target override.
+                * Jim handlers outside of target cmd tree can use
+                * jim_handler_data for any handler specific data */
        enum command_mode mode;
        struct command *next;
 };
 
-/**
- * @param c The command to be named.
- * @param delim The character to place between command names.
- * @returns A malloc'd string containing the full command name,
- * which may include one or more ancestor components.  Multiple names
- * are separated by single spaces.  The caller must free() the string
- * when done with it.
- */
-char *command_name(struct command *c, char delim);
-
 /*
  * Commands should be registered by filling in one or more of these
- * structures and passing them to register_command().
+ * structures and passing them to [un]register_commands().
  *
  * A conventioal format should be used for help strings, to provide both
  * usage and basic information:
@@ -216,11 +215,10 @@ char *command_name(struct command *c, char delim);
 struct command_registration {
        const char *name;
        command_handler_t handler;
-       Jim_CmdProc jim_handler;
-       void *jim_handler_data;
+       Jim_CmdProc *jim_handler;
        enum command_mode mode;
        const char *help;
-       /// a string listing the options and arguments, required or optional
+       /** a string listing the options and arguments, required or optional */
        const char *usage;
 
        /**
@@ -232,38 +230,9 @@ struct command_registration {
        const struct command_registration *chain;
 };
 
-/// Use this as the last entry in an array of command_registration records.
+/** Use this as the last entry in an array of command_registration records. */
 #define COMMAND_REGISTRATION_DONE { .name = NULL, .chain = NULL }
 
-/**
- * Register a command @c handler that can be called from scripts during
- * the execution @c mode specified.
- *
- * If @c parent is non-NULL, the new command will be registered as a
- * sub-command under it; otherwise, it will be available as a top-level
- * command.
- *
- * @param cmd_ctx The command_context in which to register the command.
- * @param parent Register this command as a child of this, or NULL to
- * register a top-level command.
- * @param rec A command_registration record that contains the desired
- * command parameters.
- * @returns The new command, if successful; otherwise, NULL.
- */
-struct command* register_command(struct command_context *cmd_ctx,
-               struct command *parent, const struct command_registration *rec);
-
-#define COMMAND_REGISTER(_cmd_ctx, _parent, _name, _handler, _mode, _help) \
-       ({ \
-               struct command_registration cr = { \
-                               .name = _name, \
-                               .handler = _handler, \
-                               .mode = _mode, \
-                               .help = _help, \
-                       }; \
-               register_command(_cmd_ctx, _parent, &cr); \
-       })
-
 /**
  * Register one or more commands in the specified context, as children
  * of @c parent (or top-level commends, if NULL).  In a registration's
@@ -282,16 +251,6 @@ struct command* register_command(struct command_context *cmd_ctx,
 int register_commands(struct command_context *cmd_ctx, struct command *parent,
                const struct command_registration *cmds);
 
-
-/**
- * Unregisters command @c name from the given context, @c cmd_ctx.
- * @param cmd_ctx The context of the registered command.
- * @param parent The parent of the given command, or NULL.
- * @param name The name of the command to unregister.
- * @returns ERROR_OK on success, or an error code.
- */
-int unregister_command(struct command_context *cmd_ctx,
-               struct command *parent, const char *name);
 /**
  * Unregisters all commands from the specfied context.
  * @param cmd_ctx The context that will be cleared of registered commands.
@@ -303,8 +262,6 @@ int unregister_all_commands(struct command_context *cmd_ctx,
 
 struct command *command_find_in_context(struct command_context *cmd_ctx,
                const char *name);
-struct command *command_find_in_parent(struct command *parent,
-               const char *name);
 
 /**
  * Update the private command data field for a command and all descendents.
@@ -316,16 +273,30 @@ struct command *command_find_in_parent(struct command *parent,
  */
 void command_set_handler_data(struct command *c, void *p);
 
-void command_set_output_handler(struct command_contextcontext,
+void command_set_output_handler(struct command_context *context,
                command_output_handler_t output_handler, void *priv);
 
 
 int command_context_mode(struct command_context *context, enum command_mode mode);
 
+/* Return the current command context associated with the Jim interpreter or
+ * alternatively the global default command interpreter
+ */
+struct command_context *current_command_context(Jim_Interp *interp);
+/**
+ * Creates a new command context using the startup TCL provided and
+ * the existing Jim interpreter, if any. If interp == NULL, then command_init
+ * creates a command interpreter.
+ */
+struct command_context *command_init(const char *startup_tcl, Jim_Interp *interp);
 /**
- * Creates a new command context using the startup TCL provided.
+ * Shutdown a command context.
+ *
+ * Free the command context and the associated Jim interpreter.
+ *
+ * @param context The command_context that will be destroyed.
  */
-struct command_context* command_init(const char *startup_tcl);
+void command_exit(struct command_context *context);
 /**
  * Creates a copy of an existing command context.  This does not create
  * a deep copy of the command list, so modifications in one context will
@@ -334,31 +305,31 @@ struct command_context* command_init(const char *startup_tcl);
  * @param cmd_ctx The command_context that will be copied.
  * @returns A new command_context with the same state as the original.
  */
-struct command_context* copy_command_context(struct command_context* cmd_ctx);
+struct command_context *copy_command_context(struct command_context *cmd_ctx);
 /**
  * Frees the resources associated with a command context.  The commands
  * are not removed, so unregister_all_commands() must be called first.
- * @param cmd_ctx The command_context that will be destroyed.
+ * @param context The command_context that will be destroyed.
  */
 void command_done(struct command_context *context);
 
-void command_print(struct command_context *context, const char *format, ...)
-               __attribute__ ((format (PRINTF_ATTRIBUTE_FORMAT, 2, 3)));
-void command_print_sameline(struct command_context *context, const char *format, ...)
-               __attribute__ ((format (PRINTF_ATTRIBUTE_FORMAT, 2, 3)));
+void command_print(struct command_invocation *cmd, const char *format, ...)
+__attribute__ ((format (PRINTF_ATTRIBUTE_FORMAT, 2, 3)));
+void command_print_sameline(struct command_invocation *cmd, const char *format, ...)
+__attribute__ ((format (PRINTF_ATTRIBUTE_FORMAT, 2, 3)));
 int command_run_line(struct command_context *context, char *line);
 int command_run_linef(struct command_context *context, const char *format, ...)
-               __attribute__ ((format (PRINTF_ATTRIBUTE_FORMAT, 2, 3)));
+__attribute__ ((format (PRINTF_ATTRIBUTE_FORMAT, 2, 3)));
 void command_output_text(struct command_context *context, const char *data);
 
 void process_jim_events(struct command_context *cmd_ctx);
 
-#define                ERROR_COMMAND_CLOSE_CONNECTION          (-600)
-#define                ERROR_COMMAND_SYNTAX_ERROR                      (-601)
-#define                ERROR_COMMAND_NOTFOUND                          (-602)
-#define                ERROR_COMMAND_ARGUMENT_INVALID          (-603)
-#define                ERROR_COMMAND_ARGUMENT_OVERFLOW         (-604)
-#define                ERROR_COMMAND_ARGUMENT_UNDERFLOW        (-605)
+#define ERROR_COMMAND_CLOSE_CONNECTION         (-600)
+#define ERROR_COMMAND_SYNTAX_ERROR                     (-601)
+#define ERROR_COMMAND_NOTFOUND                         (-602)
+#define ERROR_COMMAND_ARGUMENT_INVALID         (-603)
+#define ERROR_COMMAND_ARGUMENT_OVERFLOW                (-604)
+#define ERROR_COMMAND_ARGUMENT_UNDERFLOW       (-605)
 
 int parse_ulong(const char *str, unsigned long *ul);
 int parse_ullong(const char *str, unsigned long long *ul);
@@ -367,18 +338,22 @@ int parse_long(const char *str, long *ul);
 int parse_llong(const char *str, long long *ul);
 
 #define DECLARE_PARSE_WRAPPER(name, type) \
-       int parse##name(const char *str, type *ul)
+               int parse ## name(const char *str, type * ul)
 
 DECLARE_PARSE_WRAPPER(_uint, unsigned);
+DECLARE_PARSE_WRAPPER(_u64, uint64_t);
 DECLARE_PARSE_WRAPPER(_u32, uint32_t);
 DECLARE_PARSE_WRAPPER(_u16, uint16_t);
 DECLARE_PARSE_WRAPPER(_u8, uint8_t);
 
 DECLARE_PARSE_WRAPPER(_int, int);
+DECLARE_PARSE_WRAPPER(_s64, int64_t);
 DECLARE_PARSE_WRAPPER(_s32, int32_t);
 DECLARE_PARSE_WRAPPER(_s16, int16_t);
 DECLARE_PARSE_WRAPPER(_s8, int8_t);
 
+DECLARE_PARSE_WRAPPER(_target_addr, target_addr_t);
+
 /**
  * @brief parses the string @a in into @a out as a @a type, or prints
  * a command error and passes the error code to the caller.  If an error
@@ -392,14 +367,17 @@ DECLARE_PARSE_WRAPPER(_s8, int8_t);
  */
 #define COMMAND_PARSE_NUMBER(type, in, out) \
        do { \
-               int retval = parse_##type(in, &(out)); \
-               if (ERROR_OK != retval) { \
-                       command_print(CMD_CTX, stringify(out) \
+               int retval_macro_tmp = parse_ ## type(in, &(out)); \
+               if (ERROR_OK != retval_macro_tmp) { \
+                       command_print(CMD, stringify(out) \
                                " option value ('%s') is not valid", in); \
-                       return retval; \
+                       return retval_macro_tmp; \
                } \
        } while (0)
 
+#define COMMAND_PARSE_ADDRESS(in, out) \
+       COMMAND_PARSE_NUMBER(target_addr, in, out)
+
 /**
  * Parse the string @c as a binary parameter, storing the boolean value
  * in @c out.  The strings @c on and @c off are used to match different
@@ -409,13 +387,13 @@ DECLARE_PARSE_WRAPPER(_s8, int8_t);
 #define COMMAND_PARSE_BOOL(in, out, on, off) \
        do { \
                bool value; \
-               int retval = command_parse_bool_arg(in, &value); \
-               if (ERROR_OK != retval) { \
-                       command_print(CMD_CTX, stringify(out) \
+               int retval_macro_tmp = command_parse_bool_arg(in, &value); \
+               if (ERROR_OK != retval_macro_tmp) { \
+                       command_print(CMD, stringify(out) \
                                " option value ('%s') is not valid", in); \
-                       command_print(CMD_CTX, "  choices are '%s' or '%s'", \
+                       command_print(CMD, "  choices are '%s' or '%s'", \
                                on, off); \
-                       return retval; \
+                       return retval_macro_tmp; \
                } \
                out = value; \
        } while (0)
@@ -423,14 +401,13 @@ DECLARE_PARSE_WRAPPER(_s8, int8_t);
 int command_parse_bool_arg(const char *in, bool *out);
 COMMAND_HELPER(handle_command_parse_bool, bool *out, const char *label);
 
-/// parses an on/off command argument
+/** parses an on/off command argument */
 #define COMMAND_PARSE_ON_OFF(in, out) \
-               COMMAND_PARSE_BOOL(in, out, "on", "off")
-/// parses an enable/disable command argument
+       COMMAND_PARSE_BOOL(in, out, "on", "off")
+/** parses an enable/disable command argument */
 #define COMMAND_PARSE_ENABLE(in, out) \
-               COMMAND_PARSE_BOOL(in, out, "enable", "disable")
+       COMMAND_PARSE_BOOL(in, out, "enable", "disable")
 
-void script_debug(Jim_Interp *interp, const char *cmd,
-               unsigned argc, Jim_Obj *const *argv);
+void script_debug(Jim_Interp *interp, unsigned int argc, Jim_Obj * const *argv);
 
-#endif /* COMMAND_H */
+#endif /* OPENOCD_HELPER_COMMAND_H */

Linking to existing account procedure

If you already have an account and want to add another login method you MUST first sign in with your existing account and then change URL to read https://review.openocd.org/login/?link to get to this page again but this time it'll work for linking. Thank you.

SSH host keys fingerprints

1024 SHA256:YKx8b7u5ZWdcbp7/4AeXNaqElP49m6QrwfXaqQGJAOk gerrit-code-review@openocd.zylin.com (DSA)
384 SHA256:jHIbSQa4REvwCFG4cq5LBlBLxmxSqelQPem/EXIrxjk gerrit-code-review@openocd.org (ECDSA)
521 SHA256:UAOPYkU9Fjtcao0Ul/Rrlnj/OsQvt+pgdYSZ4jOYdgs gerrit-code-review@openocd.org (ECDSA)
256 SHA256:A13M5QlnozFOvTllybRZH6vm7iSt0XLxbA48yfc2yfY gerrit-code-review@openocd.org (ECDSA)
256 SHA256:spYMBqEYoAOtK7yZBrcwE8ZpYt6b68Cfh9yEVetvbXg gerrit-code-review@openocd.org (ED25519)
+--[ED25519 256]--+
|=..              |
|+o..   .         |
|*.o   . .        |
|+B . . .         |
|Bo. = o S        |
|Oo.+ + =         |
|oB=.* = . o      |
| =+=.+   + E     |
|. .=o   . o      |
+----[SHA256]-----+
2048 SHA256:0Onrb7/PHjpo6iVZ7xQX2riKN83FJ3KGU0TvI0TaFG4 gerrit-code-review@openocd.zylin.com (RSA)