[FFmpeg-devel] [PATCH] fix sdp(file) vs rtsp dep

Luca Barbato lu_zero
Sat Oct 24 14:09:58 CEST 2009


Luca Abeni wrote:
> Luca Barbato wrote:
>> Luca Abeni wrote:
>>> I think these dependencies are correct as they are now: the RTSP demuxer
>>> depends on the SDP demuxer, and not vice-versa (or, if you prefer, the
>>> RTSP demuxer is a superset of the SDP demuxer). From the practical point
>>> of view, I do not think this makes any difference, but from the
>>> theoretical point of view I think it's better to indicate the correct
>>> dependency here.
>>
>> It's the other way round.
>>
>> the current "sdp demuxer" is something that fetches a file, guesses it
>> as a stream description dump and then uses the rtsp functions to
>> actually "play" it.
> 
> Well, I do not know why the current code is written like this, but from
> the theoretical point of view an RTSP demuxer is just an SDP demuxer +
> some methods to fetch the SDP via RTSP, and some methods to seek, etc.

not completely

the sdp gives you a mapping, then you have the rtp demuxer to actually
get frames and samples and the rtsp demuxer/transport/nameit to handle
seeking, eos and such.

> The problem with the current code is IMHO just in the naming.

and the fact the rtsp/rtp model has a split between the demuxing
components that makes it different to name properly within ffmpeg ^^;

lu

-- 

Luca Barbato
Gentoo Council Member
Gentoo/linux Gentoo/PPC
http://dev.gentoo.org/~lu_zero




More information about the ffmpeg-devel mailing list