On Fri, 2012-09-07 at 19:37 +0200, Christian Lamparter wrote: > > + case NL80211_IFTYPE_WDS: > > + case NL80211_IFTYPE_MONITOR: > > + case NL80211_IFTYPE_P2P_DEVICE: > > + case NL80211_IFTYPE_UNSPECIFIED: > > + case NUM_NL80211_IFTYPES: > > + case NL80211_IFTYPE_P2P_CLIENT: > > + case NL80211_IFTYPE_P2P_GO: > > + /* shouldn't happen */ > > + WARN_ON(1); > > + break; > > + } > > + > Is there a reason not to use default: ? I like the compiler warning, when we add a new iftype it'll tell use we need to modify this code :-) 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