On Thu, May 07, 2015 at 08:03:06AM -0400, Jes Sorensen wrote: > "Gujulan Elango, Hari Prasath (H.)" <hgujulan@xxxxxxxxxxx> writes: > > Remove the goto and return error directly thereby removing a variable > > > > Signed-off-by: Hari Prasath Gujulan Elango <hgujulan@xxxxxxxxxxx> > > --- > > 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 -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
