On Mon, Dec 23, 2013 at 8:05 AM, Fred Chou <[email protected]> wrote: > Hi, > > I was studying the Linux wireless subsystem code and noticed this code (in > ieee80211_rx_handlers): > > It first defines the macro: > > #define CALL_RXH(rxh) \ > do { \ > res = rxh(rx); \ > if (res != RX_CONTINUE) \ > goto rxh_next; \ > } while (0); > > Then the macro is used to call a series of functions: > > CALL_RXH(ieee80211_rx_h_check_more_data) > CALL_RXH(ieee80211_rx_h_uapsd_and_pspoll) > CALL_RXH(ieee80211_rx_h_sta_process) > CALL_RXH(ieee80211_rx_h_decrypt) > CALL_RXH(ieee80211_rx_h_defragment) > CALL_RXH(ieee80211_rx_h_michael_mic_verify) > > > My question is, why not just call the functions directly like: > > ieee80211_rx_h_check_more_data(rx); > ieee80211_rx_h_uapsd_and_pspoll(rx); > > ... > > Is it just for the sake of outlining the code for easy reading? Appreciate > if anyone can explain. Thanks!
Hi, As you can see above the macro does also error checking.I don't necessarily think that the code is easier to read, but it can save you some typing. thanks, Daniel. _______________________________________________ Kernelnewbies mailing list [email protected] http://lists.kernelnewbies.org/mailman/listinfo/kernelnewbies
