[FFmpeg-devel] [PATCH] Add an RTSP muxer

Diego Biurrun diego
Mon Jan 4 20:15:17 CET 2010


On Mon, Jan 04, 2010 at 01:14:43PM -0500, Ronald S. Bultje wrote:
> 
> On Mon, Jan 4, 2010 at 1:11 PM, Ronald S. Bultje <rsbultje at gmail.com> wrote:
> >
> > On Mon, Jan 4, 2010 at 11:49 AM, Michael Niedermayer <michaelni at gmx.at> wrote:
> >> On Mon, Jan 04, 2010 at 04:35:33PM +0100, Diego Biurrun wrote:
> >>> On Mon, Jan 04, 2010 at 05:11:13PM +0200, Martin Storsj? wrote:
> >>> > --- a/libavformat/rtsp.c
> >>> > +++ b/libavformat/rtsp.c
> >>> > @@ -44,7 +44,7 @@
> >>> >
> >>> > -#if CONFIG_RTSP_DEMUXER
> >>> > +#if CONFIG_RTSP_DEMUXER || CONFIG_RTSP_MUXER
> >>> > ?static int tcp_read_packet(AVFormatContext *s, RTSPStream **prtsp_st,
> >>> > ? ? ? ? ? ? ? ? ? ? ? ? ? ? uint8_t *buf, int buf_size);
> >>> > ?static int rtsp_read_reply(AVFormatContext *s, RTSPMessageHeader *reply,
> >>>
> >>> All those #ifdefs make me think that it's time to split this file.
> >>
> >> And me they remind that they arent needed
> >> heres a patch to get rid of them
> >> luca, ok to apply below?
> >
> > See attached for better.
> 
> #2.

Luca gave me blanket approval for such changes some time ago.
If I got it, I cannot see how it could not be extended to a
networking comaintainer.

Diego



More information about the ffmpeg-devel mailing list