On Tue, Jan 30, 2018 at 3:25 PM, Elijah Newren <newren@xxxxxxxxx> wrote: > get_renames() has always zero'ed out diff_queued_diff.nr while only > manually free'ing diff_filepairs that did not correspond to renames. > Further, it allocated struct renames that were tucked away in the > return string_list. Make sure all of these are deallocated when we > are done with them. > > Signed-off-by: Elijah Newren <newren@xxxxxxxxx> Thanks for spotting the memleaks and fixing them in here. If this series takes longer to review/cook, this patch could go in separately? (with the last patch undone :/)
