[riot-notifications] [RIOT-OS/RIOT] drivers/w5100: Fixed netdev_driver_t::recv() API (#10412)

Marian Buschsieweke notifications at github.com
Wed Jan 23 17:50:32 CET 2019


OK, previous code did update `rp` (without using the result) in the drop case in a very sneaky way (see fixup above). As the result of this update was never used, moving the code into `static inline void drop()` cannot have introduced a bug. But I still dropped the sneaky `rp += psize` as argument in a function call, as this will only confuse the reader.

-- 
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/10412#issuecomment-456878122
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20190123/134343bd/attachment-0001.html>


More information about the notifications mailing list