[prev in list] [next in list] [prev in thread] [next in thread] 

List:       linux-bluetooth
Subject:    Re: [PATCH] misc fixes for HFP HF role
From:       Johan Hedberg <johan.hedberg () gmail ! com>
Date:       2009-06-30 9:44:57
Message-ID: 20090630094457.GA3877 () jh-x301
[Download RAW message or body]

Hi Forrest,

There were still a couple of issues:

On Tue, Jun 30, 2009, Forrest Zhao wrote:
> +		debug("connection with remote BT is closed\n");

Redundant newline character.

>  	if (cond & (G_IO_ERR | G_IO_HUP)) {
> +		debug("sco connection is released\n");
>  		GIOChannel *chan = gw->sco;

Variable declaration after code and redundant newline character.

> -	g_io_channel_close(chan);
> +	g_io_channel_shutdown(chan);

Too few parameters to g_io_channel_shutdown. Please at least compile-check
before submitting patches, even if your change is trivial.

However, I went ahead and fixed these issues myself and pushed the patch
upstream.

It seems that there were plenty of redundant newline characters in debug
logs in gateway.c (where you probably got the wrong idea to start with)
and I fixed these in a separate commit.

Johan
--
To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[prev in list] [next in list] [prev in thread] [next in thread]