[FFmpeg-devel] Realmedia patch

Luca Abeni lucabe72
Wed Sep 3 09:07:31 CEST 2008


Hi Ronald,

Ronald S. Bultje wrote:
[...]
>>>> I can move this to ff_rdt_parse_packet() if you prefer it there
>>> Yes, this belongs to ff_rdt_parse_packet(), since it has nothing to
>>> do with RTSP...
> 
> OK so this is becoming problematic for me now. Here's why:
> - RTSPStream is private to rtsp.c
> - AVStream->priv_data is a RTSPStream
> - the only way to get to the DynamicProtocolHandler private data is
> through RTPDemuxContext->dynamic_protocol_context or RTSPStream
> - I can't get to my context data without rtsp.c data
[...]

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. (BTW, is it my impression, or rtp_check_and_send_back_rr() is
still called for RDT over UDP? And is rtp_parse_open() still called?)


				Luca




More information about the ffmpeg-devel mailing list