summaryrefslogtreecommitdiff
path: root/match-trees.c
diff options
context:
space:
mode:
authorJeff King <peff@peff.net>2017-07-13 14:56:10 (GMT)
committerJunio C Hamano <gitster@pobox.com>2017-07-13 19:42:50 (GMT)
commit2eda0102beb85b97bd43708176d81feabaf161e8 (patch)
treec87815f88fbc559d434b3a8dd2c129803416de10 /match-trees.c
parent80145b1e412719c960036c8c62a9e35dd23a5b2d (diff)
downloadgit-2eda0102beb85b97bd43708176d81feabaf161e8.zip
git-2eda0102beb85b97bd43708176d81feabaf161e8.tar.gz
git-2eda0102beb85b97bd43708176d81feabaf161e8.tar.bz2
check return value of verify_ref_format()
Users of the ref-filter code must call verify_ref_format() before formatting any refs, but most ignore its return value. This means we may print an error on a syntactically bogus pattern, but keep going anyway. In most cases this results in a fatal error when we actually try to format a ref. But if you have no refs to show at all, then the behavior is confusing: git prints the error from verify_ref_format(), then exits with code 0 without showing any output. Let's instead abort immediately if we know we have a bogus format. We'll output the usage information if we have it handy (just like the existing call in cmd_for_each_ref() does), and otherwise just die(). Signed-off-by: Jeff King <peff@peff.net> Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'match-trees.c')
0 files changed, 0 insertions, 0 deletions