You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@nuttx.apache.org by je...@apache.org on 2020/07/30 14:16:38 UTC

[incubator-nuttx] 06/08: fs/mqueue/mq_close.c: Remove a redundant assignment

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

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

commit e22dffba8e2e13fa9a3384a7d292807aef972594
Author: YAMAMOTO Takashi <ya...@midokura.com>
AuthorDate: Thu Jul 30 16:58:18 2020 +0900

    fs/mqueue/mq_close.c: Remove a redundant assignment
    
    Found by clang-check:
    
    mqueue/mq_close.c:127:21: warning: Value stored to 'rtcb' during its initialization is never read
      FAR struct tcb_s *rtcb = (FAR struct tcb_s *)nxsched_self();
                        ^~~~   ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    1 warning generated.
---
 fs/mqueue/mq_close.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/fs/mqueue/mq_close.c b/fs/mqueue/mq_close.c
index c42e876..934c7eb 100644
--- a/fs/mqueue/mq_close.c
+++ b/fs/mqueue/mq_close.c
@@ -133,7 +133,6 @@ int nxmq_close(mqd_t mqdes)
 
   sched_lock();
 
-  rtcb = (FAR struct tcb_s *)nxsched_self();
   DEBUGASSERT(mqdes != NULL && rtcb != NULL && rtcb->group != NULL);
 
   /* Then perform the close operation */