On Wed, Feb 04, 2015 at 06:04:38AM -0500, Nicholas Mc Guire wrote: > A few redundant switch cases as well as a redundant if/else > within one of the cases was consolidated to a single call. > The cases are intentionally retained for documentation purposes.
This statement is not clear. It obviously causes a bug if you just start deleting case statements. > > Signed-off-by: Nicholas Mc Guire <[email protected]> > --- > case WIFI_REASSOCREQ,WIFI_PROBEREQ,WIFI_BEACON,WIFI_ACTION all > have the same effect - notably the also for WIFI_PROBEREQ where > the if/else is executing the same function. > > These redundant cases could all be dropped and consolidated into > the default but probably it is better for documentation/readability > to leave them in the switch/case explicitly. Oh. This explains what you meant. Stop putting this information below the cut off, it's annoying. regards, dan carpenter -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
