Re: [PATCH 6/8] pretty: refactor parsing of line-wrapping "%w" placeholder

From

MÅMartin Ågren <martin.agren@gmail.com>

Date

Mar 20, 2025, 16:11 UTC

Message-ID

<CAN0heSpsGUJojoiFSrhTksBt0ZYoz17ycR=91Rs44S_SWbDjQw@mail.gmail.com>

In-Reply-To

<Z9vdUTQTnctm3965@pks.im>

On Thu, 20 Mar 2025 at 10:18, Patrick Steinhardt <ps@pks.im> wrote:

>
> On Wed, Mar 19, 2025 at 08:23:39AM +0100, Martin Ågren wrote:

> > +     memset(rewrap, 0, sizeof(*rewrap));
>
> The `memset()` feels rather unnecessary as we only use the result at our
> single caller in case we return successfully. And if we do, we know to
> initialize all struct fields.

Thanks, good point. I'll drop it.

Martin