On Mon, Jun 2, 2008 at 7:54 PM, Johannes Berg <johannes@xxxxxxxxxxxxxxxx> wrote: > On Mon, 2008-06-02 at 18:44 +0300, Tomas Winkler wrote: >> This patch removes doubly defined sband variable > >> if ((info->flags & IEEE80211_TX_CTL_USE_RTS_CTS) || >> (info->flags & IEEE80211_TX_CTL_USE_CTS_PROTECT)) { >> - struct ieee80211_supported_band *sband; >> struct ieee80211_rate *rate; >> s8 baserate = -1; >> int idx; > > I guess we can remove the double assignment too since it's the same as > already done. I must have added that when doing the mechanical tx rate > idx changes. :/ > Good point. I also did only mechanical fix for the compiler warning ;) Tomas -- 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