[riot-notifications] [RIOT-OS/RIOT] net/gcoap: make options buf macros configurable (#10791)
notifications at github.com
Wed Jan 16 19:10:49 CET 2019
### Contribution description
By default, the GCOAP_xxx_OPTIONS_BUF macros set aside four bytes in a message buffer to insert a Content-Format option during gcoap_finish(). However, the new Pkt Add API no longer uses gcoap_finish(). In this case, the user can save a little memory by zeroing these macros. This PR makes the macros externally configurable. After merging this PR, I will add them to the gcoap config doc group per #10566.
This PR undoes commit 034c78d in #9156 that moved the macros into gcoap.c. It is more important to include them in the new config doc group so the user is aware of them.
### Testing procedure
Just compiling the gcoap example ensures the macros still are defined. You also could try sending and receiving a message in the example.
### Issues/PRs references
You can view, comment on, or merge this pull request online at:
-- Commit Summary --
* net/gcoap: make options buf macros configurable
-- File Changes --
M sys/include/net/gcoap.h (30)
M sys/net/application_layer/gcoap/gcoap.c (9)
-- Patch Links --
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the notifications