On Fri, Dec 14, 2012 at 10:17:24AM -0800, Javier Cardona wrote: Thanks for the review! > > + setup->is_authenticated = ifmsh->security & IEEE80211_MESH_SEC_AUTHED; > + setup->is_authenticated = !!(ifmsh->security & > IEEE80211_MESH_SEC_AUTHED)); > > i.e. to force the value to be 0 or 1. Agreed if they were ints, but in this case .is_authenticated and .is_secure are type bool, so it shouldn't make a difference, right? -- Bob Copeland %% www.bobcopeland.com -- 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