[riot-notifications] [RIOT-OS/RIOT] core: fix c11 atomic definitions (fix gcc9 compilation) (#11514)

Marian Buschsieweke notifications at github.com
Mon May 13 16:12:27 CEST 2019


maribu commented on this pull request.



>  typedef uint32_t I4;
+#endif

On Arch Linux using GCC 9.1.0 for AVR (docker image `archlinux/base`) this (using `unsigned int` instead of the matching `uintXX_t`)  seems not to be required. Can you re-check if this is really required?

To me it would make little sense if the compiler refuses to accept `uint16_t` instead of `unsigned int` on which `uint16_t` is `typedef`ed via `typedef unsigned int uint16_t` judging both common sense and my interpretation of the C standard (Section "6.7.7 Type definitions" in the C99 standard if you care to check).

-- 
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/11514#pullrequestreview-236702379
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20190513/14503f15/attachment.html>


More information about the notifications mailing list