[PATCH 19/23] vshCmdGrpHelp: Refactor formatting of help for VSH_OT_ARGV

[Peter Krempa <pkrempa@xxxxxxxxxx>]

Use the new properties rather than infer the states.

Signed-off-by: Peter Krempa <pkrempa@xxxxxxxxxx>
---
 tools/vsh.c | 31 ++++++++++---------------------
 1 file changed, 10 insertions(+), 21 deletions(-)

diff --git a/tools/vsh.c b/tools/vsh.c
index 3245e64f5e..521f222910 100644
--- a/tools/vsh.c
+++ b/tools/vsh.c
@@ -634,8 +634,6 @@ vshCmdGrpHelp(vshControl *ctl, const vshCmdGrp *grp)
 static bool
 vshCmddefHelp(const vshCmdDef *def)
 {
-    bool shortopt = false; /* true if 'arg' works instead of '--opt arg' */
-
     fputs(_("  NAME\n"), stdout);
     fprintf(stdout, "    %s - %s\n", def->name, _(def->info->help));

@@ -657,16 +655,10 @@ vshCmddefHelp(const vshCmdDef *def)
                 } else {
                     fprintf(stdout, _(" [--%1$s <number>]"), opt->name);
                 }
-
-                if (!(opt->flags & VSH_OFLAG_REQ_OPT))
-                    shortopt = true;
                 break;

             case VSH_OT_STRING:
                 fprintf(stdout, _(" [--%1$s <string>]"), opt->name);
-
-                if (!(opt->flags & VSH_OFLAG_REQ_OPT))
-                    shortopt = true;
                 break;

             case VSH_OT_DATA:
@@ -675,23 +667,20 @@ vshCmddefHelp(const vshCmdDef *def)
                 } else {
                     fprintf(stdout, " [<%s>]", opt->name);
                 }
-
-                if (!(opt->flags & VSH_OFLAG_REQ_OPT))
-                    shortopt = true;
                 break;

             case VSH_OT_ARGV:
-                if (shortopt) {
-                    if (required_option) {
-                        fprintf(stdout, _(" {[--%1$s] <string>}..."), opt->name);
+                if (opt->positional) {
+                    if (opt->required) {
+                        fprintf(stdout, " <%s>...", opt->name);
                     } else {
-                        fprintf(stdout, _(" [[--%1$s] <string>]..."), opt->name);
+                        fprintf(stdout, " [<%s>]...", opt->name);
                     }
                 } else {
-                    if (required_option) {
-                        fprintf(stdout, " <%s>...", opt->name);
+                    if (opt->required) {
+                        fprintf(stdout, _(" {[--%1$s] <string>}..."), opt->name);
                     } else {
-                        fprintf(stdout, " [<%s>]...", opt->name);
+                        fprintf(stdout, _(" [[--%1$s] <string>]..."), opt->name);
                     }
                 }
                 break;
@@ -740,10 +729,10 @@ vshCmddefHelp(const vshCmdDef *def)
                 break;

             case VSH_OT_ARGV:
-                if (shortopt) {
-                    optstr = g_strdup_printf(_("[--%1$s] <string>"), opt->name);
-                } else {
+                if (opt->positional) {
                     optstr = g_strdup_printf("<%s>", opt->name);
+                } else {
+                    optstr = g_strdup_printf(_("[--%1$s] <string>"), opt->name);
                 }
                 break;

-- 
2.44.0
_______________________________________________
Devel mailing list -- devel@xxxxxxxxxxxxxxxxx
To unsubscribe send an email to devel-leave@xxxxxxxxxxxxxxxxx