Re: [PATCH] ext4: remove some unused code lines

2012-12-06 Thread Theodore Ts'o
On Sun, Dec 02, 2012 at 08:27:26PM +0800, Zheng Liu wrote: > On Thu, Nov 29, 2012 at 06:00:00PM +0800, zwu.ker...@gmail.com wrote: > > From: Zhi Yong Wu > > Please write a commit log to describe this patch, even though it is > quite simple and straightfoward > > Otherwise, it looks good to me. >

[PATCH] ext4: remove some unused code lines

2012-12-02 Thread zwu . kernel
From: Zhi Yong Wu In ext4_ext_in_cache(), some codes are redundant, this patch will remove them. Reviewed-by: Zheng Liu Signed-off-by: Zhi Yong Wu --- fs/ext4/extents.c |2 -- 1 files changed, 0 insertions(+), 2 deletions(-) diff --git a/fs/ext4/extents.c b/fs/ext4/extents.c index 7011

Re: [PATCH] ext4: remove some unused code lines

2012-12-02 Thread Zhi Yong Wu
On Sun, Dec 2, 2012 at 8:27 PM, Zheng Liu wrote: > On Thu, Nov 29, 2012 at 06:00:00PM +0800, zwu.ker...@gmail.com wrote: >> From: Zhi Yong Wu > > Please write a commit log to describe this patch, even though it is > quite simple and straightfoward Good suggestion, thanks. > > Otherwise, it looks

Re: [PATCH] ext4: remove some unused code lines

2012-12-02 Thread Zheng Liu
On Thu, Nov 29, 2012 at 06:00:00PM +0800, zwu.ker...@gmail.com wrote: > From: Zhi Yong Wu Please write a commit log to describe this patch, even though it is quite simple and straightfoward Otherwise, it looks good to me. Reviewed-by: Zheng Liu Regards,

[PATCH] ext4: remove some unused code lines

2012-11-29 Thread zwu . kernel
From: Zhi Yong Wu Signed-off-by: Zhi Yong Wu --- fs/ext4/extents.c |2 -- 1 files changed, 0 insertions(+), 2 deletions(-) diff --git a/fs/ext4/extents.c b/fs/ext4/extents.c index 7011ac9..43ec639 100644 --- a/fs/ext4/extents.c +++ b/fs/ext4/extents.c @@ -2156,7 +2156,6 @@ ext4_ext_in_cach