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 07:00:02 UTC

[incubator-nuttx] 03/03: fs/mount/fs_mount.c: Remove a redandunt 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 1078210f7f886ee88847f6add39c8819d258a6a6
Author: YAMAMOTO Takashi <ya...@midokura.com>
AuthorDate: Thu Jul 30 13:27:35 2020 +0900

    fs/mount/fs_mount.c: Remove a redandunt assignment
    
    Found by clang-check:
    
    mount/fs_mount.c:287:8: warning: Although the value stored to 'ret' is used in
          the enclosing expression, the value is never actually read from 'ret'
          (ret = find_blockdriver(source, mountflags, &drvr_inode)) >= 0)
           ^     ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    1 warning generated.
---
 fs/mount/fs_mount.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/fs/mount/fs_mount.c b/fs/mount/fs_mount.c
index 3e33c75..921decd 100644
--- a/fs/mount/fs_mount.c
+++ b/fs/mount/fs_mount.c
@@ -284,7 +284,7 @@ int nx_mount(FAR const char *source, FAR const char *target,
 
 #ifdef BDFS_SUPPORT
   if (source != NULL &&
-      (ret = find_blockdriver(source, mountflags, &drvr_inode)) >= 0)
+      find_blockdriver(source, mountflags, &drvr_inode) >= 0)
     {
       /* Find the block based file system */