Re: [PATCH BlueZ 00/14] Add TS Large Database, more server tests.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Luiz,

>> This patchset reworks the way that GATT test databases are defined,
>> defines a "large database" as defined by the test spec, adds a set
>> of new tests from the test spec.
>> 
>> It also includes some minor cleanup.
>> 
>> Michael Janssen (14):
>>  shared/gatt-db: Add gatt_db_find_by_type_value
>>  shared/gatt-server: support Discover by UUID
>>  unit/gatt: Bugfix small database construction
>>  unit/gatt: Add add_desc_with_value function
>>  unit/gatt: Rework service db definitions
>>  unit/gatt: Add Test Spec Large Database 1
>>  unit/gatt: Comment cleanup
>>  unit/gatt: Add /TP/GAD/SR/BV-02-C test
>>  unit/gatt: Add /TP/GAD/SR/BV-03-C test
>>  unit/gatt: Add /TP/GAD/SR/BV-04-C test
>>  unit/gatt: Add clarifying note about dup tests.
>>  unit/gatt: Add /TP/GAD/SR/BV-06-C test
>>  unit/gatt: Add /TP/GAR/SR/BV-01-C test
>>  unit/gatt: Add /TP/GAR/SR/BI-02-C test
>> 
>> src/shared/gatt-db.c     |  27 ++
>> src/shared/gatt-db.h     |   7 +
>> src/shared/gatt-server.c | 107 +++++++
>> unit/test-gatt.c         | 808 +++++++++++++++++++++++++++++++++++++++--------
>> 4 files changed, 812 insertions(+), 137 deletions(-)
>> 
>> --
>> 2.2.0.rc0.207.ga3a616c
> 
> I went ahead and applied patches 3-5, patch 6/14 does not apply:
> 
> Applying: unit/gatt: Add Test Spec Large Database 1
> WARNING:LONG_LINE: line over 80 characters
> #91: FILE: unit/test-gatt.c:961:
> + "11111222223333344444555556666677777888889999900000111112222233333444445555566666777778888899999000001111122222333334444455555666667777788888999990000011111222223333344444555556666677777888889999900000111112222233333444445555566666777778888899999000001111122222333334444455555666667777788888999990000011111222223333344444555556666677777888889999900000111112222233333444445555566666777778888899999000001111122222333334444455555666667777788888999990000011111222223333344444555556666677777888889999900000111112222233"),
> 
> WARNING:LONG_LINE: line over 80 characters
> #295: FILE: unit/test-gatt.c:1165:
> + "1111122222333334444455555666667777788888999"),
> 
> WARNING:LONG_LINE: line over 80 characters
> #306: FILE: unit/test-gatt.c:1176:
> + "22222333334444455555666667777788888999990000"),
> 
> WARNING:LONG_LINE: line over 80 characters
> #317: FILE: unit/test-gatt.c:1187:
> + "333334444455555666667777788888999990000011111"),

maybe it makes sense to split these over multiple lines. If it doesn't, then just force git am to ignore the warning and apply them anyway.

Regards

Marcel

--
To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Bluez Devel]     [Linux Wireless Networking]     [Linux Wireless Personal Area Networking]     [Linux ATH6KL]     [Linux USB Devel]     [Linux Media Drivers]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Big List of Linux Books]

  Powered by Linux