[FFmpeg-devel] [PATCH] SAP muxer/demuxer

[Luca Abeni]

Hi Martin,

On 10/07/2010 01:12 PM, Martin Storsj? wrote:
[...]
>>> And both the RTSP demuxer and muxer
>>> declare the SDP demuxer as one of its dependencies, so this way, there
>>> should be no linking error regardless of which of them is
>>> enabled/disabled.
>>
>> Ok, this works... But having the SDP demuxer as a dependency for the RTSP
>> muxer looks quite strange to me...
>
> Yeah, it's not too obvious. If you're interested in seeing how complicated
> it is, have a look at the other RFC thread I posted recently - it's
> possible, but it's not easy at all. :-)

I know... I had a similar (but less complete) patch locally. I lost interest
in it after the third time it broke :)

[...]
> So, reviews that still is required:
> - Opinion on the connect() stuff, Ronald, Luca B, someone else?
> - Splitting out the RTP chained muxer setup from the RTSP code, Ronald,
> Luca B?
>
> - Luca A, any opinion on how the demuxer is used? That is, currently it is
> used via a sap:// url that specifies the announcement ip to listen on, so
> for most IPv4 cases, one would use sap://sap.mcast.net

I possible (I think it's easy, but I might be missing some evil detail), I'd
allow to use a simple "sap://" URL (without any address specified), which
defaults to the multicast group generally used by SAP (224.2.127.254, I
think... I'd use the numeric IP, so that it works even if the name server
does not know sap.mcast.net)


				Luca