[PATCH 5 of 5] chg: downgrade "failed to read channel" from abortmsg to debugmsg

Yuya Nishihara yuya at tcha.org
Thu Mar 17 10:26:07 EDT 2016


On Tue, 15 Mar 2016 09:58:36 +0000, Jun Wu wrote:
> # HG changeset patch
> # User Jun Wu <quark at fb.com>
> # Date 1457998094 0
> #      Mon Mar 14 23:28:14 2016 +0000
> # Node ID d087d514ee79bad4a269bba32fd6dec7103e290f
> # Parent  1dcd622a1f8ffdbfedad1a284dc09c5ea409712d
> chg: downgrade "failed to read channel" from abortmsg to debugmsg

Queued this, thanks.

> diff --git a/contrib/chg/hgclient.c b/contrib/chg/hgclient.c
> --- a/contrib/chg/hgclient.c
> +++ b/contrib/chg/hgclient.c
> @@ -106,8 +106,10 @@
>  	assert(hgc);
>  
>  	ssize_t rsize = recv(hgc->sockfd, &hgc->ctx.ch, sizeof(hgc->ctx.ch), 0);
> -	if (rsize != sizeof(hgc->ctx.ch))
> -		abortmsg("failed to read channel");
> +	if (rsize != sizeof(hgc->ctx.ch)) {
> +		debugmsg("failed to read channel");
> +		exit(255);
> +	}

I've inserted a short comment why we don't use abortmsg().


More information about the Mercurial-devel mailing list