Re: [PATCH] misc fixes for HFP HF role

Johan Hedberg
2009-06-30T09:44:57Z

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