[FFmpeg-devel] [PATCH] lavf: Add SMJPEG demuxer.

Michael Niedermayer michaelni at gmx.at
Tue Dec 20 14:03:00 CET 2011


On Tue, Dec 20, 2011 at 11:50:28AM +0100, Reimar Döffinger wrote:
> On Tue, Dec 20, 2011 at 04:48:03AM +0000, Paul B Mahol wrote:
[...]
> > +    default:
> > +        ret = AVERROR(EAGAIN);
> 
> Why EAGAIN?? No normal demuxer should ever generate that one IMO.

EAGAIN is sometimes convenient in a demuxer instead of a loop or
goto retry

The theoretical advantage of EAGAIN is in case of a long stream of
unsupported packets. Where otherwise the demuxer may be locked for a
long time.

but as you mention it now, i do also realize that its semantically not
exactly correct to use EAGAIN for this.


[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

The real ebay dictionary, page 1
"Used only once"    - "Some unspecified defect prevented a second use"
"In good condition" - "Can be repaird by experienced expert"
"As is" - "You wouldnt want it even if you were payed for it, if you knew ..."
-------------- 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/20111220/305e7373/attachment.asc>


More information about the ffmpeg-devel mailing list