Return-path: Received: from mx1.redhat.com ([209.132.183.28]:60745 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932192AbbELLRH (ORCPT ); Tue, 12 May 2015 07:17:07 -0400 From: Jes Sorensen To: "Gujulan Elango\, Hari Prasath \(H.\)" Cc: Dan Carpenter , "devel\@driverdev.osuosl.org" , "gregkh\@linuxfoundation.org" , "linux-wireless\@vger.kernel.org" , "Larry.Finger\@lwfinger.net" , "Babu\, Viswanathan \(V.\)" Subject: Re: [PATCH] staging: rtl8723au : remove goto & return error directly References: <20150507111652.GA10574@IND12F0122> <20150507121934.GH14154@mwanda> <20150507134344.GA11599@IND12F0122> <20150512103213.GA5519@IND12F0122> Date: Tue, 12 May 2015 07:16:56 -0400 In-Reply-To: <20150512103213.GA5519@IND12F0122> (Gujulan Elango's message of "Tue, 12 May 2015 10:37:45 +0000") Message-ID: (sfid-20150512_131720_631437_BD6394CF) MIME-Version: 1.0 Content-Type: text/plain Sender: linux-wireless-owner@vger.kernel.org List-ID: "Gujulan Elango, Hari Prasath (H.)" writes: > On Thu, May 07, 2015 at 10:09:59AM -0400, Jes Sorensen wrote: >> "Gujulan Elango, Hari Prasath (H.)" writes: >> > On Thu, May 07, 2015 at 03:19:34PM +0300, Dan Carpenter wrote: >> >> On Thu, May 07, 2015 at 08:03:06AM -0400, Jes Sorensen wrote: >> >> > "Gujulan Elango, Hari Prasath (H.)" writes: >> >> > > Remove the goto and return error directly thereby removing a variable >> >> > > >> >> > > Signed-off-by: Hari Prasath Gujulan Elango >> >> > > --- >> >> > > drivers/staging/rtl8723au/os_dep/ioctl_cfg80211.c | 7 ++----- >> >> > > 1 file changed, 2 insertions(+), 5 deletions(-) >> >> > >> >> > NACK! >> >> >> >> SHOUTING! >> >> >> >> > >> >> > The goto is cleaner and more consistent, rather than hiding a return in >> >> > the middle of the code. >> >> >> >> More consistent with what? There are already returns earlier in the >> >> function. Do-nothing gotos are pointless and annoying. >> >> >> >> regards, >> >> dan carpenter >> >> >> > >> > I sent this patch because goto & return was mixed up in this function. >> > There was a return as well as goto used. And i guess goto is not >> > encouraged anywhere. In this case especially,I couldn't see any >> > significant cleanup being done in the goto label. >> >> I will accept a patch to remove the returns in the middle of the >> function, if you wish to submit that. >> >> A return at the very beginning of a function is OK, but in the middle of >> a larger function they make it harder to catch when code exists. >> >> Jes > > Is this accepted as it is or should I send v2 with the goto retained but > the return in the middle of the function also converted to goto for > consistency.I agree that goto's are used at many places in the kernel > but this is really a small function where it doesn't seem to help > much.Also there is no signficant cleanup being done in the definition of > the label. As I stated previously, this is not accepted. Jes