[OpenSIPS-Devel] [ opensips-Feature Requests-2856492 ] Improvements for Codec operations (textops)

SourceForge.net noreply at sourceforge.net
Thu Sep 17 11:42:36 CEST 2009


Feature Requests item #2856492, was opened at 2009-09-11 01:20
Message generated for change (Settings changed) made by andreidragus
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=1086413&aid=2856492&group_id=232389

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: modules
Group: trunk
>Status: Closed
Priority: 5
Private: No
Submitted By: Thomas Gelf (thomas_gelf)
Assigned to: Andrei Dragus (andreidragus)
Summary: Improvements for Codec operations (textops)

Initial Comment:
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!

c) Wildcards/Regexps?
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
insensitiveness.

Best regards,
Thomas Gelf

--
 mail: thomas at gelf.net
  web: http://thomas.gelf.net/


----------------------------------------------------------------------

Comment By: Andrei Dragus (andreidragus)
Date: 2009-09-17 12:42

Message:
Every function now has an equivalent function that takes a regular
expression as a parameter.
It has the same name, only followed by _re. 
Example: codec_exists_re("sp[a-z]*"). 

Please report if there are any problems.



----------------------------------------------------------------------

Comment By: Andrei Dragus (andreidragus)
Date: 2009-09-16 19:01

Message:
Points a) and b) have been solved. They are up on svn. Feel free to test. 
I am currently working on c). 


----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=1086413&aid=2856492&group_id=232389



More information about the Devel mailing list