On Mo, 2011-08-29 at 15:12 +0200, Patrick Ohly wrote:
> > 1) In the case that we have no PnPInformation info we have...
> > 
> > deviceName = User-modifiable name
> > peerName = User-modifiable name
> 
> Better leave the peerName unset. It's semantic will be "we know for sure
> that this device is a "<vendor>[ <product>]". 

You decided to not implement it like this, did you?

>From your patch:
             // This is the user-modifiable device name. Could be shown in 
GUIs, for example
             localConfigs.insert(pair<string, string>("peerName", 
peerTemplate->m_peerName));

+        // This can be either the user-modifiable device name, vendor
+        // name, or product name (vendor + model). This depends on
+        // whether the device supports the Bluetooth Device ID profile
+        // and, if so, whether we have the model in the lookup table.
         std::string m_peerName;

Was it simply easier to implement this way or do you prefer that
semantic of "peerName" in a template reported by the D-Bus server?

My concern is that if the D-Bus server always reports a value, the D-Bus
client won't be able to determine whether it has reliable information
about the manufacturer and model.

-- 
Best Regards, Patrick Ohly

The content of this message is my personal opinion only and although
I am an employee of Intel, the statements I make here in no way
represent Intel's position on the issue, nor am I authorized to speak
on behalf of Intel on this matter.


_______________________________________________
SyncEvolution mailing list
[email protected]
http://lists.syncevolution.org/listinfo/syncevolution

Reply via email to