Jeff King <peff@xxxxxxxx> writes: > diff --git a/pretty.c b/pretty.c > index 0674027..95fe39a 100644 > --- a/pretty.c > +++ b/pretty.c > @@ -1094,7 +1094,8 @@ void pretty_print_commit(enum cmit_fmt fmt, const struct commit *commit, > if (fmt == CMIT_FMT_EMAIL && sb->len <= beginning_of_body) > strbuf_addch(sb, '\n'); > > - if (fmt != CMIT_FMT_ONELINE && fmt != CMIT_FMT_RAW) > + if (context->show_notes || > + (fmt != CMIT_FMT_ONELINE && fmt != CMIT_FMT_RAW)) > get_commit_notes(commit, sb, encoding, > NOTES_SHOW_HEADER | NOTES_INDENT); Heh, without this hunk I would have thought Peff and Gitster were the same person ;-). Once you introduce --no-notes, the above condition would not work well. -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html