On Thu, Dec 8, 2011 at 10:25 AM, Johannes Berg
<johannes@xxxxxxxxxxxxxxxx> wrote:
>> +{
>> + return (seq_ctrl & cpu_to_le16(IEEE80211_SCTL_FRAG)) != 0;
>> +}
>> +
>
> Maybe you should move that close to ieee80211_has_morefrags() so it's
> easy to tell the difference? :-) (that doesn't use bool either, heh)
Since all ieee80211_is/has_* functions use int already I took the same
approach.
And ieee80211_has_morefrags operates on the frame control while
this operates on seq_ctrl. Hence, I placed it at the end but if you prefer
to have it closer to ieee80211_has_morefrags I can do that too :)
Helmut
--
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
