Thread (6 messages) flat view 6 messages, 4 authors, 2021-10-25

Re: [PATCH] completion: fix incorrect bash/zsh string equality check

From: Robert Estelle <hidden>
Date: 2021-10-08 22:05:45

On Fri, Oct 8, 2021 at 1:50 PM Junio C Hamano [off-list ref] wrote:
Would it be equally a valid fix to use "=" instead of "==", or would
that change the meaning? This is a bash-only piece of code, so use
of [[ ... ]] is not technically incorrect, but if the basic [] works
well enough with "=", we should prefer that.
Yes, `[` is preferable for portability and they'd behave the same in
this case. I consciously chose to use `[[` because that's what all the
other comparisons in that script use. (I think I noted that in the
commit message, maybe). I think there's value in consistency, and not
enough value of `[` over `[[` to justify changing all the other lines.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help