Attention is currently required from: neels.

Patch set 1:Code-Review +1

View Change

2 comments:

• Commit Message:

  • Patch Set #1, Line 18: Related: SYS#5130

    Fixes: ae98b97382285420ba81549bc874b9fea5e7daa9

• File src/libmsc/msc_a.c:

  • Patch Set #1, Line 165: if (conn_accepted) {

    Can we please try refactoring this function? I see "if (conn_accepted)" spread like in 4 different places now.
    Without knowing this, code, looks like stuff could be moved around to really simplify the logic here.
    That would have probably also helped back in time when the bug was introduced.

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