On Thu, Sep 19, 2013 at 2:11 PM, Andreas Eversberg <[email protected]> wrote:
> +       lc = &trx->ts[0].lchan[4].lapdm_ch;

I would recommend to move "trx->ts[0].lchan[4].lapdm_ch" to a macros
and document why do we refer to TS0, lchan 4 here (some references to
the would be helpful).

Otherwise this patch looks good to me from the code structure perspective.

-- 
Regards,
Alexander Chemeris.
CEO, Fairwaves LLC / ООО УмРадио
http://fairwaves.ru

Reply via email to