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

Francisco notifications at github.com
Mon Aug 23 10:18:41 CEST 2021


> > Tested for my nrf52dk and the changes do as intended (after adding the missing `C` to the `FLAGS` at least ;P).
> > You are talking about saving three bytes, but I can only see two bytes that are missing. Where is the third byte hidden?
> 
> Hmm I think its because nimble is then using AltBeacons where the ADFlags are encoded in only two bytes, while iBeacons use a LTV (length type value) encoding which is 3 bytes, I'll double check and update accordingly.

Never mind this, I got confused myself when looking at the debug output, but the if you look at the raw payloads you can see that actually 3 bytes are saved, since LTV `0x020106` is actually gone compared with the second message.

-- 
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#issuecomment-903547273
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20210823/383f19e5/attachment-0001.htm>


More information about the notifications mailing list