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

[Diego Biurrun]

On Thu, Jan 03, 2008 at 12:27:12AM +0100, Aurelien Jacobs wrote:
> On Thu, 3 Jan 2008 00:00:28 +0100
> Diego Biurrun <diego at biurrun.de> wrote:
> 
> > On Wed, Jan 02, 2008 at 11:50:50PM +0100, Luca Abeni wrote:
> > >
> > > --- libavformat/Makefile.orig	2008-01-02 23:29:02.000000000 +0100
> > > +++ libavformat/Makefile	2008-01-02 23:41:51.000000000 +0100
> > > @@ -122,8 +122,7 @@
> > >  OBJS-$(CONFIG_ROQ_DEMUXER)               += idroq.o
> > >  OBJS-$(CONFIG_ROQ_MUXER)                 += raw.o
> > >  OBJS-$(CONFIG_RTP_MUXER)                 += rtp.o rtp_h264.o rtp_mpv.o rtp_aac.o
> > > -OBJS-$(CONFIG_RTSP_DEMUXER)              += rtsp.o
> > > -OBJS-$(CONFIG_SDP_DEMUXER)               += rtsp.o
> > > +OBJS-$(CONFIG_SDP_DEMUXER)               += rtsp.o rtp.o rtp_h264.o rtp_mpv.o rtp_aac.o
> > 
> > Umm, yes, your patch makes the RTSP demuxer depend on the SDP demuxer,
> > but still I think we should keep that line here.
> 
> I agree we should keep the RTSP line here.
> Moreover, this still seems to be a dependency mess...
> I feel there is a need for file splitting here (especially rtp.c).
> Maybe I will try to have a deeper look at it.

Go for it :)

Maybe we should create a list of things that still need to be
refactored.  It could come in handy.

Diego