Siddharth Asthana <siddharthasthana31@xxxxxxxxx> writes: > @@ -3835,8 +3859,8 @@ static int commit_match(struct commit *commit, struct rev_info *opt) > if (!buf.len) > strbuf_addstr(&buf, message); > > - commit_rewrite_person(&buf, "\nauthor ", opt->mailmap); > - commit_rewrite_person(&buf, "\ncommitter ", opt->mailmap); > + const char *commit_headers[] = { "author ", "committer ", NULL }; This is decl-after-statement our codebase avoids. > + commit_rewrite_person(&buf, commit_headers, opt->mailmap); > } > > /* Append "fake" message parts as needed */