From mboxrd@z Thu Jan 1 00:00:00 1970 X-Spam-Checker-Version: SpamAssassin 3.4.4 (2020-01-24) on inbox.vuxu.org X-Spam-Level: X-Spam-Status: No, score=-3.4 required=5.0 tests=DKIM_SIGNED,DKIM_VALID, DKIM_VALID_AU,MAILING_LIST_MULTI,RCVD_IN_DNSWL_MED,UNPARSEABLE_RELAY autolearn=ham autolearn_force=no version=3.4.4 Received: (qmail 8626 invoked from network); 27 Aug 2021 11:52:39 -0000 Received: from zero.zsh.org (2a02:898:31:0:48:4558:7a:7368) by inbox.vuxu.org with ESMTPUTF8; 27 Aug 2021 11:52:39 -0000 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=zsh.org; s=rsa-20210803; h=List-Archive:List-Owner:List-Post:List-Unsubscribe: List-Subscribe:List-Help:List-Id:Sender:Message-ID:Date:Content-ID: Content-Type:MIME-Version:Subject:To:From:Reply-To:Cc: Content-Transfer-Encoding:Content-Description:Resent-Date:Resent-From: Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:In-Reply-To:References; bh=4yk1bMWE7+SUDt35odlO9f/9kjnXkRTI53/MJ484mSQ=; b=ctnUF2VNgpgADJuwZyvDjdHJo2 noZp0g8xFB4nPElgYQIROeDEa1JXHLDM6bgny3GOx/3d4Zgn46B23CJv8pBzHlaagxv2Wl/bmnWSz R5KODZvAEEvWhls4PcvNGGBMRc/5sCvxd5+8RmJVxiI/P6ZIm8l1GTrHvd7cKPmK+3YvGzr8XdPo6 H9LEMlzF7f78jsIy4worFL4ZYf5AiUVr5byvApV++mupZrz5SbPc8n2aPx9p/ho0m3KH3FtBZWxbs aqsQTcRpsRc6DvfXcPd0I7bPb2m+TN4YTaCYVr7Y4yx1kpo6xuwIsiFp3hVM70gduSQr1nlVJ8ytp dlDdqG8w==; Received: from authenticated user by zero.zsh.org with local id 1mJaPV-0008FO-Tq; Fri, 27 Aug 2021 11:52:37 +0000 Received: from authenticated user by zero.zsh.org with esmtpsa (TLS1.3:TLS_AES_256_GCM_SHA384:256) id 1mJaPA-0007xX-K2; Fri, 27 Aug 2021 11:52:17 +0000 Received: from [192.168.178.21] (helo=hydra) by mail.kiddle.eu with esmtp(Exim 4.94.2) (envelope-from ) id 1mJaP9-000JCR-Ro for zsh-workers@zsh.org; Fri, 27 Aug 2021 13:52:15 +0200 From: Oliver Kiddle To: Zsh workers Subject: PATCH: list units in brackets at the end of completion group descriptions MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-ID: <73805.1630065135.1@hydra> Date: Fri, 27 Aug 2021 13:52:15 +0200 Message-ID: <73806-1630065135.858692@atah.qJX1.Jyyu> X-Seq: 49315 Archived-At: X-Loop: zsh-workers@zsh.org Errors-To: zsh-workers-owner@zsh.org Precedence: list Precedence: bulk Sender: zsh-workers-request@zsh.org X-no-archive: yes List-Id: List-Help: List-Subscribe: List-Unsubscribe: List-Post: List-Owner: List-Archive: Continuing with the bulk completion fixups for single issues, this applies the convention of units in round brackets at the end of the per-group descriptions just before the defaults I was adding in 49156 in square brackets. There aren't so many here because this is harder to grep for and we mostly already do this consistently. For anyone writing completions (or --help output), I'd recommend always considering units and defaults because it's not always obvious. There might be some more useful way we might indicate where suffixes can be appended to a value to denote units. Completing them after a number doesn't really make the options clear before a number has been entered. Perhaps an initial slash, e.g. 'timeout (seconds) [5] /ms /us /ns' As I've alluded to before, with consistent forms, _description could parse these and allow formatting for these components to be configured. But how often do we put random comments in brackets in descriptions or other things that would break this? Yet more backslashes will be ugly. Oliver diff --git Completion/Debian/Command/_dak Completion/Debian/Command/_dak index 8ecfd84b3..9976c7371 100644 --- Completion/Debian/Command/_dak +++ Completion/Debian/Command/_dak @@ -207,7 +207,7 @@ case $cmd in (poolize) args+=( - '(-l --limit)'{-l,--limit=}'[only migrate amount of packages]:amount in kb:' + '(-l --limit)'{-l,--limit=}'[only migrate amount of packages]:amount (kb)' '(-n --no-action)'{-n,--no-action}'[do not do anything]' '(-v --verbose)'{-v,--verbose}'[explain what is being done]' ) diff --git Completion/Linux/Command/_tpb Completion/Linux/Command/_tpb index c4c4c2bbd..2bdee0942 100644 --- Completion/Linux/Command/_tpb +++ Completion/Linux/Command/_tpb @@ -9,7 +9,7 @@ _arguments \ '(-x --xevents)'{-x,--xevents=}':state:(on off)' \ '(-m --mixer)'{-m,--mixer=}':state:(on off)' \ '(-o --osd)'{-o,--osd=}':state:(on off)' \ - '(-p --poll)'{-p,--poll=}':delay in microseconds:' \ + '(-p --poll)'{-p,--poll=}':delay (microseconds)' \ '(-t --thinkpad)'{-t,--thinkpad=}':command to execute:_command_names' \ '(-H --home)'{-H,--home=}':command to execute:_command_names' \ '(-S --search)'{-S,--search=}':command to execute:_command_names' \ diff --git Completion/Mandriva/Command/_urpmi Completion/Mandriva/Command/_urpmi index 9ab9e45b7..be6350129 100644 --- Completion/Mandriva/Command/_urpmi +++ Completion/Mandriva/Command/_urpmi @@ -73,7 +73,7 @@ _urpmi() { opts_net=( "($help --wget)--curl[use curl to retrieve distant files]" - "($help)--limit-rate[limit the download speed (b/s)]:download speed in b/s: " + "($help)--limit-rate[limit the download speed]:download speed (b/s)" "($help)--proxy:proxy host:_hosts" "($help)--proxy-user:proxy user:_users" "($help --curl)--wget[use wget to retrieve distant files]" diff --git Completion/Unix/Command/_cpio Completion/Unix/Command/_cpio index 699b1d8f2..20f8c712d 100644 --- Completion/Unix/Command/_cpio +++ Completion/Unix/Command/_cpio @@ -25,15 +25,15 @@ else '--message=:message at end of volume:' '--null' '--reset-access-time' '--verbose' '--dot' '--append' - '--block-size=:block size in 512 byte units:' + '--block-size=:block size (512 byte units)' '--dereference' - '--io-size=:block size in bytes' + '--io-size=:block size (bytes)' '--quiet' '--force-local' '--help' '--version') fi args+=( '-A[append files to archive]' '-B[block size 5120 bytes with special file]' - '-C[set block size per record]:block size in bytes:(8192)' + '-C[set block size per record]:block size (bytes)' '-F[set archive file to use]:archive file:->afile' '(-H)-c[read/write header in ASCII]' "(-c)-H[set format type for archive header]:$fmts" @@ -49,7 +49,7 @@ else '--preserve-modification-time' '--numeric' '--rename' '--list' '--swap-bytes' '--swap-halfwords' '--dot' '--unconditional' '--verbose' - '--block-size=:block size in 512 byte units:' + '--block-size=:block size (512 byte units)' '--swap-halfwords' '--io-size=:block size in bytes:' '--pattern-file=:file with list of patterns:_files' @@ -63,7 +63,7 @@ else '-b[reverse bytes in word]' '-B[block size 5120 bytes with special file]' '-d[create directories as needed]' - '-C[set block size per record]:block size in bytes:(8192)' + '-C[set block size per record]:block size (bytes)' '-E[read filenames from file]:file name for list of files:_files' '-f[only copy files not matching patterns]' '-F[set archive file to use]:archive file:->afile' diff --git Completion/Unix/Command/_dropbox Completion/Unix/Command/_dropbox index 115f9eb45..45c77ae09 100644 --- Completion/Unix/Command/_dropbox +++ Completion/Unix/Command/_dropbox @@ -55,8 +55,8 @@ else (throttle) _arguments \ - '2:download limit in KB/s:(unlimited)' \ - '3:upload limit in KB/s:(unlimited)' + '2:download limit (KB/s):(unlimited)' \ + '3:upload limit (KB/s):(unlimited)' ;; (*) diff --git Completion/Unix/Command/_growisofs Completion/Unix/Command/_growisofs index 8f4da4030..206d2c2c7 100644 --- Completion/Unix/Command/_growisofs +++ Completion/Unix/Command/_growisofs @@ -270,7 +270,7 @@ else '-sort[sort files on the image based on weights defined in the given file]:sort-weighting file:_files' \ '-sparc-label[set the sun disk-label-name used for the -sparc-boot option]:label' \ '-split-output[split the output image into several files ~1GiB in size]' \ - '-stream-media-size[enable streaming operation and set media size to given number of sectors]:media size in sectors' \ + '-stream-media-size[enable streaming operation and set media size to given number of sectors]:media size (sectors)' \ '-stream-file-name[reserved for future use]:name' \ '-sunx86-boot[specify a comma-separated list of file-system images needed to make a bootable Solaris-x86-CD]:list' \ '-sunx86-label[specify the SVr4 disk-label-name for the SVr4 disk-label created with the -sunx86-boot option]:label' \ diff --git Completion/Unix/Command/_mysql_utils Completion/Unix/Command/_mysql_utils index a7b285f1c..9f70687b0 100644 --- Completion/Unix/Command/_mysql_utils +++ Completion/Unix/Command/_mysql_utils @@ -220,8 +220,8 @@ _mysql_utils() { ) _mysql_variables_opts=( - '--max_allowed_packet[maximum allowed packet size]:packet size in bytes' - '--net_buffer_length[network buffer length]:buffer length in bytes' + '--max_allowed_packet[maximum allowed packet size]:packet size (bytes)' + '--net_buffer_length[network buffer length]:buffer length (bytes)' '--connect_timeout:timeout' '--select_limit:limit' '--max_join_size:size' diff --git Completion/Unix/Command/_perforce Completion/Unix/Command/_perforce index cb6cdb79f..a52ac1d06 100644 --- Completion/Unix/Command/_perforce +++ Completion/Unix/Command/_perforce @@ -2668,12 +2668,12 @@ _perforce_cmd_passwd() { _perforce_cmd_ping() { _arguments -s : \ '-c[specify count of messages]:count of messages: ' \ - '-t[specify total time of test]:time in seconds: ' \ + '-t[specify total time of test]:time (seconds)' \ '-i[specify iterations for test]:number of iterations: ' \ '-f[transmit continuously without waiting for responses]' \ - '-p[specify pause between tests]:pause in seconds: ' \ - '-s[specify send size]:send size in octets: ' \ - '-r[specify receive size]:receive size in octets: ' + '-p[specify pause between tests]:pause (seconds)' \ + '-s[specify send size]:send size (octets)' \ + '-r[specify receive size]:receive size (octets)' } @@ -2942,7 +2942,7 @@ _perforce_cmd_status() { _perforce_cmd_sizes() { _arguments -s : \ '-a[list all revisions in range]' \ - '-b[specify blocksize]:blocksize in bytes: ' \ + '-b[specify blocksize]:blocksize (bytes)' \ '(-H)-h[print sizes in human-readable form (GiB)]' \ '(-h)-H[print sizes in human-readable form (GB)]' \ '-m[limit max files]:max files: ' \ diff --git Completion/Unix/Command/_transmission Completion/Unix/Command/_transmission index 8951eacb0..c060cbae3 100644 --- Completion/Unix/Command/_transmission +++ Completion/Unix/Command/_transmission @@ -13,10 +13,10 @@ local general_options=( ) # `general_actions`: *actions* that can be specified when a specific set of torrents is either specified or either not local general_actions=( - '(- :)'{-gsr,--global-seedratio}'[all torrents, unless overridden by a per-torrent setting, should seed until a specific ratio]:ratio:{_message "ratio in kB/sec"}' + '(- :)'{-gsr,--global-seedratio}'[all torrents, unless overridden by a per-torrent setting, should seed until a specific ratio]:ratio (kB/s)' '(- :)'{-GSR,--no-global-seedratio}'[all torrents, unless overridden by a per-torrent setting, should seed regardless of ratio]' '(- :)'{-pr,--peers}'[set the maximum number of peers]:number: ' - '(- :)'{-d+,--downlimit=}'[limit the maximum download speed to limit kB/s]:limit:{_message "limit in kB/sec"}' + '(- :)'{-d+,--downlimit=}'[limit the maximum download speed to limit kB/s]:limit (kB/s)' '(- :)'{-D,--no-downlimit}'[disable download speed limits]' ) # `global_only_actions`: *actions* that can be specified only when no specific torrent is specified @@ -27,8 +27,8 @@ local global_only_actions=( '(- :)'{-asu,--alt-speed-uplimit}'[limit the alternate upload speed to limit kilobytes per second]' '(- :)'{-asc,--alt-speed-scheduler}'[use the scheduled on/off times]' '(- :)'{-asc,--not-alt-speed-scheduler}'[don'"'"'t use the scheduled on/off times]' - '(- :)--alt-speed-time-begin[time to start using the alt speed limits]:hhmm:_transmission-remote_time' - '(- :)--alt-speed-time-end[time to stop using the alt speed limits]:hhmm:_transmission-remote_time' + '(- :)--alt-speed-time-begin[time to start using the alt speed limits]:time (HHMM)' + '(- :)--alt-speed-time-end[time to stop using the alt speed limits]:time (HHMM)' '(- :)--alt-speed-days[set the number of days on which to enable the speed scheduler]:days:_transmission-remote_days' '(- :)--torrent-done-script[specify a file to run each time a torrent finishes]:script:_files' '(- :)--no-torrent-done-script[don'"'"'t run any script when a torrent finishes]' @@ -51,7 +51,7 @@ local global_only_actions=( '(- :)'{-M,--no-portmap}'[disable portmapping]' '(- :)'{-o,--dht}'[enable distributed hash table]' '(- :)'{-O,--no-dht}'[disable distributed hash table]' - '(- :)'{-u,--uplimit}'[limit the maximum upload speed to limit kB/s]:limit:{_message "limit in kB/sec"}' + '(- :)'{-u,--uplimit}'[limit the maximum upload speed to limit kB/s]:limit (kB/s)' '(- :)'{-U,--no-uplimit}'[disable upload speed limits]' '(- :)--utp[enable uTP for peer connections]' '(- :)--no-utp[disable uTP for peer connections]' @@ -89,7 +89,7 @@ local torrent_action_only_actions=( {-rad,--remove-and-delete}'[remove the current torrent(s) and delete the downloaded data]' '--reannounce[reannounce the current torrent(s)]' '--move[move the current torrents'"'"' data from their current locations to the specified directory]:{_files -/}' - {-sr+,--seedratio=}'[let the current torrent(s) seed until a specific ratio]:ratio:{_message "ratio in kB/sec"}' + {-sr+,--seedratio=}'[let the current torrent(s) seed until a specific ratio]:ratio' {-SR,--no-seedratio}'[let the current torrent(s) use the global seedratio settings]' {-hl,--honor-session}'[make the current torrent(s) honor the session limits]' {-HL,--no-honor-session}'[make the current torrent(s) not honor the session limits]' @@ -104,12 +104,6 @@ local torrent_action_only_actions=( # helper functions -# complete argument specified after global only action: `--alt-speed-time-begin` -# completes time in HHMM format -(( $+functions[_transmission-remote_time] )) || -_transmission-remote_time(){ - _message "time in format HHMS" -} # complete argument specified after global only action: `--alt-speed-days` # completes days in a list such as "2,4-6". (( $+functions[_transmission-remote_days] )) || diff --git Completion/X/Command/_urxvt Completion/X/Command/_urxvt index 661897c3a..2d7c09579 100644 --- Completion/X/Command/_urxvt +++ Completion/X/Command/_urxvt @@ -43,8 +43,8 @@ _x_arguments \ '-sl:save lines' \ '-embed:window id to embed terminal in:_x_window' \ '-pty-fd:file descriptor of pty to use' \ - '-w:external border in pixels' \ - '-b:internal border in pixels' \ + {-bw,-w}':external border (pixels)' \ + '-b:internal border (pixels)' \ '-lsp:number of extra pixels between rows' \ '-letsp:letter spacing adjustment' \ '-mod:meta modifier:_x_modifier' \