Jeff King <peff@xxxxxxxx> writes:
> 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@xxxxxxxx>
> ---
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.
> 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)
