]> cloudbase.mooo.com Git - z180-stamp.git/blobdiff - avr/command.c
Print prefix of subcommands
[z180-stamp.git] / avr / command.c
index b4a972c1b0a07f783c235aa3dad25099c7bad8df..bb179b800d4b8d88e24533bca0aa7e8ec9abea8d 100644 (file)
 jmp_buf cmd_jbuf;
 
 
-static void print_usage_line(const FLASH char *name, int width,
-                               const FLASH char *usage)
-{
-       width -= strlen_P(name);
-       if (width < 0)
-               width = 0;
-       my_puts_P(name);
-       print_blanks(width);
-       my_puts_P(PSTR(" - "));
-       my_puts_P(usage);
-       my_puts_P(PSTR("\n"));
-}
-
-int strcmp_PP(const FLASH char *s1, const FLASH char *s2)
-{
-       unsigned char c1, c2;
-
-       while ((c1 = *(const FLASH unsigned char *)s1++)
-                       == (c2 = *(const FLASH unsigned char *)s2++))
-               if (c1 == 0)
-                       return 0;
-
-       return c1 - c2;
-}
-
-int cmpstring_PP(const void *p1, const void *p2)
-{
-       return strcmp_PP((*(const FLASH cmd_tbl_t **) p1)->name,
-                        (*(const FLASH cmd_tbl_t **) p2)->name);
-}
-
-/****************************************************************************/
-
-int cmd_tbl_item_count(cmd_tbl_t *p)
+static int cmd_tbl_item_count(cmd_tbl_t *p)
 {
        int count = 0;
 
@@ -84,8 +51,7 @@ int cmd_tbl_item_count(cmd_tbl_t *p)
        return count;
 }
 
-
-cmd_tbl_t *get_cmd_tbl_base(cmd_tbl_t  *cmdtp)
+static cmd_tbl_t *get_cmd_tbl_base(cmd_tbl_t  *cmdtp)
 {
        cmd_tbl_t *p = cmdtp;
 
@@ -95,74 +61,99 @@ cmd_tbl_t *get_cmd_tbl_base(cmd_tbl_t  *cmdtp)
        return p->subcmd;
 }
 
-/*
- * find command table entry for a command
- */
-typedef struct {
-               size_t len;
-               uint_fast8_t level;
-               bool opt_debug;
-       } find_cmd_para_t;
-
-cmd_tbl_t *_find_cmd (const char *cmd, cmd_tbl_t *table, find_cmd_para_t *para)
+static void print_name_prefix(cmd_tbl_t *p)
 {
-       cmd_tbl_t *cmdtp_ret = NULL;
-       uint_fast8_t n_found = 0, sub_found = 0;
-
-       for (cmd_tbl_t *cmdtp = table; cmdtp->name != NULL; cmdtp++) {
-               if (strncmp_P(cmd, cmdtp->name, para->len) == 0 &&
-                                       (para->opt_debug || !(cmdtp->flags & CTBL_DBG))) {
-                       if (para->len == strlen_P(cmdtp->name))
-                               return cmdtp;   /* full match */
-
-                       cmdtp_ret = cmdtp;      /* abbreviated command ? */
-                       n_found++;
-               } else if (cmdtp->subcmd) {
-                       cmd_tbl_t *sub = _find_cmd(cmd, cmdtp->subcmd, para);
-                       if (sub && sub->flags & CTBL_SUBCMDAUTO) {
-                               cmdtp_ret = sub;
-                               ++n_found;
-                       }
-               }
+       cmd_tbl_t *tbl_start = get_cmd_tbl_base(p);
+       cmd_tbl_t *top;
+
+       if (tbl_start == cmd_tbl)
+               top = NULL;
+       else {
+               top = cmd_tbl;
+               while (top->subcmd != tbl_start)
+                       ++top;
        }
-       if (n_found == 1) {                     /* exactly one match */
-               if (sub_found)
-                       para->level++;
-               return cmdtp_ret;
+
+       if (top && (p->flags & CTBL_SUBCMDAUTO) == 0) {
+               printf_P(PSTR("%S "), top->name);
        }
+}
 
-       return NULL;    /* not found or ambiguous command */
+static void print_usage_line(cmd_tbl_t *p, int width)
+{
+       width -= strlen_P(p->name);
+       if (width < 0)
+               width = 0;
+       print_name_prefix(p);
+       my_puts_P(p->name);
+       print_blanks(width);
+       my_puts_P(PSTR(" - "));
+       puts_P(p->usage);
 }
 
+static int strcmp_PP(const FLASH char *s1, const FLASH char *s2)
+{
+       unsigned char c1, c2;
+
+       while ((c1 = *(const FLASH unsigned char *)s1++)
+                       == (c2 = *(const FLASH unsigned char *)s2++))
+               if (c1 == 0)
+                       return 0;
+
+       return c1 - c2;
+}
 
-cmd_tbl_t *find_cmd (const char *cmd, cmd_tbl_t *table, uint_fast8_t *cmdlevel)
+static int cmpstring_PP(const void *p1, const void *p2)
 {
-       find_cmd_para_t para;
+       return strcmp_PP((*(const FLASH cmd_tbl_t **) p1)->name,
+                        (*(const FLASH cmd_tbl_t **) p2)->name);
+}
 
+/****************************************************************************/
+
+/*
+ * find command table entry for a command
+ */
+
+static cmd_tbl_t *find_cmd (const char *cmd, cmd_tbl_t *table)
+{
        if (!cmd)
                return NULL;
 
        char *optenv = getenv_str(PSTR("cmd"));
-       para.level = 0;
-       para.opt_debug = optenv && strstr_P(optenv, PSTR("debug")) != NULL;
-       para.len = strlen(cmd);
+       uint8_t opt_debug = optenv && strstr_P(optenv, PSTR("debug")) != NULL;
 
-       cmd_tbl_t *cmdtp = _find_cmd(cmd, table, &para);
-
-       if (cmdlevel)
-               *cmdlevel = para.level;
-       return cmdtp;
-}
+       cmd_tbl_t *cmdtp_ret = NULL;
+       uint_fast8_t n_found = 0;
+       uint_fast8_t len = strlen(cmd);
 
-cmd_tbl_t *find_cmd_sub(const char *cmd, cmd_tbl_t *table)
-{
-       cmd_tbl_t *entry = NULL;
+       for (cmd_tbl_t *cmdtp = table; cmdtp->name != NULL; cmdtp++) {
+               if (cmdtp->subcmd) {
+                       for (cmd_tbl_t *sub = cmdtp->subcmd; sub->name != NULL; sub++) {
+                               if (sub->flags & CTBL_SUBCMDAUTO &&
+                                                       strncmp_P(cmd, sub->name, len) == 0 &&
+                                                       (opt_debug || !(sub->flags & CTBL_DBG))) {
+                                       if (len == strlen_P(sub->name))
+                                               return sub;                             /* full match */
+                                       cmdtp_ret = sub;                        /* abbreviated command ? */
+                                       ++n_found;
+                               }
+                       }
+               }
+               if ((cmdtp->flags & CTBL_SUBCMDAUTO) == 0 &&
+                                       strncmp_P(cmd, cmdtp->name, len) == 0 &&
+                                       (opt_debug || !(cmdtp->flags & CTBL_DBG))) {
+                       if (len == strlen_P(cmdtp->name))
+                               return cmdtp;                                   /* full match */
+                       cmdtp_ret = cmdtp;                                      /* abbreviated command ? */
+                       ++n_found;
+               }
+       }
 
-       for (cmd_tbl_t *tp = get_cmd_tbl_base(table); tp->name && entry == NULL; tp++)
-               if (tp->subcmd && tp->flags & CTBL_SUBCMDAUTO)
-                       entry = find_cmd(cmd, tp->subcmd, NULL);
+       if (n_found == 1)
+               return cmdtp_ret;                       /* exactly one match */
 
-       return entry;
+       return NULL;    /* not found or ambiguous command */
 }
 
 /*
@@ -173,6 +164,7 @@ cmd_tbl_t *find_cmd_sub(const char *cmd, cmd_tbl_t *table)
 command_ret_t do_help(cmd_tbl_t *cmdtp, uint_fast8_t flag UNUSED, int argc, char * const argv[])
 {
        cmd_tbl_t *tbl_start = get_cmd_tbl_base(cmdtp);
+       command_ret_t rc = CMD_RET_SUCCESS;
 
        char *optenv = getenv_str(PSTR("cmd"));
        bool opt_debug = optenv && strstr_P(optenv, PSTR("debug")) != NULL;
@@ -211,34 +203,28 @@ command_ret_t do_help(cmd_tbl_t *cmdtp, uint_fast8_t flag UNUSED, int argc, char
 
                /* print short help (usage) */
                for (int i = 0; i < cmd_items; i++) {
-                       if (opt_debug || !(cmd_list[i]->flags & CTBL_DBG)) {
-                               const FLASH char *usage = cmd_list[i]->usage;
+                       if ((cmd_list[i]->flags & CTBL_DBG) && !opt_debug)
+                               continue;
+                       if (cmd_list[i]->usage == NULL)
+                               continue;
 
-                               /* allow user abort */
-                               if (ctrlc ()) {
-                                       free(cmd_list);
-                                       return CMD_RET_FAILURE;
-                               }
-                               if (usage == NULL)
-                                       continue;
-#if defined(GCC_BUG_61443) || 1
-                               print_usage_line(cmd_list[i]->name, maxlen_cmd, usage);
-#else
-                               printf_P(PSTR("%-" stringify(8) /*FIXME*/ "S - %S\n"),
-                                               cmd_list[i]->name, usage);
-#endif
+                       print_usage_line(cmd_list[i], maxlen_cmd);
+
+                       /* allow user abort */
+                       if (ctrlc ()) {
+                               rc = CMD_RET_FAILURE;
+                               break;
                        }
                }
                free(cmd_list);
-               return CMD_RET_SUCCESS;
+               return rc;
        }
 
        /*
         * command help (long version)
         */
        for (uint8_t i = 1; i < argc; ++i) {
-               if ((cmdtp = find_cmd(argv[i], tbl_start, NULL)) != NULL &&
-                                       (opt_debug || !(cmdtp->flags & CTBL_DBG))) {
+               if ((cmdtp = find_cmd(argv[i], tbl_start)) != NULL) {
                        cmd_usage(cmdtp);
                } else {
                        printf_P(PSTR("Unknown command '%s' - try 'help'"
@@ -254,28 +240,17 @@ command_ret_t do_help(cmd_tbl_t *cmdtp, uint_fast8_t flag UNUSED, int argc, char
 
 command_ret_t cmd_usage(cmd_tbl_t *cmdtp)
 {
-//     printf("%s - %s\n\n", cmdtp->name, cmdtp->usage);
-       print_usage_line(cmdtp->name, 0, cmdtp->usage);
-#if 0
-       my_puts_P(cmdtp->name);
-       print_blanks(/*FIXME*/ 8 - strlen_P(cmdtp->name));
-       my_puts_P(PSTR(" - "));
-       my_puts_P(cmdtp->usage);
-       my_puts_P(PSTR("\n\n"));
-#endif
+       print_usage_line(cmdtp, 0);
 #ifdef CONFIG_SYS_LONGHELP
-//     printf("Usage:\n%s ", cmdtp->name);
        my_puts_P(PSTR("Usage:\n"));
+       print_name_prefix(cmdtp);
        my_puts_P(cmdtp->name);
        my_puts_P(PSTR(" "));
 
-       if (!cmdtp->help) {
+       if (cmdtp->help && *cmdtp->help != '\0')
+               puts_P(cmdtp->help);
+       else
                my_puts_P(PSTR(" - No additional help available.\n"));
-               return CMD_RET_FAILURE;
-       }
-
-       my_puts_P(cmdtp->help);
-       my_puts_P(PSTR("\n"));
 #endif /* CONFIG_SYS_LONGHELP */
        return CMD_RET_FAILURE;
 }
@@ -569,16 +544,15 @@ command_ret_t cmd_process(uint_fast8_t flag, int argc, char * const argv[],
 {
        command_ret_t rc = CMD_RET_SUCCESS;
        cmd_tbl_t *cmdtp;
-       uint_fast8_t cmdlevel;
 
        /* Look up command in command table */
-       cmdtp = find_cmd(argv[0], cmd_tbl, &cmdlevel);
+       cmdtp = find_cmd(argv[0], cmd_tbl);
        if (cmdtp != NULL) {
                /* Check if this command has subcommands */
                if (cmdtp->subcmd && argc > 1) {
 
                        /* Look up subcommand in subcommand table */
-                       cmd_tbl_t *cmdtpsub = find_cmd(argv[1], cmdtp->subcmd, &cmdlevel);
+                       cmd_tbl_t *cmdtpsub = find_cmd(argv[1], cmdtp->subcmd);
                        if (cmdtpsub == NULL) {
                                printf_P(PSTR("Unknown '%s' subcommand '%s' - try '%s help'\n"), argv[0], argv[1], argv[0]);
                                return CMD_RET_FAILURE;
@@ -588,12 +562,6 @@ command_ret_t cmd_process(uint_fast8_t flag, int argc, char * const argv[],
                        ++argv;
                }
        }
-#if 0
-        else {
-               /* Search subcommands */
-               cmdtp = find_cmd_sub(argv[0], cmd_tbl, &cmdlevel);
-       }
-#endif
 
        if (cmdtp == NULL) {
                printf_P(PSTR("Unknown command '%s' - try 'help'\n"), argv[0]);