summaryrefslogtreecommitdiff
path: root/t/t1506-rev-parse-diagnosis.sh
diff options
context:
space:
mode:
authorLibravatar Jeff King <peff@peff.net>2020-11-10 16:38:03 -0500
committerLibravatar Junio C Hamano <gitster@pobox.com>2020-11-10 13:46:27 -0800
commit9033addfa6a910325fbe9619dc623c17e989261c (patch)
tree8ad3e9b515451c08b13aa4b41719cd6cf4d130fb /t/t1506-rev-parse-diagnosis.sh
parentrev-parse: don't accept options after dashdash (diff)
downloadtgif-9033addfa6a910325fbe9619dc623c17e989261c.tar.xz
rev-parse: put all options under the "-" check
The option-parsing loop of rev-parse checks whether the first character of an arg is "-". If so, then it enters a series of conditionals checking for individual options. But some options are inexplicably outside of that outer conditional. This doesn't produce the wrong behavior; the conditional is actually redundant with the individual option checks, and it's really only its fallback "continue" that we care about. But we should at least be consistent. One obvious alternative is that we could get rid of the conditional entirely. But we'll be using the extra block it provides in the next patch. Signed-off-by: Jeff King <peff@peff.net> Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 't/t1506-rev-parse-diagnosis.sh')
0 files changed, 0 insertions, 0 deletions