Re: [PATCH 0/2] commit-graph: more leak fixes

2018-10-03 Thread Martin Ågren
On Wed, 3 Oct 2018 at 18:19, Derrick Stolee wrote: > I'm fine with squashing it in with both our sign-offs. It is the same > idea, it just requires a different set of arguments to hit it. I'll > adjust the commit message as necessary. > I'll add your PATCH 2/2 to my v2. Thanks! Cool, thanks a lo

Re: [PATCH 0/2] commit-graph: more leak fixes

2018-10-03 Thread Derrick Stolee
On 10/3/2018 11:36 AM, Martin Ågren wrote: Hi Derrick, These two patches on top of yours make the test suite (i.e., the subset of it that I run) leak-free with respect to builtin/commit-graph.c and commit-graph.c. Thanks! The first could be squashed into your patch 1/2. It touches the same f

[PATCH 0/2] commit-graph: more leak fixes

2018-10-03 Thread Martin Ågren
Hi Derrick, These two patches on top of yours make the test suite (i.e., the subset of it that I run) leak-free with respect to builtin/commit-graph.c and commit-graph.c. The first could be squashed into your patch 1/2. It touches the same function, but it requires a different usage to trigger, s