[FFmpeg-devel] Realmedia patch
Wed Sep 3 22:09:47 CEST 2008
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 :)
More information about the ffmpeg-devel