Attention is currently required from: pespin.
2 comments:
Patchset:
looking at this patch again, it looks rather unclear why the codecs filter result ends up in assignment... it looks much cleaner in a later patch, where i must still investigate a minor odd change in MNCC behavior. I adjusted the commit log to explain...
I think i will wait until that follow up patch is ready before submitting a new patch set.
BTW, it is super complex to explain the legacy code and its transition to a new implementation, i'm beginning to think i should just have secretly merged that code bomb without anyone noticing, instead of kneading through all of these weird details ;)
File tests/msc_vlr/msc_vlr_test_call.err:
Patch Set #1, Line 320: MGW <--CRCX to RTP_TO_CN-- MSC: callref=0x80000001 codecs=VND.3GPP.IUFP#96
Is this change AMR->IUFP expected?
yes, i also find it curious... note that the surrounding logging indicates UTRAN, so I expect that previous code erratically used AMR in the first CRCX instead of the IUFP that we are actually (so far) feeding through to the CN. It is definitely correct to show IUFP here.
To view, visit change 30126. To unsubscribe, or for help writing mail filters, visit settings.