From: Martin Ågren <martin.agren@gmail.com>
Date: Thu, 20 Mar 2025 16:11:41 GMT
Subject: Re: [PATCH 6/8] pretty: refactor parsing of line-wrapping "%w" placeholder
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