Re: [PATCH] refs: fix duplicated word in comment

From: Martin Ågren <martin.agren@gmail.com>
Date: Apr 5, 2025, 10:01 UTC
Message-ID: <CAN0heSp9RhH=UMDqLs=Kd57RoseySrFT4sFeXrVV3h7f=daSaw@mail.gmail.com>
In-Reply-To: <pull.1940.git.git.1743791986754.gitgitgadget@gmail.com>

Hi Christian Fredrik,

Thanks for posting a patch.

On Fri, 4 Apr 2025 at 20:39, Christian Fredrik Johnsen via GitGitGadget <gitgitgadget@gmail.com> wrote:

Show 5 quoted lines

>
> From: Christian Fredrik Johnsen <christian@johnsen.no>
>
> Fix a typo in a comment in refs.c: "checking checking" → "checking".
> Signed-off-by: Christian Fredrik Johnsen <christian@johnsen.no>

There should be an empty line between the main part of the commit message and the Signed-off-by trailer.

Show 6 quoted lines

>                         /*
>                          * If we've already seen the directory we don't need to
> -                        * process it again. Skip it to avoid checking checking
> +                        * process it again. Skip it to avoid checking
>                          * common prefixes like "refs/heads/" repeatedly.
>                          */

Well-spotted. No doubt that this is the correct fix.

Martin