RE: [Bluez,v1] shared/gatt-server: Fix read multiple charc values

[bluez.test.bot@gmail.com]

[-- Attachment #1: Type: text/plain, Size: 1707 bytes --]


This is automated email and please do not reply to this email!

Dear submitter,

Thank you for submitting the patches to the linux bluetooth mailing list.
While we are preparing for reviewing the patches, we found the following
issue/warning.


Test Result:
Checkbuild Failed

Patch Series:
[Bluez,v1] shared/gatt-server: Fix read multiple charc values


Outputs:
ar: `u' modifier ignored since `D' is the default (see `U')
src/shared/gatt-server.c: In function ‘read_multiple_complete_cb’:
src/shared/gatt-server.c:42:24: error: comparison of unsigned expression < 0 is always false [-Werror=type-limits]
   42 | #define MAX(a, b) ((a) > (b) ? (a) : (b))
      |                        ^
src/shared/gatt-server.c:46:27: note: in definition of macro ‘MIN’
   46 | #define MIN(a, b) ((a) < (b) ? (a) : (b))
      |                           ^
src/shared/gatt-server.c:1065:13: note: in expansion of macro ‘MAX’
 1065 |    MIN(len, MAX(0, data->mtu - data->length - 3)) :
      |             ^~~
src/shared/gatt-server.c:42:24: error: comparison of unsigned expression < 0 is always false [-Werror=type-limits]
   42 | #define MAX(a, b) ((a) > (b) ? (a) : (b))
      |                        ^
src/shared/gatt-server.c:46:39: note: in definition of macro ‘MIN’
   46 | #define MIN(a, b) ((a) < (b) ? (a) : (b))
      |                                       ^
src/shared/gatt-server.c:1065:13: note: in expansion of macro ‘MAX’
 1065 |    MIN(len, MAX(0, data->mtu - data->length - 3)) :
      |             ^~~
cc1: all warnings being treated as errors
make[1]: *** [Makefile:6807: src/shared/gatt-server.lo] Error 1
make: *** [Makefile:4010: all] Error 2



---
Regards,
Linux Bluetooth