On Mon, Sep 30, 2019 at 07:28:54PM +0530, Aliasgar Surti wrote: > From: Aliasgar Surti <aliasgar.surti500@xxxxxxxxx> > > Removed unused error variable. Instead of using error variable, > returned the value directly as it wasn't updated. > > Signed-off-by: Aliasgar Surti <aliasgar.surti500@xxxxxxxxx> Looks ok, Reviewed-by: Darrick J. Wong <darrick.wong@xxxxxxxxxx> --D > --- > fs/xfs/scrub/refcount.c | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) > > diff --git a/fs/xfs/scrub/refcount.c b/fs/xfs/scrub/refcount.c > index 93b3793b..0cab11a 100644 > --- a/fs/xfs/scrub/refcount.c > +++ b/fs/xfs/scrub/refcount.c > @@ -341,7 +341,6 @@ xchk_refcountbt_rec( > xfs_extlen_t len; > xfs_nlink_t refcount; > bool has_cowflag; > - int error = 0; > > bno = be32_to_cpu(rec->refc.rc_startblock); > len = be32_to_cpu(rec->refc.rc_blockcount); > @@ -366,7 +365,7 @@ xchk_refcountbt_rec( > > xchk_refcountbt_xref(bs->sc, bno, len, refcount); > > - return error; > + return 0; > } > > /* Make sure we have as many refc blocks as the rmap says. */ > -- > 2.7.4 >
