Wireshark-dev: Re: [Wireshark-dev] [Wireshark-commits] rev 53676: / /trunk/codecs/G722/: G722de

Date Prev · Date Next · Thread Prev · Thread Next
From: Joerg Mayer <jmayer@xxxxxxxxx>
Date: Sat, 30 Nov 2013 22:32:10 +0100
On Sat, Nov 30, 2013 at 03:42:14PM +0000, mmann@xxxxxxxxxxxxx wrote:
> http://anonsvn.wireshark.org/viewvc/viewvc.cgi?view=rev&revision=53676
> 
> User: mmann
> Date: 2013/11/30 03:42 PM
> 
> Log:
>  Add G.722, G.726 and SBC codecs.  G.722 and G.726 are from bug 5619 (https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=5619) and SBC is from bug 7893 (https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=7893).
>  
>  Codecs are added, but (intentionally) not hooked to the RTP player as a "more generic architecture" is desired.  There are some discussions in bug 7893 on how to do this.  One thing to add would be how to handle codecs that may not be supported on all platforms.  Should the codec not be "registered" at all (with a #define over the whole module) or should it's register functions be stubbed (with a #define in each function that requires a non-supported library)

The plugins should only be built in case the necessary libraries are available
(just like we do not build wireshark in case we do not find gtk2 or gtk3).

Ciao
     Jörg
-- 
Joerg Mayer                                           <jmayer@xxxxxxxxx>
We are stuck with technology when what we really want is just stuff that
works. Some say that should read Microsoft instead of technology.