Subject: Re: [PATCH v2 2/3] docs: clarify cmd_psuh signature and explain UNUSED macro

From: Junio C Hamano <[email-masked]>
Date: Sat, 17 May 2025 11:39:42 -0700

Junio C Hamano <[email-masked]> writes:

> K Jayatheerth <[email-masked]> writes:
>
>> The documentation previously omitted the UNUSED macro,
>> which often led to confusion for new contributors
>> when they encountered compiler warnings related to unused parameters.
>
> The above is not quite easy to reason about.  It is more like we
> wrote this document, and then later tightened the default compiler
> warnings for developer builds.  So "omitted" may technically be
> correct, but it was more like "did not use it, because there was no
> need".
>
>     The sample program, as written, would not build for at least two
>     reasons:

"not" -> "no longer" to hint that the change in the environment,
not a bug in the document, is the reason of breakage.

>     - Since this document was first written, the calling convention
>       to subcommand implementation has changed, and now cmd_psuh()
>       needs to accept the third parameter, repository.

"third" -> "fourth"; I cannot count X-<.