Hello, this series aims to clean up the code for pickaxe (-S/-G). The first three patches plug minor to medium memory leaks. [PATCH 1/7] pickaxe: plug diff filespec leak with empty needle [PATCH 2/7] pickaxe: plug regex leak [PATCH 3/7] pickaxe: plug regex/kws leak The next one moves a duplicate if/else cascade into its own helper function: [PATCH 4/7] pickaxe: factor out has_changes The remainder unifies the code for pickaxe (-S) and log grep (-G). [PATCH 5/7] pickaxe: pass diff_options to contains and has_changes [PATCH 6/7] pickaxe: give diff_grep the same signature as has_changes [PATCH 7/7] pickaxe: factor out pickaxe As a result the code should be shorter and easier to maintain. diffcore-pickaxe.c | 178 ++++++++++++++++++++------------------------------- 1 files changed, 70 insertions(+), 108 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