On Sat, May 06, 2017 at 11:01:05PM +0100, Colin King wrote:
> From: Colin Ian King <colin.king@xxxxxxxxxxxxx>
>
> Variable ret is assigned to zero and is always zero throughout the
> function. Thus the check for ret being less than zero is always
> false and so mapping_set_error always has an -EIO error passed to
> it. Hence we can remove the redundant assignment and check on ret.
>
> Detected by CoverityScan, CID#1414312 ("Logically dead code")
>
> Signed-off-by: Colin Ian King <colin.king@xxxxxxxxxxxxx>
Reviewed-by: David Sterba <dsterba@xxxxxxxx>
--
To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html