On Mon, Oct 25, 2010 at 11:58 AM, Ben Greear <greearb@xxxxxxxxxxxxxxx> wrote: > On 10/25/2010 12:16 AM, Mohammed Shafi Shajakhan wrote: >> - is_cw40 = sta->ht_cap.cap& IEEE80211_HT_CAP_SUP_WIDTH_20_40; >> + is_cw40 = (sta->ht_cap.cap& IEEE80211_HT_CAP_SUP_WIDTH_20_40) ? >> + true : false; > > Maybe use: > is_cw40 = !!(sta->ht_cap.cap& IEEE80211_HT_CAP_SUP_WIDTH_20_40); > > Seems a bit simpler to me... Agreed, !! is idiomatic. -- Bob Copeland %% www.bobcopeland.com -- 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
