xiaoxiang781216 commented on PR #6613:
URL: https://github.com/apache/incubator-nuttx/pull/6613#issuecomment-1186553363

   > Yea, there won't be a "-1" bug now, but this will trigger an assert for 
unsigned ll bigger than `ULONG_MAX/2`. I guess you could do:
   > 
   > `DEBUGASSERT((LONG_MIN <= x && x <= LONG_MAX) || (0 < x && x <= 
ULONG_MAX));`
   > 
   > To handle both signed and unsigned cases.
   
   Done.


-- 
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: commits-unsubscr...@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to