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
Outputs:
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
src/adv_monitor.c: In function ‘monitor_proxy_removed_cb’:
src/adv_monitor.c:756:14: error: unused variable ‘path’ [-Werror=unused-variable]
756 | const char *path = g_dbus_proxy_get_path(proxy);
| ^~~~
src/adv_monitor.c: In function ‘app_remove_monitor’:
src/adv_monitor.c:1014:38: error: passing argument 2 of ‘queue_find’ from incompatible pointer type [-Werror=incompatible-pointer-types]
1014 | monitor = queue_find(app->monitors, removed_monitor_match, user_data);
| ^~~~~~~~~~~~~~~~~~~~~
| |
| _Bool (*)(void *, void *)
In file included from src/adv_monitor.c:42:
./src/shared/queue.h:52:7: note: expected ‘queue_match_func_t’ {aka ‘_Bool (*)(const void *, const void *)’} but argument is of type ‘_Bool (*)(void *, void *)’
52 | void *queue_find(struct queue *queue, queue_match_func_t function,
| ^~~~~~~~~~
src/adv_monitor.c: In function ‘adv_monitor_removed_callback’:
src/adv_monitor.c:1031:43: error: initialization discards ‘const’ qualifier from pointer target type [-Werror=discarded-qualifiers]
1031 | struct mgmt_ev_adv_monitor_removed *ev = param;
| ^~~~~
src/adv_monitor.c: In function ‘adv_match_per_pattern’:
src/adv_monitor.c:1230:17: error: unused variable ‘eir’ [-Werror=unused-variable]
1230 | const uint8_t *eir = info->eir;
| ^~~
src/adv_monitor.c: In function ‘btd_adv_monitor_content_filter’:
src/adv_monitor.c:1286:2: error: ISO C90 forbids mixed declarations and code [-Werror=declaration-after-statement]
1286 | struct adv_content_filter_info info = {
| ^~~~~~
src/adv_monitor.c: In function ‘btd_adv_monitor_notify_monitors’:
src/adv_monitor.c:1322:2: error: ISO C90 forbids mixed declarations and code [-Werror=declaration-after-statement]
1322 | struct adv_rssi_filter_info info = {
| ^~~~~~
cc1: all warnings being treated as errors
make[1]: *** [Makefile:6863: src/adv_monitor.o] Error 1
make: *** [Makefile:4056: all] Error 2
---
Regards,
Linux Bluetooth
