From: Peter Krempa <pkrempa@redhat.com>
Date: Tue, 5 Mar 2024 15:32:40 +0000 (+0100)
Subject: vshCmdGrpHelp: Refactor formatting of help for VSH_OT_ARGV
X-Git-Url: http://xenbits.xensource.com/gitweb?a=commitdiff_plain;h=e177b0fca66f569cb05ce408b36a5d248f95b0e2;p=libvirt.git

vshCmdGrpHelp: Refactor formatting of help for VSH_OT_ARGV

Use the new properties rather than infer the states.

Signed-off-by: Peter Krempa <pkrempa@redhat.com>
Reviewed-by: Ján Tomko <jtomko@redhat.com>
---

diff --git a/tools/vsh.c b/tools/vsh.c
index e80c6e84f1..1907e778f2 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;