[FFmpeg-devel] [PATCH] mov: replace AVERROR(EOF) with AVERROR_EOF.

Nicolas George nicolas.george at normalesup.org
Wed Dec 21 19:47:06 CET 2011


Le primidi 1er nivôse, an CCXX, Michael Niedermayer a écrit :
> and i think it was one file from
> http://www.google-melange.com/gci/task/view/google/gci2011/7209212

Indeed, it was "lastfile". Thanks. I had not realized this was so recent a
change, I just found it while grepping "AVERROR(E".

The patch changes:

[mov,mp4,m4a,3gp,3g2,mj2 @ 0x13b57a0] Could not find codec parameters (Audio: none, 0 channels)
/tmp/lastfile.mov: could not find codec parameters

into:

[mov,mp4,m4a,3gp,3g2,mj2 @ 0x13b57a0] error reading header: -1
/tmp/lastfile.mov: Operation not permitted

The latter is more correct, I believe.

The "openration not permitted" is the usual mistranslation of -1 as an error
code: mov is very sloppy about return values, but I do not know enough of
the format to start fixing it now.

I will push the change in a few moments.

Regards,

-- 
  Nicolas George
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20111221/17987d59/attachment.asc>


More information about the ffmpeg-devel mailing list