The first one might be considered a bug, although triggering it is probably impossible. Not much of a price to pay for correctness though. The next two patches modernize and update the test a bit. The next 3 patches try and reduce the code size by libifying(?) fmt-merge-msg. The final patch is just a "while I'm here" thing and might be unnecessary if we decide to just remove the summary option. Stephen Boyd (7): fmt-merge-msg: be quiet if nothing to merge t6200: modernize with test_tick t6200: test fmt-merge-msg more fmt-merge-msg: use pretty.c routines string-list: add unsorted_string_list_lookup() fmt-merge-msg: remove custom string_list implementation fmt-merge-msg: hide summary option Documentation/technical/api-string-list.txt | 6 +- builtin/fmt-merge-msg.c | 159 ++++++++-------------- string-list.c | 13 ++- string-list.h | 3 +- t/t6200-fmt-merge-msg.sh | 196 ++++++++++++++++++++++----- 5 files changed, 235 insertions(+), 142 deletions(-) -- 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