On Mon, 2013-12-23 at 19:07 +0800, fred.chou.nd@xxxxxxxxx wrote: > From: Fred Chou <fred.chou.nd@xxxxxxxxx> > > It's not necessary to check a flag setting and then > set it anyway. The outer condition check is redundant > in the code below. Remove the outer if. It may be redundant in theory, but I think it makes the code easier to read, and it avoids all the other checks and the flags write. I don't think I want to apply this. johannes -- 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
