Re: [PATCH v2 0/6] revisions API: fix more memory leaks
From: Jeff King <hidden>
Date: 2022-07-29 19:02:32
From: Jeff King <hidden>
Date: 2022-07-29 19:02:32
On Fri, Jul 29, 2022 at 10:31:02AM +0200, Ævar Arnfjörð Bjarmason wrote:
A late re-roll of this set of revisions API and API user memory leak fixes. I think the much simpler code here in 4/6 should address the points Jeff King brought up in the v1 review. Other than that I renamed the variable in 3/6 s/cp/pending/g, per Jeff's suggestion (FWIW "cp" = "copy").
Thanks (and for the explanation regarding t91xx; I did totally miss in the v1 review that those were removing "FAIL" lines, not "PASS" ones). I don't think there's anything incorrect here, but I did make a few cosmetic / explanation suggestions. -Peff