Re: [PATCH 00/29] Memory leak fixes (pt.2)

From: Karthik Nayak <karthik.188@gmail.com>
Date: Jun 6, 2024, 14:33 UTC
Message-ID: <CAOLa=ZS39hobSHXzyq=vAL9KqBkr9PaSZyqyELvFxvAA=uSofw@mail.gmail.com>
In-Reply-To: <cover.1717402439.git.ps@pks.im>

Patrick Steinhardt <ps@pks.im> writes:

Show 8 quoted lines

> Hi,
>
> this is another random assortment of memory leak fixes for Git. With
> these changes, another 77 test suites start to pass with the leak
> sanitizer enabled. With that, we're down to ~270 test suites that do not
> yet pass with the sanitizer enabled. My goal is that we can reduce this
> number to 0 this year -- any help here would be appreciated.
>

I was thinking of picking up some here, how do I avoid not clashing in with the work that you're doing?

Show 11 quoted lines

> While most of the commits are trivial, I realize that the overall series
> is quite large. If you think that this is too big, please feel free to
> speak up up and I'll split this and future series into batches of at
> most 20 patches or less, if you think that's still too many.
>
> The series is built on top of 9eaef5822c (Sync with 'maint', 2024-05-31)
> with ps/leakfixes at 164937678c (Merge remote-tracking branch
> 'junio/ps/leakfixes' into HEAD, 2024-06-03) merged into it.
>
> Thanks!
>

I only have some small typo nits apart from that this was a nice read. Thanks!