Re: [PATCH 13/29] merge-recursive: fix memory leak when finalizing merge
- From
- Phillip Wood <phillip.wood123@gmail.com>
- Date
- Jun 6, 2024, 15:52 UTC
- Message-ID
- <4ba1343d-1467-46a0-85b0-b249e1ddedf7@gmail.com>
- In-Reply-To
- <CAOLa=ZQ5qN9+GGp2Cmks+J2dCbZpNvd0ZEGTWejedUCtzXF9Ew@mail.gmail.com>
On 06/06/2024 11:50, Karthik Nayak wrote:
Show 5 quoted lines
> 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