[riot-notifications] [RIOT-OS/RIOT] gnrc_sixlowpan_frag_rb: migrate more functions from gnrc_sixlowpan_frag (#12324)

benpicco notifications at github.com
Sun Sep 29 17:55:02 CEST 2019


benpicco commented on this pull request.



>   * compiled in */
-#define gnrc_sixlowpan_frag_rbuf_remove(rbuf)       (void)(rbuf)
-#define gnrc_sixlowpan_frag_rbuf_dispatch_when_complete(rbuf, netif) \
+#define gnrc_sixlowpan_frag_rb_remove(rbuf)     (void)(rbuf)
+#define gnrc_sixlowpan_frag_rb_dispatch_when_complete(rbuf, netif) \
     (void)(rbuf); (void)(netif)

I'm not suggesting to change this here as this is just about moving functionality around and that style is used throughout the rest of the module, but generally an empty `static inline` function will achieve the same result with less way for something to go wrong.

-- 
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/12324#pullrequestreview-294689918
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20190929/37348845/attachment.htm>


More information about the notifications mailing list