Attention is currently required from: pespin.

View Change

1 comment:

• File include/osmocom/msc/transaction.h:

  • Patch Set #1, Line 113: struct csd_filter csd;

    This doesn't sound right to me, since the "audio" codec is always CLEARMODE:8000 for CSD. The bearer services aren't audio codecs or part of it, they are a combination of data rates, sync/asnc, transparent/non-transparent and rate adaption.

    I think in theory union made sense here, but it doesn't work since we also need to set the audio codec to clearmode (the fake audio codec that was invented for CSD).

    I'm not saying it should be done that way, just raising concerns.

    Sure, and thanks for that!

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