From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: From: Johan Hedberg To: bluez-devel@lists.sourceforge.net Subject: Re: [Bluez-devel] [PATCH] Variable arguments to Inquiry and PeriodicInquiry D-BUS methods Message-ID: <20051108173152.GA10590@localhost.localdomain> References: <20051108134311.GA7150@localhost.localdomain> <1131458070.5824.181.camel@blade> <20051108143016.GA7919@localhost.localdomain> <1131462224.5824.198.camel@blade> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii In-Reply-To: Sender: bluez-devel-admin@lists.sourceforge.net Errors-To: bluez-devel-admin@lists.sourceforge.net Reply-To: bluez-devel@lists.sourceforge.net List-Unsubscribe: , List-Id: BlueZ development List-Post: List-Help: List-Subscribe: , List-Archive: Date: Tue, 8 Nov 2005 19:31:52 +0200 On Tue, Nov 08, 2005, Claudio Takahasi wrote: > HAL implements 5 different methods to set device properties. > void SetProperty(string key, any value) > void SetPropertyString(string key, string value) > void SetPropertyInteger(string key, int32 value) > void SetPropertyBoolean(string key, bool value) > void SetPropertyDouble(string key, double value) > > We don't need use this approach because we are extracting/comparing > the message signature before call the function responsible to execute > the task. Our design of having the same method name with differents > signatures is fine! Regarding the name, it's better use the same > method name "SetProperty" or only "Set" for both(device configuration > and Controller). I think it's a matter of what the interface name is. If the interface is "org.bluez.Device", then "SetProperty" describes the method better than "Set". If the interface is "org.bluez.Defaults" then just "Set" would probably be enough. Johan ------------------------------------------------------- SF.Net email is sponsored by: Tame your development challenges with Apache's Geronimo App Server. Download it for free - -and be entered to win a 42" plasma tv or your very own Sony(tm)PSP. Click here to play: http://sourceforge.net/geronimo.php _______________________________________________ Bluez-devel mailing list Bluez-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/bluez-devel