diff options
author | kbuild test robot <fengguang.wu@intel.com> | 2014-11-30 16:24:58 -0500 |
---|---|---|
committer | Dave Chinner <david@fromorbit.com> | 2014-11-30 16:24:58 -0500 |
commit | b72091f2fb28fb9e2746010b54d6cc3c27924e6c (patch) | |
tree | 30d6692ed59fe45db6d4d01815877de45c6f17d2 /fs | |
parent | d2a5e3c6fccb25234a594bacb056e0b3974faf49 (diff) |
libxfs: fix simple_return.cocci warnings
fs/xfs/libxfs/xfs_ialloc.c:1141:1-6: WARNING: end returns can be simpified
Simplify a trivial if-return sequence. Possibly combine with a
preceding function call.
Generated by: scripts/coccinelle/misc/simple_return.cocci
Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>
Reviewed-by: Dave Chinner <dchinner@redhat.com>
Signed-off-by: Dave Chinner <david@fromorbit.com>
Diffstat (limited to 'fs')
-rw-r--r-- | fs/xfs/libxfs/xfs_ialloc.c | 6 |
1 files changed, 1 insertions, 5 deletions
diff --git a/fs/xfs/libxfs/xfs_ialloc.c b/fs/xfs/libxfs/xfs_ialloc.c index 23dcb72fc5e6..705a7530176e 100644 --- a/fs/xfs/libxfs/xfs_ialloc.c +++ b/fs/xfs/libxfs/xfs_ialloc.c | |||
@@ -1137,11 +1137,7 @@ xfs_dialloc_ag_update_inobt( | |||
1137 | XFS_WANT_CORRUPTED_RETURN((rec.ir_free == frec->ir_free) && | 1137 | XFS_WANT_CORRUPTED_RETURN((rec.ir_free == frec->ir_free) && |
1138 | (rec.ir_freecount == frec->ir_freecount)); | 1138 | (rec.ir_freecount == frec->ir_freecount)); |
1139 | 1139 | ||
1140 | error = xfs_inobt_update(cur, &rec); | 1140 | return xfs_inobt_update(cur, &rec); |
1141 | if (error) | ||
1142 | return error; | ||
1143 | |||
1144 | return 0; | ||
1145 | } | 1141 | } |
1146 | 1142 | ||
1147 | /* | 1143 | /* |