Attention is currently required from: dexter, fixeria, pespin.

View Change

3 comments:

• File include/osmo-bts/msg_utils.h:

  • Patch Set #1, Line 27: /* Accessor macros for control buffer words in RTP input path */

    Can you add a a bit more framing to clearly separate the 2 groups of cb fields? […]

    Done

  • Patch Set #1, Line 39: #define tch_ul_msg_bfi(x) ((x)->cb[0])

    I wonder whether this is a boolean, a flag, or an integer.

    New patch includes clarifying comments.

• File src/common/l1sap.c:

  • Patch Set #1, Line 2229: teh = 0xE0 | (tch_ul_msg_bfi(msg) << 1);

    and this kind of operation together with "I wonder whether this is a boolean, a flag, or an integer. […]

    Done

To view, visit change 41047. To unsubscribe, or for help writing mail filters, visit settings.