Re: [PATCH] sideband: diagnose more incoming packet anomalies

From: Junio C Hamano <gitster@pobox.com>
Date: Oct 27, 2020, 19:04 UTC
Message-ID: <xmqq7drbcwx8.fsf@gitster.c.googlers.com>
In-Reply-To: <20201027071309.GD3005508@coredump.intra.peff.net>

Jeff King <peff@peff.net> writes:

Show 11 quoted lines

> And one other small cleanup, worth doing with or without my patch.
>
> -- >8 --
> Subject: [PATCH] test-pkt-line: drop colon from sideband identity
>
> We pass "sideband: " as our identity for errors to recv_sideband(). But
> it already adds the trailing colon and space. This doesn't invalidate
> any tests, but it looks funny when you examine the test output.
>
> Signed-off-by: Jeff King <peff@peff.net>
> ---

Makes sense and it is a no-brainer to understand why it is the right thing to do. I'll queue it directly on top of two patches from Dscho.

Thanks.

Show 16 quoted lines

>  t/helper/test-pkt-line.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/t/helper/test-pkt-line.c b/t/helper/test-pkt-line.c
> index 0bf20642be..5e638f0b97 100644
> --- a/t/helper/test-pkt-line.c
> +++ b/t/helper/test-pkt-line.c
> @@ -100,7 +100,7 @@ static int send_split_sideband(void)
>  
>  static int receive_sideband(void)
>  {
> -	return recv_sideband("sideband: ", 0, 1);
> +	return recv_sideband("sideband", 0, 1);
>  }
>  
>  int cmd__pkt_line(int argc, const char **argv)