From 0d0abc5609f22fd0923a2fe8acdbdb585354aa18 Mon Sep 17 00:00:00 2001 From: midipix Date: May 28 2024 15:07:41 +0000 Subject: argv.h: usage screen: avoid extra when all long options are hybrid-only. --- diff --git a/src/internal/argv/argv.h b/src/internal/argv/argv.h index e7661e7..0c3ec5c 100644 --- a/src/internal/argv/argv.h +++ b/src/internal/argv/argv.h @@ -833,6 +833,7 @@ static void argv_usage_impl( { const struct argv_option ** optv; const struct argv_option * option; + int nlong; bool fshort,flong,fboth; size_t len,optlen,desclen; char cache; @@ -865,7 +866,7 @@ static void argv_usage_impl( if (header) argv_dprintf(fd,"%s",header); - for (optlen=0,optv=options; *optv; optv++) { + for (optlen=0,nlong=0,optv=options; *optv; optv++) { option = *optv; /* indent + comma */ @@ -884,6 +885,11 @@ static void argv_usage_impl( /* optlen */ if (len > optlen) optlen = len; + + /* long (vs. hybrid-only) option? */ + if (option->long_name) + if (!(option->flags & ARGV_OPTION_HYBRID_ONLY)) + nlong++; } if (optlen >= optcap) { @@ -918,6 +924,10 @@ static void argv_usage_impl( prefix = option->flags & ARGV_OPTION_HYBRID_ONLY ? " -" : " --"; + /* avoid extra when all long opts are hybrid-only */ + if (nlong == 0) + prefix++; + /* option string */ if (fboth && option->short_name && option->long_name) sprintf(optstr,"%s-%c,%s%s",