Zsh Mailing List Archive Messages sorted by: Reverse Date, Date, Thread, Author

Re: [PATCH] completion: add _uids, _gids

X-seq: zsh-workers 54937
From: "Daniel Shahaf" <d.s@xxxxxxxxxxxxxxxxxx>
To: zsh-workers@xxxxxxx
Subject: Re: [PATCH] completion: add _uids, _gids
Date: Fri, 10 Jul 2026 05:13:46 +0000
Archived-at: <https://zsh.org/workers/54937>
Feedback-id: i425e4195:Fastmail
In-reply-to: <639370f0-4166-420e-ae3b-0f58fb434c25@app.fastmail.com>
List-id: <zsh-workers.zsh.org>
References: <639370f0-4166-420e-ae3b-0f58fb434c25@app.fastmail.com>

dana wrote on Fri, 10 Jul 2026 02:40 +00:00:
> +++ b/Completion/Unix/Type/_gids
> @@ -0,0 +1,21 @@
> +#autoload
> +
> +# see _uids
> +
> +local -a match mbegin mend gids disp inst cmd=( getent group )
> +
> +(( $+commands[getent] )) || cmd=( cat /etc/group )
> +
> +[[ $OSTYPE == darwin* ]] && gids=( ${${(f)"$(
> +  _call_program dscl-list dscl . -list /Groups PrimaryGroupID
> +)"}/#(#b)([^ ]##)[ ]##(-#<->)*/$match[2]:$match[1]} )
> +
> +(( $#gids )) || {

Consider changing this to «(( $+gids[1] ))», which is equivalent logically but takes O(1) to compute?

Ditto in _uids.

Cheers,

Daniel

References:
- [PATCH] completion: add _uids, _gids
  - From: dana

Messages sorted by: Reverse Date, Date, Thread, Author