Attention is currently required from: fixeria.

Patch set 1:Code-Review +1

View Change

1 comment:

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

  • Patch Set #1, Line 135: struct l1sched_meas_set buf[24]; /* up to 24 (BUFMAX) entries */

    Because `BUFMAX` is defined in a local header of osmo-bts-trx, while this header is in the common di […]

    As you see, buf if this 24 is directly related to BUFMAX then I really think it makes sense to have some define used here too, even if that means moving the defines to somewhere under common/.

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