Thread (15 messages) flat view 15 messages, 4 authors, 2021-06-08

Re: [PATCH v2 try2 3/4] completion: bash: fix for multiple dash commands

From: Junio C Hamano <hidden>
Date: 2021-06-08 05:15:08

Felipe Contreras [off-list ref] writes:
SZEDER Gábor wrote:
quoted
On Sun, Jun 06, 2021 at 01:47:25PM -0500, Felipe Contreras wrote:
quoted
Otherwise commands like 'for-each-ref' are not completed.
I think you meant that options of commands like 'for-each-ref' are not
completed.
...
Yes, I meant commands like 'for-each-ref' are not completed correctly...
by __gitcomp_builtin. Which in practical tearms means what you just
described above.
I couldn't exactly tell if that's a roundabout way to say "yes,
yours is better and I'll use it in an update, thanks".  I'll assume
that is the case and will mark the topic as expecting the final
update (as it seemed that all the other patches in the series were
received favourably).

Or was it meant as "no, the version you commented on was already
correct (if you squint your eyes)"?

Thanks.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help