Attention is currently required from: fixeria, jolly.

View Change

2 comments:

• Patchset:

  • Patch Set #3:

    To me it's even clearer now that these structs and related APIs are not really needed. I mean, it's just a buffer plus a related buffer len. We have/use tons of those all around, imagine we had to add structs for all types of buffer+len, it makes no sense imho.

    Letting others provide feedback here.

• File include/osmocom/gsm/mncc.h:

  • Patch Set #3, Line 37: int len;

    this should be unsigned. And probably uint8_t.

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