Re: [PATCH 13/29] merge-recursive: fix memory leak when finalizing merge
- From
Karthik Nayak <karthik.188@gmail.com>
- Date
- Jun 12, 2024, 09:33 UTC
- Message-ID
- <CAOLa=ZQDCwE2KZVBV16vtA9-=iqLW7c1TAFv4Cj0SQPwEC=8FQ@mail.gmail.com>
- In-Reply-To
- <4ba1343d-1467-46a0-85b0-b249e1ddedf7@gmail.com>
Phillip Wood <phillip.wood123@gmail.com> writes:
Show 13 quoted lines
> On 06/06/2024 11:50, Karthik Nayak wrote: >> Patrick Steinhardt <ps@pks.im> writes: >> >>> We do not free some members of `struct merge_options`' private data. >> >> Nit: s/`struct merge_options`'/`struct merge_options`'s/ > > Either is correct, it is a matter of style whether one adds the trailing > "s". > > Best Wishes > > Phillip
TIL, thanks Phillip.