Attention is currently required from: pespin.

falconia has posted comments on this change. ( https://gerrit.osmocom.org/c/libosmocore/+/32961 )

Change subject: gsm0503_tch_hr_encode(): accept both TS101318 and RFC5993 payloads ......................................................................

Patch Set 1:

(2 comments)

File src/coding/gsm0503_coding.c:

https://gerrit.osmocom.org/c/libosmocore/+/32961/comment/ef3b095d_3df90f13 PS1, Line 1591: for (i = 0; i < 112; i++)

iiuc here we are removing 1 byte with this change.

Yes, I am changing the internal static function to take unprefixed 14-byte payloads as input, rather than prefixed 15-byte kind with the prefix byte content ignored.

https://gerrit.osmocom.org/c/libosmocore/+/32961/comment/9a370f79_71206d60 PS1, Line 2068: case GSM_HR_BYTES: /* TCH HR in "pure" form */

case GSM_HR_BYTES_RTP_RFC5993: /* TCH HR with RFC 5993 prefix */ […]

Good idea, I'll change the code like you suggest.