[FFmpeg-devel] [PATCH/RFC] change function prototype of parse_sdp_a_line

Ronald S. Bultje rsbultje
Tue Nov 18 14:26:34 CET 2008

Hi Luca,

On Tue, Nov 18, 2008 at 3:08 AM, Luca Abeni <lucabe72 at email.it> wrote:
> I believe that the problem is (once again :) due to the fact that the
> original design of the RTSP client did not account for multiple qualities.
> So, every media is associated to one and only one stream... Now, you are
> changing this assumption. I think the best way to support multiple qualities
> would be to redesign this part of the RTSP client, changing the RTSPStream
> in a "Media descriptor" (or adding this "Media descriptor" to RTSPStream)
> or similar, that can contain a link to all the streams provided for a media.
> And the parse_sdp_a_line() method can receive a pointer to the
> "Media descriptor" as a parameter.

So, I have local patches (for MS-RTSP, but still) that export
RTSPState ("contains a link to all media descriptors") and RTSPStream
("descriptor of a single m= line") to rtp_internal.h (or maybe rtp.h,
I forgot). I could add an array of AVStreams into RTSPStream (similar
to the array in AVFormatContext), where the first member of the array
is the first stream this RTSPStream defines. Any more would indicate
RDT-style stream selection support. Is that what you're thinking of?

If you do it that way, a pointer to RTSPState or RTSPStream in the
parse_sdp_a_line() is sufficient to suit my needs. Oh, and FYI, for
MS-RTSP, I'd need a RTSPState at the very least (I need data from
"global" (pre - m= line) SDP data inside the DynamicProtocolHandler
callback...). :-).


More information about the ffmpeg-devel mailing list