[FFmpeg-devel] [PATCH] Add protocol documentation on RTSP

[Ronald S. Bultje]

Hi,

On Sat, Oct 2, 2010 at 1:38 PM, Martin Storsj? <martin at martin.st> wrote:
> On Sat, 2 Oct 2010, Ronald S. Bultje wrote:
>
>> On Sat, Oct 2, 2010 at 1:12 PM, Martin Storsj? <martin at martin.st> wrote:
>> > Even though RTSP isn't a protocol handler but a demuxer/muxer, having
>> > documentation on it (including some examples) in protocols.texi may still
>> > be useful.
>> >
>> > Ronald, can you check that the references to Real-RTSP and MS-RTSP are
>> > right, or should it be formulated in some other way?
>>
>> Real is good to mention separately, I'd include MS in the main RTSP,
>> i.e. "RTSP (with data transferred over RTP; this is used by e.g. Apple
>> and Microsoft) as well as Real-RTSP (...", becuase MS is surprisingly
>> (although not completely) standards-compliant so it's good to give
>> them credit for that.
>
> Ok, updated.
>
>> > + at item http
>> > +Use HTTP tunneling as lower transport protocol
>> > + at end table
>>
>> Include a note that this is useful for passing proxies.
>
> Done.
>
>> Can you also include a note on stream selection (-as <num>, -vs <num>,
>> I believe, or pressing a/n while running in ffplay) if the server
>> supports multi-bitrate.
>
> Sure - I'm not all that familiar with that piece of code, but I wrote
> something up that shouldn't be too far from the truth :-)
>
>> Thanks for this patch, documentation is highly appreciated and
>> currently often missing!
>
> Thanks to Stefano for pioneering this piece of documentation in the first
> place. :-)
[..]

This version LGTM. I dind't check the grammar closely so please follow
Stefano's advice there (thanks for checking, Stefano!) and then feel
free to apply.

Ronald