[FFmpeg-devel] [PATCH] add ENOENT support

François Revol revol
Sun May 13 12:05:47 CEST 2007


> > > AVERROR_* are *deprecated*.
> > > They are only kept for compatibility with apps.
> > > There is no reason to not use POSIX error codes directly with the 
> > > AV[UN]ERROR() macros, that's what they are for. The libraries 
> > > already 
> > > returns more errors than the listed ones.
> > > If you want to print an error just use strerror(AVUNERROR(e)).
> > 
> > all true, but why are there still AVERROR_* in the source then
> >  (not counting
> > the compatibility stuff in the header of course)?
> > 
> > having AVERROR_* in some code and POSIX errors in other is not good
> >  it leads
> > to people writing such patches with noone realizing that its wrong 
> > ...
> 
> Which is preferred? There are much more occurances of AVERROR_* than
> AVERROR(*)

AVERROR() is preferred. I didn't feel like changing all the others, but 
I suppose if I don't nobody will.

Fran?ois.




More information about the ffmpeg-devel mailing list