Subject: Re: [PATCH 1/1] imap-send: include strbuf.h

From: Junio C Hamano <[email-masked]>
Date: Wed, 17 May 2023 08:49:37 -0700

Christian Hesse <[email-masked]> writes:

> From: Christian Hesse <[email-masked]>
>
> We use xstrfmt() here, so let's include the header file.
>
> Signed-off-by: Christian Hesse <[email-masked]>
> ---
>  imap-send.c | 1 +
>  1 file changed, 1 insertion(+)

Puzzled.  For me Git 2.41-rc0 builds as-is without this change just
fine, it seems.  I know there are many header file shuffling patches
flying around, and I have seen some of them, but is this a fix for
one of these patches?

Thanks.

>
> diff --git a/imap-send.c b/imap-send.c
> index a62424e90a..7f5426177a 100644
> --- a/imap-send.c
> +++ b/imap-send.c
> @@ -29,6 +29,7 @@
>  #include "run-command.h"
>  #include "parse-options.h"
>  #include "setup.h"
> +#include "strbuf.h"
>  #include "wrapper.h"
>  #if defined(NO_OPENSSL) && !defined(HAVE_OPENSSL_CSPRNG)
>  typedef void *SSL;