Wireshark-bugs: [Wireshark-bugs] [Bug 4467] Add speex support to RTP analyzer
Date Index · Thread Index · Other Months · All Mailing Lists
Date Prev · Date Next · Thread Prev · Thread Next
From: bugzilla-daemon@xxxxxxxxxxxxx
Date: Mon, 8 Feb 2010 04:50:24 -0800 (PST)
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=4467

--- Comment #7 from Jaap Keuter <jaap.keuter@xxxxxxxxx> 2010-02-08 13:50:03 CET ---
(In reply to comment #6)
> I've retried to be sure with 3 speex tuples and with the patch it was able to
> show speex/8000 deltas. I guess, than speex/16000 and 32000 would be wrong.

And you applied the patch attached here? That's strange since
get_dyn_pt_clock_rate() works its way back to front through the list and would
match {"speex",       32000} first.

> I agree that this patch might be just a quick workaround and it would be better
> to use the SDP for the codec rate. Unfortunately I don't know well C
> programming and neither wireshark to do that.
> 
> off: I have found other codecs which should be worth adding, e.g. BV16/8000,
> BV32/16000, CELT/32000, CELT/44100, CELT/48000.

-- 
Configure bugmail: https://bugs.wireshark.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.