[FFmpeg-devel] Realmedia patch

Luca Abeni lucabe72
Wed Sep 3 22:09:47 CEST 2008


Hi Ronald,

Ronald S. Bultje wrote:
[...]
>> I need to think a little bit more about this.
>> I know real rtsp is a mess, but there must be some way to support
>> it without filling all the rtsp code with hacks, layering violations,
>> and so on...
>>
>> I suspect part of the problem is due to the fact that you are reusing
>> the RTP infrastructure without making the RTSP code more generic, but
>> I am not sure about it... I need to re-read all the code, and to think
>> about it.
> 
> What if I export rdt_read_header() and call that from
> rtsp_read_packet()? At least it centralizes the parsing into rdt.c,
> but the ID numbering is still done inside rtsp.c. :-).

I do not know... I still had not enough time to think about this issue
(I had an half idea about using a single rdt_data structure for all the
streams, or about linking together the structures describing the various
streams, but I suspect this will not work).

Unfortunately, I have some strict deadlines to respect, and I have only
few spare time for ffmpeg. I do not think I'll be able to look at your
patches before next week, sorry... :(
If someone else can help reviewing these patches, he is welcome.
Otherwise, I hope to be able to return to these issues in a week.
In the meanwhile, I think you still have some approved patch to commit :)


				Luca




More information about the ffmpeg-devel mailing list