On Thu, Jun 07, 2007 at 08:38:27PM -0700, Michael Wu wrote:
> On Thursday 07 June 2007 20:03, Larry Finger wrote:
> > for (i = 0; i < mode->num_rates; i++) {
> > if ((sta->supp_rates & BIT(i)) &&
> > (mode->rates[i].flags & IEEE80211_RATE_SUPPORTED))
> > sta->txrate = i;
> > + break;
> > }
> Uh, what's the point of having a loop if you're gonna do that? ;)
How about "I think you forgot some curly braces"? Until Johannes
posted his patch (which you could have done just as easily) I had no
idea what you were talking about here...
John
--
John W. Linville
linville@xxxxxxxxxxxxx
-
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
