[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a diff in pull request #6995: mm: Check the function result with suitable macro.

[GitBox]

xiaoxiang781216 commented on code in PR #6995:
URL: https://github.com/apache/incubator-nuttx/pull/6995#discussion_r962406496


##########
mm/mm_heap/mm_extend.c:
##########
@@ -52,6 +52,7 @@
 void mm_extend(FAR struct mm_heap_s *heap, FAR void *mem, size_t size,
                int region)
 {
+  bool ret;

Review Comment:
   move after line 60



##########
mm/mm_heap/mm_malloc.c:
##########
@@ -104,6 +104,7 @@ void mm_dump_handler(FAR struct tcb_s *tcb, FAR void *arg)
 
 FAR void *mm_malloc(FAR struct mm_heap_s *heap, size_t size)
 {
+  bool val;

Review Comment:
   ditto



##########
mm/mm_heap/mm_realloc.c:
##########
@@ -64,6 +64,7 @@
 FAR void *mm_realloc(FAR struct mm_heap_s *heap, FAR void *oldmem,
                      size_t size)
 {
+  bool ret;

Review Comment:
   ditto



##########
mm/mm_heap/mm_initialize.c:
##########
@@ -58,6 +58,7 @@
 void mm_addregion(FAR struct mm_heap_s *heap, FAR void *heapstart,
                   size_t heapsize)
 {
+  bool ret;

Review Comment:
   move after line 63



##########
mm/mm_heap/mm_memalign.c:
##########
@@ -51,6 +51,7 @@
 FAR void *mm_memalign(FAR struct mm_heap_s *heap, size_t alignment,
                       size_t size)
 {
+  bool ret;

Review Comment:
   ditto



-- 
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