Re: [PATCH 00/15] thermometer plugin updates

[Johan Hedberg <johan.hedberg@gmail.com>]

Hi Andrzej,

On Fri, Nov 09, 2012, Andrzej Kaczmarek wrote:
> Here are few patches to update thermometer plugin with features already
> implemented in other plugins:
> - store attributes handles directly in thermometer structure instead of
>   nested lists of structures which we need to traverse later (and we
>   only need 3 handles stored to support all use cases)
> - register attio handlers for specific handles instead of one ind and
>   notif handler globally
> - change properties handling to DBus.Properties
> - and some minor fixes
> 
> This is tested with PTS 4.5.3. There's only problem with patch #9 which
> "breaks" testcase TP/THF/CO/BV-09-I - this is because PTS sends invalid
> properties for Measurement Interval characteristic (it does not have
> indicate property so we do not register ind handler for it). I already
> filled issue on PTS.
> 
> Comments are welcome.
> 
> 
> Andrzej Kaczmarek (15):
>   thermometer: Store Temperature Measurement CCC handle in struct
>   thermometer: Store Intermediate Temperature CCC handle in struct
>   thermometer: Store Measurement Interval value handle in struct
>   thermometer: Use dedicated handler for Intermediate Temperature
>   thermometer: Use dedicated handler for Temperature Measurement
>   thermometer: Use dedicated handler for Measurement Interval
>   thermometer: Remove descriptor structure
>   thermometer: Remove storage of all discovered characteristics
>   thermometer: Discover Measurement Interval descriptors only if needed
>   thermometer: Always write CCC value when connecting
>   thermometer: Make temp_type array static
>   thermometer: Add DBus.Properties support
>   thermometer: Remove legacy properties code
>   doc: Update thermometer API document
>   test: Update test-thermometer for DBus.Properties
> 
>  doc/thermometer-api.txt            |   18 -
>  profiles/thermometer/thermometer.c | 1008 +++++++++++++++++-------------------
>  test/test-thermometer              |   13 +-
>  3 files changed, 478 insertions(+), 561 deletions(-)

All patches in this set have been applied. Thanks.

Johan