diff options
author | Jeff King <peff@peff.net> | 2014-07-29 21:53:40 +0400 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2014-07-30 23:30:02 +0400 |
commit | ae18165fbbf5e15ab2371d09f79cd12ee403c51e (patch) | |
tree | 817ac31ccf7e423059f7ee6072b68598c2a14f1e /commit.h | |
parent | 583b61c1af1bac12d6b48b2583ad508b5ab9964c (diff) |
revision: drop useless string offset when parsing "--pretty"
Once upon a time, we parsed pretty options by looking for
"--pretty" at the start of the string, and then feeding the
rest (including an "=") to get_commit_format. Later, commit
48ded91 (log --pretty: do not accept bogus "--prettyshort",
2008-05-25) split this into a separate check for "--pretty"
versus "--pretty=".
However, when parsing "--pretty", we still passed "arg+8" to
get_commit_format. This is useless, since it will always
point to the NUL terminator at the end of the string. We can
simply pass NULL instead; both parameters are treated the
same by get_commit_format.
Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'commit.h')
0 files changed, 0 insertions, 0 deletions