On Tue, Jan 30, 2018 at 3:25 PM, Elijah Newren <new...@gmail.com> 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 <new...@gmail.com>
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 :/)