Re: [RFC] bloodpressure: Add Blood Pressure API

[Rymarkiewicz Waldemar <waldemar.rymarkiewicz@tieto.com>]

Hi,

>>> +Interface	org.bluez.BloodPressure1
>>> +Object path	[variable prefix]/{hci0,hci1,...}/dev_XX_XX_XX_XX_XX_XX
>>> +
>>> +Methods		RegisterWatcher(object watcher)
>>> +
>>> +			Registers a watcher to monitor blood pressure
>>> +			measurements. This watcher will be notified about
>>> +			final pressure measurements.
>>> +
>>> +			Possible Errors: org.bluez.Error.InvalidArguments
>>> +
>>> +		UnregisterWatcher(object watcher)
>>> +
>>> +			Unregisters a watcher.
>>> +
>>> +		EnableIntermediateMeasurement(object watcher)
>>> +
>>> +			Enables intermediate measurement notifications
>>> +			for this watcher. Intermediate measurements will
>>> +			be enabled only for devices which support it.
>>> +
>>> +			Possible Errors: org.bluez.Error.InvalidArguments
>>> +					org.bluez.Error.NotSupported
>>> +
>>> +		DisableIntermediateMeasurement(object watcher)
>>> +
>>> +			Disables intermediate measurement notifications
>>> +			for this watcher. It will disable notifications in
>>> +			devices when the last watcher is removed for
>>> +			intermediate measurements.
>>> +
>>> +			Possible Errors: org.bluez.Error.InvalidArguments
>>> +					org.bluez.Error.NotFound
>>
>> why don't we make these two methods parts of a dict for RegisterWatcher. Enabling them manually seems rather pointless.
>
> It's been done intentionally to be able to return "not supported" error
> when intermediate measurement is not available.
>
> Having RegisterWatcher(object watcher, boolean intermediate) and
> additionally Intermediate property [read only] is an another option.
>
> In this case app will check intermediate property and set intermediate
> param to RegisterWatcher appropriately.
>

Any comments on this.

/Waldek