[FFmpeg-devel] [PATCH 6/9] Replace AVERROR_NOFMT by AVERROR(EILSEQ)

Michael Niedermayer michaelni
Fri Jul 20 00:06:27 CEST 2007


Hi

On Thu, Jul 19, 2007 at 05:47:32PM +0200, Panagiotis Issaris wrote:
> Hi
> 
> Op 19-jul-07, om 17:39 heeft Michael Niedermayer het volgende  
> geschreven:
> 
> > Hi
> >
> > On Thu, Jul 19, 2007 at 02:55:21PM +0200, Panagiotis Issaris wrote:
> >> The attached patch replaces all occurrances of AVERROR_NOFMT with
> >> AVERROR(EILSEQ).
> >
> > AVERROR_NOFMT "unknown format" != EILSEQ "Invalid or incomplete  
> > multibyte or wide character"
> 
> You are right, but that's how it is currently defined in avcodec.h: 
> 2932. The Doxygen
> docs explain it as being "unknown format". It was added in commit 7965.

well ...
there are several possibly choices for E* for AVERROR_NOFMT

EILSEQ might be ok if the file is truly broken that is valid AVI with
negative width for example

if OTOH we just dont support a specific version or type then ENOTSUP seems
better or maybe EFTYPE, i dunno 

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Why not whip the teacher when the pupil misbehaves? -- Diogenes of Sinope
-------------- 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/20070720/68fc9ae4/attachment.pgp>



More information about the ffmpeg-devel mailing list