Jonathan Nieder <jrnieder@xxxxxxxxx> writes: > FWIW if I were writing it, I would make changes in something like this > order: > > - Change fmt_merge_msg API to > > int fmt_merge_msg(struct strbuf *in, struct strbuf *out, > int shortlog_len); > > where shortlog_len is 0 for no shortlog, 20 for a shortlog > with 20 items. Update all callers (the changed function > signature makes it easy to find them). > > - Update merge --log and fmt-merge-msg --log options to take > an optional "size of shortlog" argument. > > - update [merge] log configuration to accept an integer > "size of shortlog" value. > > The first step might look like this. Thanks; it is very pleasing to see a more experienced contributor mentoring another with a good example to show _how_ the design of a new feature should be done. -- 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