[FFmpeg-soc] RTP (de)packetizer registration [was: Re: G723.1 Frame Parser]

Martin Storsjö martin at martin.st
Mon Apr 5 19:38:20 CEST 2010


On Mon, 5 Apr 2010, Ronald S. Bultje wrote:

> On Mon, Apr 5, 2010 at 8:55 AM, Mohamed Naufal <naufal11 at gmail.com> wrote:
> > --- /dev/null
> > +++ b/libavformat/rtpdec_g723_1.h
> > @@ -0,0 +1,8 @@
> > +#ifndef AVFORMAT_RTPDEC_G723_1_H
> > +#define AVFORMAT_RTPDEC_G723_1_H
> > +
> > +#include "rtpdec.h"
> > +
> > +extern RTPDynamicProtocolHandler ff_g723_1_handler;
> > +
> > +#endif /* AVFORMAT_RTPDEC_G723_1_H */
> 
> Can we please fix this awful way of having to explicitely declare each
> RTP payload in a header and move to a system similar to codec/mux
> registration, and then get rid of this exponential increase in rtp*.h
> header files that do nothing more than declaring a dynamic protocol
> handler? If people are OK I'll cook up a patch (basically make a macro
> that does "extern ...; register();").

Absolutely, that'd get rid of a lot of redundant files.

// Martin


More information about the FFmpeg-soc mailing list