[FFmpeg-devel] [PATCH 2/9] Replace AVERROR_UNKNOWN by AVERROR(EINVAL)

Michael Niedermayer michaelni
Thu Jul 19 18:01:46 CEST 2007


Hi

On Thu, Jul 19, 2007 at 05:53:32PM +0200, Panagiotis Issaris wrote:
> Hi
> 
> Op 19-jul-07, om 17:43 heeft Michael Niedermayer het volgende  
> geschreven:
> 
> > On Thu, Jul 19, 2007 at 02:55:17PM +0200, Panagiotis Issaris wrote:
> >> The attached patch replaces the only occurrance of AVERROR_UNKNOWN  
> >> with
> >> AVERROR(EINVAL).
> >
> > AVERROR_UNKNOWN "unknown error" !=
> > EINVAL "Invalid argument"
> 
> Currently, in the ffmpeg sourcecode it is :)

i know but changing all AVERROR_UNKNOWN to EINVAL is wrong as it
discards information about the true meaning of the code
they should rather individually and manually be changed to the correct
E*

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Many that live deserve death. And some that die deserve life. Can you give
it to them? Then do not be too eager to deal out death in judgement. For
even the very wise cannot see all ends. -- Gandalf
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20070719/6443fcaf/attachment.pgp>



More information about the ffmpeg-devel mailing list