[FFmpeg-devel] [PATCH] rtsp - alternate protocol

[Michael Niedermayer]

On Wed, Jan 02, 2008 at 10:38:18AM +0100, Luca Abeni wrote:
> Hi Michael,
> 
> first of all, thanks for the feedback.
> 
> Michael Niedermayer wrote:
> [...]
> > [...]
> >> -#ifdef CONFIG_RTSP_DEMUXER
> >>  static void opt_rtp_tcp(void)
> >>  {
> >>      /* only tcp protocol */
> >> -    rtsp_default_protocols = (1 << RTSP_PROTOCOL_RTP_TCP);
> >> +    rtsp_default_protocols = AVTRANSPORT_TCP;
> >>  }
> >> -#endif
> > 
> > this would be unneeded if ffplay would use AVOptions, but thats seperate
> > and belongs in some future patch
> 
> I did not know much about AVOptions... I am looking at them right now, 
> and I'll update the patch to use AVOptions.
> 
> 
> > ... if you want to volunteer to implement it :)
> 
> I see that the whole ffplay.c has to be converted, right? I'll have a 
> look, and see if I can find some time for this conversion.

theres no need to convert all at once, ffmpeg.c also still has tons of
old style options
whats nice if one would add avoption support for ffplay.c would be that
many options would suddenly become available ...


[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

I hate to see young programmers poisoned by the kind of thinking
Ulrich Drepper puts forward since it is simply too narrow -- Roman Shaposhnik
-------------- 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/20080102/39125a2a/attachment.pgp>