On Thu, May 07, 2015 at 08:03:06AM -0400, Jes Sorensen wrote:
> "Gujulan Elango, Hari Prasath (H.)" <hguju...@visteon.com> writes:
> > Remove the goto and return error directly thereby removing a variable
> >
> > Signed-off-by: Hari Prasath Gujulan Elango <hguju...@visteon.com>
> > ---
> >  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

_______________________________________________
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

Reply via email to