#38 Improvements for Codec operations (textops)

trunk
closed
modules (91)
5
2009-09-17
2009-09-10
Thomas Gelf
No

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@gelf.net
web: http://thomas.gelf.net/

Discussion

  • Bogdan-Andrei Iancu

    • assigned_to: nobody --> andreidragus
     
  • Andrei Dragus

    Andrei Dragus - 2009-09-16

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

     
  • Andrei Dragus

    Andrei Dragus - 2009-09-17

    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.

     
  • Andrei Dragus

    Andrei Dragus - 2009-09-17
    • status: open --> closed
     

Get latest updates about Open Source Projects, Conferences and News.

Sign up for the SourceForge newsletter:





No, thanks