| Message ID | 20230424234648.577673-1-lizetao1@huawei.com (mailing list archive) |
|---|---|
| State | Accepted |
| Commit | 1223e432d9e16df39ba51f496c6ad3d7d560f612 |
| Headers | show |
| Series | [f2fs-dev,-next] f2fs: remove redundant goto statement in f2fs_read_single_page() | expand |
On 2023/4/25 7:46, Li Zetao wrote: > After the commit "0a4ee518185", this "goto" statement was redundant, > remote it for clean code. > > Signed-off-by: Li Zetao <lizetao1@huawei.com> Reviewed-by: Chao Yu <chao@kernel.org> Thanks,
Hello: This patch was applied to jaegeuk/f2fs.git (dev) by Jaegeuk Kim <jaegeuk@kernel.org>: On Mon, 24 Apr 2023 23:46:48 +0000 you wrote: > After the commit "0a4ee518185", this "goto" statement was redundant, > remote it for clean code. > > Signed-off-by: Li Zetao <lizetao1@huawei.com> > --- > fs/f2fs/data.c | 1 - > 1 file changed, 1 deletion(-) Here is the summary with links: - [f2fs-dev,-next] f2fs: remove redundant goto statement in f2fs_read_single_page() https://git.kernel.org/jaegeuk/f2fs/c/1223e432d9e1 You are awesome, thank you!
diff --git a/fs/f2fs/data.c b/fs/f2fs/data.c index 06b552a0aba2..c9a71b4b09dd 100644 --- a/fs/f2fs/data.c +++ b/fs/f2fs/data.c @@ -2171,7 +2171,6 @@ static int f2fs_read_single_page(struct inode *inode, struct page *page, f2fs_update_iostat(F2FS_I_SB(inode), NULL, FS_DATA_READ_IO, F2FS_BLKSIZE); *last_block_in_bio = block_nr; - goto out; out: *bio_ret = bio; return ret;
After the commit "0a4ee518185", this "goto" statement was redundant, remote it for clean code. Signed-off-by: Li Zetao <lizetao1@huawei.com> --- fs/f2fs/data.c | 1 - 1 file changed, 1 deletion(-)