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

Luca Abeni lucabe72
Tue Jan 5 19:30:04 CET 2010


Hi Michael,

On 05/01/10 12:33, Michael Niedermayer wrote:
> On Tue, Jan 05, 2010 at 12:04:51PM +0100, Luca Abeni wrote:
> [...]
>>> Given this comparison, what's your opinion - internal RTP muxer interface
>>> or proper chained AVFormatContexts within the RTSP muxer?
>>
>> I still think the "chained muxers" approach is better, but maybe some
>> comments from Michael would help.
>> Of course, we have to implement chained muxers in a clean way, without
>> making the code too complex, and this might require some work... As I wrote
>> in the previous email, I am willing to help in this.
>
> The only comment i wanted to make was to point to the generic discussions
> there where about chained demuxers. That probably could be usefull in
> relation to doing this cleanly. A quick search about chained demuxers
> points me to
> "[PATCH] seperate parser from codec selection"

Thanks for the reference. I read this thread some time ago, and I think 
my suggestion is in line with your description... Anyway, I'll re-read 
this thread to check if it contains more details that can help here.

I think the big question here was if the "chained muxers" approach is 
better than the approach currently taken - for example - by the RTSP 
demuxer (directly invoking the RTP demuxing functions, instead of having 
an RTP demuxer), or not.


			Thanks,
				Luca



More information about the ffmpeg-devel mailing list