[riot-notifications] [RIOT-OS/RIOT] add ARRAYSIZE macro and make use of it (#11865)

Kees Bakker notifications at github.com
Thu Jul 25 21:02:10 CEST 2019


keestux approved this pull request.

It looks OK to me.

I've used spatch as well to find potential candidates for the macro, and most of these candidates have been incorporated.

I did not do much testing (yet), only visual inspection. That isn't hard to do, except that there are so many. I'm hoping that our testing system will catch introduced errors, if there happen to be any.



-- 
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/11865#pullrequestreview-266825001
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20190725/b3040b3d/attachment.htm>


More information about the notifications mailing list