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

Michael Niedermayer michaelni
Mon Jan 4 20:53:21 CET 2010


On Mon, Jan 04, 2010 at 01:14:43PM -0500, Ronald S. Bultje wrote:
> Hi,
> 
> On Mon, Jan 4, 2010 at 1:11 PM, Ronald S. Bultje <rsbultje at gmail.com> wrote:
> > Hi,
> >
> > 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.
> 
> Ronald

>  rtsp.c |  246 +++++++++++++++++++++++++++++++----------------------------------
>  1 file changed, 119 insertions(+), 127 deletions(-)
> c496cc1d23b29d364cfeccbf68d014c0fc5fe2dc  remove-fwd-decls-rtsp.patch

i realize now that moving functions around might break martins patchset
so iam not sure if this should be applied now or not

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Those who are too smart to engage in politics are punished by being
governed by those who are dumber. -- Plato 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20100104/d435d646/attachment.pgp>



More information about the ffmpeg-devel mailing list