From mboxrd@z Thu Jan 1 00:00:00 1970 From: Rik van Riel Subject: Re: [PATCH] block: fix returnvar.cocci warnings Date: Wed, 27 May 2015 21:36:06 -0400 Message-ID: <55667106.9000000@redhat.com> References: Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Cc: Jens Axboe , Jeff Moyer , Alexander Viro , linux-fsdevel@vger.kernel.org, linux-kernel@vger.kernel.org To: Julia Lawall , Christoph Hellwig Return-path: Received: from mx1.redhat.com ([209.132.183.28]:53946 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750812AbbE1BgN (ORCPT ); Wed, 27 May 2015 21:36:13 -0400 In-Reply-To: Sender: linux-fsdevel-owner@vger.kernel.org List-ID: On 05/26/2015 03:59 PM, Julia Lawall wrote: > Remove unneeded variable used to store return value. > > Generated by: scripts/coccinelle/misc/returnvar.cocci > > Signed-off-by: Fengguang Wu > Signed-off-by: Julia Lawall > --- > > One could check whether the ret variable was intended to be used for > anything. Which tree is this against? Linus's latest tree still has this in _submit_bh (unless git didn't pull the latest updates right): if (buffer_meta(bh)) rw |= REQ_META; if (buffer_prio(bh)) rw |= REQ_PRIO; bio_get(bio); submit_bio(rw, bio); if (bio_flagged(bio, BIO_EOPNOTSUPP)) ret = -EOPNOTSUPP; bio_put(bio); return ret; } -- All rights reversed