Return-Path: Date: Tue, 12 Oct 2010 10:35:31 +0100 From: Johan Hedberg To: Jose Antonio Santos Cadenas Cc: linux-bluetooth@vger.kernel.org Subject: Re: Pull request git://gitorious.org/bluez-mcap-hdp/mcap-hdp.git for_upstream Message-ID: <20101012093531.GA22165@jh-x301> References: <20101011213154.GA12131@jh-x301> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii In-Reply-To: Sender: linux-bluetooth-owner@vger.kernel.org List-ID: Hi Jose, On Tue, Oct 12, 2010, Jose Antonio Santos Cadenas wrote: > I'm sorry but I can't connect to the IRC rigth now. Nevertheless, I > think that I've found the issues: > > - The iochannel should be unrefereed when the watcher is explicitly removed Yes, though do you need to have a reference at all? As long as you have the GLib watch callback GLib itself holds a reference, so if you don't need one just remove chan->echo_chan and do an unref after calling add_watch. > - The commit message s/paramter/parameter > > Are there any more? s/Imcoming/Incoming/ s/DBus/D-Bus/ And the following doesn't make much sense to me: "Delete all channels DBus interface when the instance is removed" Could you rephrase it somehow, maybe by putting most of it in the message body instead of the summary line. Is it trying to say "Delete the D-Bus interfaces of all channels"? The summary line shouldn't be too complex. You can just say "Fix D-Bus channel removal when removing instances" and then in the message body do the more detailed explanation. Johan