[riot-notifications] [RIOT-OS/RIOT] pkg/nimble/autoadv: make AD flag optionnal (#16703)

Francisco notifications at github.com
Mon Aug 23 10:20:47 CEST 2021


@fjmolinas commented on this pull request.



> @@ -160,8 +160,14 @@ void nimble_autoadv_reset(void)
     int rc = 0;
     (void) rc;
 
-    rc = bluetil_ad_init_with_flags(&_ad, buf, sizeof(buf), BLUETIL_AD_FLAGS_DEFAULT);
-    assert(rc == BLUETIL_AD_OK);
+    if (IS_ACTIVE(NIMBLE_AUTOADV_FLAG_FIELD)) {

Hmm this is always defined though since it's not actually a configuration(yet), so I guess it does not hurt adding this.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/RIOT-OS/RIOT/pull/16703#discussion_r693761103
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20210823/05069875/attachment.htm>


More information about the notifications mailing list