I've never before used the codec_* functions that have recently been
added to textops module - but today they helped me to solve a really
nasty problem ;-)
While playing around with those functions a few possible improvements
came to my mind:
a) Make codec_exists/codec_delete case_insensitive
I didn't intensively test it, but it seems that thosw functions are
currently case sensitive, "speex" for example didn't match "Speex"
b) Kill related fmtp-lines
If I remove a codec, it's payload type number is removed from media
line, and related rtpmap-entries are trashed. fmtp-lines however will
remain, please remove them too!
Writing a codec_exists/codec_delete for G726-[16|24|30|40] or something
similar is nasty, what about supporting something like "G726-*" - or
just use regular expressions? The latter would also provide case
Log in to post a comment.