diff options
author | René Scharfe <l.s.r@web.de> | 2023-07-07 20:46:26 +0200 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2023-07-07 21:05:42 +0200 |
commit | 1dd14e8e934873eac6168649da4f200cda029b7b (patch) | |
tree | 48f217c3061cf16bc2e79fd30b83cedcb404ce1e /pretty.c | |
parent | strbuf: simplify strbuf_expand_literal_cb() (diff) | |
download | git-1dd14e8e934873eac6168649da4f200cda029b7b.tar.xz git-1dd14e8e934873eac6168649da4f200cda029b7b.zip |
pretty: avoid double negative in format_commit_item()
Test for equality with no_flush, which has enough negation already. Get
rid of the unnecessary parentheses while at it.
Signed-off-by: René Scharfe <l.s.r@web.de>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'pretty.c')
-rw-r--r-- | pretty.c | 6 |
1 files changed, 3 insertions, 3 deletions
@@ -1852,10 +1852,10 @@ static size_t format_commit_item(struct strbuf *sb, /* in UTF-8 */ } orig_len = sb->len; - if ((context)->flush_type != no_flush) - consumed = format_and_pad_commit(sb, placeholder, context); - else + if (context->flush_type == no_flush) consumed = format_commit_one(sb, placeholder, context); + else + consumed = format_and_pad_commit(sb, placeholder, context); if (magic == NO_MAGIC) return consumed; |