[riot-notifications] [RIOT-OS/RIOT] pkg/wakaama: Add basic LWM2M client implementation (#11036)

Juan I Carrano notifications at github.com
Fri Jun 7 10:25:15 CEST 2019


jcarrano commented on this pull request.



> @@ -27,6 +27,19 @@
 /* Set to true if reboot requested. */
 static bool reboot;
 
+/* Lookup table for static resources of device object */
+static const char *_static_resources[] = {

The array element are not contiguous, so the array will have holes and be bigger than needed. A possible solution is to rearrange LWM2M_RES enum (btw, it is better for generating documentation if enums have a name) so that all static resources appear at the beginning.

-- 
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/11036#pullrequestreview-246969715
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20190607/9e3011ed/attachment.html>


More information about the notifications mailing list