On Wed, Oct 18, 2006 at 03:56:07PM +0200, Jiri Benc wrote: > On Mon, 9 Oct 2006 13:11:02 -0700, David Kimdon wrote: > > --- wireless-dev.orig/net/d80211/ieee80211_sta.c > > +++ wireless-dev/net/d80211/ieee80211_sta.c > > @@ -930,8 +930,8 @@ static void ieee80211_rx_mgmt_auth(struc > > printk(KERN_DEBUG "%s: AP denied authentication (auth_alg=%d " > > "code=%d)\n", dev->name, ifsta->auth_alg, status_code); > > if (status_code == WLAN_STATUS_NOT_SUPPORTED_AUTH_ALG) { > > - const int num_algs = 3; > > - u8 algs[num_algs]; > > + u8 algs[3]; > > + const int num_algs = ARRAY_SIZE(algs); > > Wouldn't it be better just to use ARRAY_SIZE(algs) and get rid of > num_algs completely?
I actually think the code reads slightly cleaner using num_algs, but don't have a strong preference. I'd be happy to make the change if removing num_algs is preferred. -David - To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html