[GitHub] [nuttx] jlaitine commented on a diff in pull request #8075: Minor fs/mmap improvement

[GitBox <gi...@apache.org>]

jlaitine commented on code in PR #8075:
URL: https://github.com/apache/nuttx/pull/8075#discussion_r1067970433


##########
fs/mmap/fs_rammap.c:
##########
@@ -176,7 +175,7 @@ int rammap(FAR struct file *filep, FAR struct mm_map_entry_s *entry,
   rdbuffer = kernel ? kmm_malloc(length) : kumm_malloc(length);
   if (!rdbuffer)
     {
-      ferr("ERROR: Region allocation failed, length: %d\n", (int)length);
+      ferr("ERROR: Region allocation failed, length: %zu\n", length);

Review Comment:
   %z is C99 addition AFAIK, is it ok to use here?



-- 
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-unsubscribe@nuttx.apache.org

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