[FFmpeg-devel] [PATCH] avformat/matroskadec: retain error codes in matroska_resync() and matroska_read_packet()
george at nsup.org
Tue Aug 9 15:58:24 EEST 2016
Le tridi 23 thermidor, an CCXXIV, Michael Niedermayer a écrit :
> prior to this the "done" field was used to stop further work when
> EOF was reached, now this is removed.
> The patches commit message doesnt say anything why all reads for the
> done field are removed or what if any performance impact that has
> the patch even adds code setting the done variable even though all
> reads of the varable are removed
> This doesnt look right
If I read the code correctly, on the first round the result will be roughly
the same, with just a more accurate error code. The difference about the
done flag will appear if the application tries again to read a packet after
an error: without the check on done, it will try to resync again, which may
be a good thing in some cases.
But of course, changing it inadvertently is not a good thing, and neither
keeping dead code for setting an unused flag.
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 819 bytes
Desc: Digital signature
More information about the ffmpeg-devel