[FFmpeg-devel] question about patch rtsp EPROTONOSUPPORT => AVERROR_INVALIDDATA

[Michael Niedermayer]

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
or flames).
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
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20080401/5004134e/attachment.pgp>