[Cluster-devel] [GFS2 Patch] Eliminate useless err variable
Steven Whitehouse
swhiteho at redhat.com
Wed May 12 09:30:17 UTC 2010
Hi,
Now in the -nmw git tree. Thanks,
Steve.
On Tue, 2010-05-11 at 17:35 -0400, Bob Peterson wrote:
> Hi,
>
> This patch removes an unneeded "err" variable that is always
> returned as zero.
>
> Regards,
>
> Bob Peterson
> Red Hat File Systems
> --
> fs/gfs2/meta_io.c | 4 +---
> 1 files changed, 1 insertions(+), 3 deletions(-)
>
> diff --git a/fs/gfs2/meta_io.c b/fs/gfs2/meta_io.c
> index abafda1..18176d0 100644
> --- a/fs/gfs2/meta_io.c
> +++ b/fs/gfs2/meta_io.c
> @@ -34,7 +34,6 @@
>
> static int gfs2_aspace_writepage(struct page *page, struct writeback_control *wbc)
> {
> - int err;
> struct buffer_head *bh, *head;
> int nr_underway = 0;
> int write_op = (1 << BIO_RW_META) | ((wbc->sync_mode == WB_SYNC_ALL ?
> @@ -86,11 +85,10 @@ static int gfs2_aspace_writepage(struct page *page, struct writeback_control *wb
> } while (bh != head);
> unlock_page(page);
>
> - err = 0;
> if (nr_underway == 0)
> end_page_writeback(page);
>
> - return err;
> + return 0;
> }
>
> const struct address_space_operations gfs2_meta_aops = {
More information about the Cluster-devel
mailing list