[riot-notifications] [RIOT-OS/RIOT] crypto/cbc: add return values to documentation (#8641)

Pieter Willemsen notifications at github.com
Mon Feb 26 13:38:14 CET 2018


pwillem commented on this pull request.



> @@ -20,6 +20,7 @@
 #ifndef CRYPTO_MODES_CBC_H
 #define CRYPTO_MODES_CBC_H
 
+#include <stddef.h>

I tought RIOT's header files should be stand alone. This header file only includes `ciphers.h` which only includes `stdint.h`.

More practical: I removed RIOT's `assert.h` in order to test with c11 `static_assert` and I got an compilation error in this header because `size_t` wasn't defined in my app. 

-- 
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/8641#discussion_r170577808
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20180226/8f972491/attachment.html>


More information about the notifications mailing list