Re: [PATCH] squash! completion: fill COMPREPLY directly when completing refs
- From
Junio C Hamano <gitster@pobox.com>
- Date
- Feb 10, 2017, 21:44 UTC
- Message-ID
- <xmqq7f4xk9es.fsf@gitster.mtv.corp.google.com>
- In-Reply-To
- <20170206181545.12869-1-szeder.dev@gmail.com>
SZEDER Gábor <szeder.dev@gmail.com> writes:
Show 17 quoted lines
> Care should be taken, though, because that prefix might contain
> 'for-each-ref' format specifiers as part of the left hand side of a
> '..' range or '...' symmetric difference notation or fetch/push/etc.
> refspec, e.g. 'git log "evil-%(refname)..br<TAB>'. Doubling every '%'
> in the prefix will prevent 'git for-each-ref' from interpolating any
> of those contained format specifiers.
> ---
>
> This is really pathological, and I'm sure this has nothing to do
> with whatever breakage Jacob experienced. The shell
> metacharacters '(' and ')' still cause us trouble in various ways,
> but that's nothing new and has been the case for quite a while
> (always?).
>
> It's already incorporated into (the rewritten)
>
> https://github.com/szeder/git completion-refs-speedupShould I expect a reroll to come, or is this the only fix-up to the series that begins at <20170203025405.8242-1-szeder.dev@gmail.com>?
No hurries.