[FFmpeg-devel] question about patch rtsp EPROTONOSUPPORT => AVERROR_INVALIDDATA
Tue Apr 1 15:47:47 CEST 2008
On Tue, Apr 01, 2008 at 10:16:25PM +1000, pross at xvid.org wrote:
> On Tue, Apr 01, 2008 at 12:35:43PM +0200, Michael Niedermayer wrote:
> > On Tue, Apr 01, 2008 at 09:15:32AM +0000, Johan Mellberg wrote:
> > > Hi,
> > >
> > > just a quick question: I currently manually apply the patch from http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/2008-March/044249.html.
> > >
> > > Any chance of that being applied to the trunk?
> > > If not, why?
> > i can only speak for myself:
> > Iam not RTSP maintainer :)
> There was an alternate suggestion to wrap EPROTONOSUPPORT with FF_NETWORK
> to make rtsp.c compile with Winsock. Enclosed is a patch that implements
> this _and_ provides the missing error message in cmdutils.c.
> IMHO, it would be better to drop EPROTONOSUPPORT altogether, because it is
> raised when the RTSP demuxer encouters an unknown/unsupported datatype. Many
> existing format demuxers use AVERROR_INVALIDDATA to indicate this form of
> error (that said RTSP isn't strictly a format).
The cmdutils change is unrelated to the rtsp one (=seperate patch / commit
Also the cmdutils change is ok. ... and i still dont maintain rtsp :)
Besides this, if you want a svn write account and agree to our "rules"/policy,
you can have one.
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
Asymptotically faster algorithms should always be preferred if you have
asymptotical amounts of data
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Size: 189 bytes
Desc: Digital signature
More information about the ffmpeg-devel