diff options
author | kbuild test robot <fengguang.wu@intel.com> | 2014-11-30 16:42:52 -0500 |
---|---|---|
committer | Dave Chinner <david@fromorbit.com> | 2014-11-30 16:42:52 -0500 |
commit | d254aaec5d1aa2997dad035db340c298eaa8d089 (patch) | |
tree | 61835f26b74687ac2e7d76bcc460539b2bbbe7b7 /fs | |
parent | 8300475ebf03a54ec3c56b6b808ccbd9e1bc320d (diff) |
xfs: fix simple_return.cocci warning in xfs_bmse_shift_one
fs/xfs/libxfs/xfs_bmap.c:5591: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
CC: Brian Foster <bfoster@redhat.com>
Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>
Diffstat (limited to 'fs')
-rw-r--r-- | fs/xfs/libxfs/xfs_bmap.c | 6 |
1 files changed, 1 insertions, 5 deletions
diff --git a/fs/xfs/libxfs/xfs_bmap.c b/fs/xfs/libxfs/xfs_bmap.c index 79c981984dca..20d2e96aef6a 100644 --- a/fs/xfs/libxfs/xfs_bmap.c +++ b/fs/xfs/libxfs/xfs_bmap.c | |||
@@ -5592,12 +5592,8 @@ shift_extent: | |||
5592 | XFS_WANT_CORRUPTED_GOTO(i == 1, out_error); | 5592 | XFS_WANT_CORRUPTED_GOTO(i == 1, out_error); |
5593 | 5593 | ||
5594 | got.br_startoff = startoff; | 5594 | got.br_startoff = startoff; |
5595 | error = xfs_bmbt_update(cur, got.br_startoff, got.br_startblock, | 5595 | return xfs_bmbt_update(cur, got.br_startoff, got.br_startblock, |
5596 | got.br_blockcount, got.br_state); | 5596 | got.br_blockcount, got.br_state); |
5597 | if (error) | ||
5598 | return error; | ||
5599 | |||
5600 | return 0; | ||
5601 | 5597 | ||
5602 | out_error: | 5598 | out_error: |
5603 | return error; | 5599 | return error; |