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