Attention is currently required from: pespin.
dexter has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-mgw/+/31214 )
Change subject: mgcp_sdp: add fmtp string to define HR GSM RTP format ......................................................................
Patch Set 6:
(2 comments)
File src/libosmo-mgcp-client/mgcp_client.c:
https://gerrit.osmocom.org/c/osmo-mgw/+/31214/comment/480f6435_b922a9c8 PS5, Line 1254: pt = map_codec_to_pt(mgcp_msg->ptmap, mgcp_msg->ptmap_len, mgcp_msg->codecs[i]);
shouldn't this line be moved inside each case? […]
its probably better, yes.
File src/libosmo-mgcp/mgcp_network.c:
https://gerrit.osmocom.org/c/osmo-mgw/+/31214/comment/d64839b9_b8c98777 PS5, Line 704: if (target_fmt_present && target_fmt == MGCP_GSM_HR_FMT_TS101318)
This looks wrong. If it's the target, then the comment with the " =>" is wrong.
Its correct, but a bit confusing though. I have added some comments to make it clearer.
The function should convert every input into the opposite format. Unless the format is specified. So wehen we detect that the input is TS101813 and the target format is also TS101813, then we abort. If nothing is specified we go ahead and do the conversion blindly. If the target format is specified and it is RFC5993, which is the opposite then we also go ahead and do the conversion.