Attention is currently required from: osmith, dexter.

View Change

1 comment:

• File src/osmo-bsc/pcu_sock.c:

  • Patch Set #1, Line 552: if (pch->confirmed_imm_ass)

    I don't think that it is required to mention this, we do changes to PCUIF all the time. […]

    I disagree with this. I'm now more or less understanding the different changes you are doing to the protocol, but even for myself it's starting to be confusing the amount of changes. Imagine for others, or ourselves in a few months time. It will be difficult to understand which changes are part of which versions of the protocol. So yeah, in general I'd love seeing a bit more thoughts and clear explanations on what these changes are part of (eg v10->v11 changes).

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