Re: [PATCH 2/2] grep: prefer UNUSED to MAYBE_UNUSED for pcre allocators

From: Eric Sunshine <sunshine@sunshineco.com>
Date: Aug 29, 2024, 20:27 UTC
Message-ID: <CAPig+cS0fT6+j3nQx7rh7bTDoMCsEUUWK3_YnpuiJVo+bVZWBQ@mail.gmail.com>
In-Reply-To: <20240829200953.GB432235@coredump.intra.peff.net>

On Thu, Aug 29, 2024 at 4:10 PM Jeff King <peff@peff.net> wrote:

> We prove custom malloc/free callbacks for the pcre library to use. Those

s/prove/provide/

Show 10 quoted lines

> take an extra "data" parameter, but we don't use it. Back when these
> were added in 513f2b0bbd (grep: make PCRE2 aware of custom allocator,
> 2019-10-16), we only had MAYBE_UNUSED. But these days we have UNUSED,
> which we should prefer, as it will let the compiler inform us if the
> code changes to actually use the parameters.
>
> I also moved the annotations to come after the variable name, which is
> how we typically spell it.
>
> Signed-off-by: Jeff King <peff@peff.net>