Attention is currently required from: neels.

pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-mgw/+/35434?usp=email )

Change subject: add fmtp.h ......................................................................

Patch Set 4:

(6 comments)

Commit Message:

https://gerrit.osmocom.org/c/osmo-mgw/+/35434/comment/587ddad9_c963b35b PS4, Line 9: Upcoming patches will add handling of arbitrary ftmp strings to fmtp strings

https://gerrit.osmocom.org/c/osmo-mgw/+/35434/comment/b9932791_b59f7fb2 PS4, Line 13: Add generic API for handling ftmp strings. The primary intended user is fmtp

Patchset:

PS4: Only minor stuff needs improving/fixing, feel free to merge it once you fix those.

File src/libosmo-mgcp-client/fmtp.c:

https://gerrit.osmocom.org/c/osmo-mgw/+/35434/comment/eb0de39f_5f52c1e6 PS4, Line 29: for (; fmtp && *fmtp && *fmtp != ';'; fmtp++); fmtp being nonull should be checked only once at start.

https://gerrit.osmocom.org/c/osmo-mgw/+/35434/comment/97ac2864_6c03457c PS4, Line 41: * if (osmo_sdp_fmtp_get_val(res, sizeof(res), fmtp_vals, "mode-set")) IIUC res contains "0,2,4,7" here, maybe worth saying that.

https://gerrit.osmocom.org/c/osmo-mgw/+/35434/comment/cad9b599_1fe5110a PS4, Line 89: int osmo_sdp_fmtp_get_int(const char *fmtp, const char *option_name, int default_value) did you check in SDP spec if int range is enough?