> +             if (ieee80211_iface_work_handle_pkt_type(skb, sdata) &&
> +                 ieee80211_iface_work_handle_frame_control(skb, sdata)) {
> +                     ieee80211_iface_work_handle_vif_type(skb, sdata);
>               }
> 
This ends up being pretty strange IMHO, using boolean short-circuit
evaluation to avoid doing the next thing...

Not sure how to better do this though.

(You also don't need braces here for a single-statement if branch
contents)

johannes
--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to