[riot-notifications] [RIOT-OS/RIOT] core/cib: fix documented maximum size restrictions (#14076)

Martine Lenders notifications at github.com
Tue May 19 08:43:51 CEST 2020


@miri64 commented on this pull request.



> @@ -39,6 +39,9 @@ typedef struct {
 
 /**
  * @brief   Initialize cib_t to a given size.
+ *
+ * @param[in]  SIZE     Size of the buffer, must not exceed `UINT_MAX` + 1.

Yes, because in the function below the value is restricted by the value space of the parameter, here it is not, and it needs only be max(value_space) - 1 for the member. So yes, the values don't match but so does the behavior.

-- 
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/14076#discussion_r427064907
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20200518/aecc41ae/attachment.htm>


More information about the notifications mailing list