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@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
