From: Marcel Holtmann <marcel@holtmann.org>
To: BlueZ development <bluez-devel@lists.sourceforge.net>
In-Reply-To: <20061110081232.GA27901@localhost.localdomain>
References: <20060817122559.GA6422@srcf.ucam.org>
	<20060817124514.GA27427@localhost.localdomain>
	<1155819813.18545.36.camel@cookie.hadess.net>
	<1155839761.4075.165.camel@aeonflux.holtmann.net>
	<44E5B2A5.1070002@palmsource.com>
	<1155924372.4075.191.camel@aeonflux.holtmann.net>
	<45362E18.1040909@palmsource.com>
	<20061018140352.GA11008@localhost.localdomain>
	<45390275.1010607@palmsource.com> <4550C24B.9080703@palmsource.com>
	<20061110081232.GA27901@localhost.localdomain>
Date: Fri, 10 Nov 2006 17:40:12 +0100
Message-Id: <1163176812.4944.1.camel@aeonflux.holtmann.net>
Mime-Version: 1.0
Subject: Re: [Bluez-devel]
	DBus	interface	-	determining	whether	a	device	exists
Reply-To: BlueZ development <bluez-devel@lists.sourceforge.net>
Content-Type: text/plain; charset="us-ascii"
Sender: bluez-devel-bounces@lists.sourceforge.net
Errors-To: bluez-devel-bounces@lists.sourceforge.net

Hi Johan,

> > Are you OK with this patch ?
> 
> I'm fine with it (although I don't have a good use case for it). However, 
> it'll still need Marcel's blessing before being included.

looks okay to me. Except there are two or three minor whitespace issues
in the patch. One in a function call and others for the casts. Please
fix them and then I am okay with that patch. I would prefer that apitest
also gets updated to use these two new methods.

Regards

Marcel


-------------------------------------------------------------------------
Using Tomcat but need to do more? Need to support web services, security?
Get stuff done quickly with pre-integrated technology to make your job easier
Download IBM WebSphere Application Server v.1.0.1 based on Apache Geronimo
http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642
_______________________________________________
Bluez-devel mailing list
Bluez-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/bluez-devel