pussuw commented on code in PR #16194:
URL: https://github.com/apache/nuttx/pull/16194#discussion_r2059313697
##########
libs/libc/semaphore/sem_init.c:
##########
@@ -62,15 +62,15 @@
*
****************************************************************************/
-int nxsem_init(FAR sem_t *sem, int pshared, unsigned int value)
+int nxsem_init(FAR sem_t *sem, int pshared, int32_t value)
Review Comment:
I now see why you had to do it:
Error: misc/lib_mutex.c:94:40: error: large integer implicitly truncated to
unsigned type [-Werror=overflow]
int ret = nxsem_init(&mutex->sem, 0, NXSEM_NO_MHOLDER);
On some platforms unsigned int is 16-bit, not large enough to hold the u32
mask for NXSEM_NO_MHOLDER
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
