You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@nuttx.apache.org by xi...@apache.org on 2023/02/09 12:12:10 UTC

[nuttx] 08/08: mm/mempool: fix visual studio Compiler Warning C4098

This is an automated email from the ASF dual-hosted git repository.

xiaoxiang pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/nuttx.git

commit 2d536059f0bc60d777c7692fdc615392d22d3d65
Author: chao an <an...@xiaomi.com>
AuthorDate: Wed Feb 8 17:06:59 2023 +0800

    mm/mempool: fix visual studio Compiler Warning C4098
    
    D:\archer\code\nuttx\mm\mempool\mempool_multiple.c(180,72): warning C4098: "mempool_multiple_free_callback":"void" void function returning a value
    
    Compiler Warning C4098:
    A function declared with return type void has a return statement that returns a value. The compiler assumes the function returns a value of type int.
    
    Reference:
    https://learn.microsoft.com/en-us/cpp/error-messages/compiler-warnings/compiler-warning-level-1-c4098?view=msvc-170
    
    Signed-off-by: chao an <an...@xiaomi.com>
---
 mm/mempool/mempool_multiple.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/mm/mempool/mempool_multiple.c b/mm/mempool/mempool_multiple.c
index 9e28212917..d5a7f79fac 100644
--- a/mm/mempool/mempool_multiple.c
+++ b/mm/mempool/mempool_multiple.c
@@ -177,7 +177,7 @@ static void mempool_multiple_free_callback(FAR struct mempool_s *pool,
 {
   FAR struct mempool_multiple_s *mpool = pool->priv;
 
-  return mpool->free(mpool->arg, (FAR char *)addr - mpool->minpoolsize);
+  mpool->free(mpool->arg, (FAR char *)addr - mpool->minpoolsize);
 }
 
 /****************************************************************************