[FFmpeg-devel] [PATCH] avformat/avienc: Remove unneeded seekable tests
Ronald S. Bultje
rsbultje at gmail.com
Sun Mar 6 13:29:29 CET 2016
On Sun, Mar 6, 2016 at 6:51 AM, Mats Peterson <
matsp888-at-yahoo.com at ffmpeg.org> wrote:
> On 03/06/2016 12:09 PM, Michael Niedermayer wrote:
> Once again, why are these tests unneeded?
You shouldn't send the same message 5x within 2 seconds. Please be patient.
Why are they unneeded? Imagine this situation:
a = create_a();
int a = create_a();
Oddly, both codes do exactly the same thing, assuming they don't change
global state. In this case, avio_tell() shouldn't change global state, that
is, we're assuming that it will not cause bitstream corruption simply
because it's unseekable, which is a fair assumption.
On the positive side, with that assumption, code fragment #2 is much
simpler than #1, which is why the patch is net positive.
More information about the ffmpeg-devel