Attention is currently required from: jolly, pespin.

Patch set 5:Code-Review -1

View Change

2 comments:

• File src/host/layer23/include/osmocom/bb/mobile/gsm48_rr.h:

  • Patch Set #5, Line 65: struct gsm48_chan_desc ch_desc;

    did you check all users of this struct to make sure eveyrthing is updated? Is this field always pres […]

    Ack, having a complete Information Element within a header definition looks wrong.

• File src/host/layer23/src/mobile/gsm48_mm.c:

  • Patch Set #5, Line 871: struct gsm48_chan_desc *ch_desc

    `const`

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