On Wed, 2012-04-25 at 15:45 +0300, Andrei Emeltchenko wrote: > From: Andrei Emeltchenko <andrei.emeltchenko@xxxxxxxxx> > > Use default key to handle similar iface types. This removes compile warnings: > > net/wireless/util.c: In function ‘cfg80211_change_iface’: > net/wireless/util.c:846:3: warning: enumeration value ‘NL80211_IFTYPE_MAX’ > not handled in switch [-Wswitch] > > Signed-off-by: Andrei Emeltchenko <andrei.emeltchenko@xxxxxxxxx> > --- > net/wireless/util.c | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) > > diff --git a/net/wireless/util.c b/net/wireless/util.c > index 1b7a08d..468dc8b 100644 > --- a/net/wireless/util.c > +++ b/net/wireless/util.c > @@ -862,8 +862,7 @@ int cfg80211_change_iface(struct cfg80211_registered_device *rdev, > case NL80211_IFTYPE_MONITOR: > /* monitor can't bridge anyway */ > break; > - case NL80211_IFTYPE_UNSPECIFIED: > - case NUM_NL80211_IFTYPES: > + default: When did this warning start occurring? I was pretty sure it didn't use to? I think we should explicitly list the cases here though because we want the compiler to warn here when we add something to the interface type list so we catch this place that may have to be modified. 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
