On 2022/10/24 21:59, Yangtao Li wrote:
Replace
        if (A) {
                ......
                return err;
        }
        return err;
with
        if (A) {
                ......
        }
        return err;

Signed-off-by: Yangtao Li <[email protected]>
---
  fs/f2fs/segment.c | 1 -
  1 file changed, 1 deletion(-)

diff --git a/fs/f2fs/segment.c b/fs/f2fs/segment.c
index acf3d3fa4363..f6f16d691226 100644
--- a/fs/f2fs/segment.c
+++ b/fs/f2fs/segment.c
@@ -647,7 +647,6 @@ int f2fs_create_flush_cmd_control(struct f2fs_sb_info *sbi)
                err = PTR_ERR(fcc->f2fs_issue_flush);
                kfree(fcc);
                SM_I(sbi)->fcc_info = NULL;
-               return err;
        }
return err;

It may looks more clean if it return 0 here?

Thanks,



_______________________________________________
Linux-f2fs-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/linux-f2fs-devel

Reply via email to