[FFmpeg-devel] [PATCH] Realmedia / RTSP (RDT)

Aurelien Jacobs aurel
Sun Dec 30 00:03:47 CET 2007


On Sat, 29 Dec 2007 15:43:16 -0500
"Ronald S. Bultje" <rsbultje at gmail.com> wrote:

> Hi,
> 
> On Dec 29, 2007 3:36 PM, Aurelien Jacobs <aurel at gnuage.org> wrote:
> 
> > Well, a demuxer, in the ffmpeg terminology, is defined by an
> > AVInputFormat.
> >
> 
> That's most likely a bug. Any file containing code required for
> functionality X is part of that functionality, whether that includes the
> AVInputFormat or not. rtp.c contains both RTP muxer *and* decoder code, and
> is thus required for both.

I will try to be clearer:
The codebase don't containt any AVInputFormat rtp_demuxer anywhere.
Thus allformats.c only do REGISTER_MUXER(RTP, rtp).
And ./ffmpeg -formats only list rtp encoder (no decoder).
So I don't know if rtp.c really contain any demuxing code (I know almost
nothing about rtp), but I'm sure there is no rtp demuxer available
in the user interface.
I wouldn't call this a bug, just a missing feature.

> I can actually remove my autotools patch, compile
> w/o the RTP muxer but with the demuxer and show that linking fails, but that
> shouldn't be needed, should it?

Linking won't fail, because there is no way to enable/build the
non-existing rtp demuxer.

> > Anyway, right now, rtp.c is only compiled when CONFIG_RTP_MUXER is
> > defined.
> >
> 
> Want to fix it? :-). rtp_h264.c also contains a parse_packet() (demuxer)
> implementation (the other rtp_*.c are mux-only).

There is nothing to fix as long as AVInputFormat rtp_demuxer is not
implemented.

Aurel




More information about the ffmpeg-devel mailing list