[FFmpeg-devel] [PATCH] libavformat/cache: don't treat 0 as EOF libavformat/aviobuf: don't treat 0 as EOF libavformat/avio: retry_transfer_wrapper: don't treat 0 as EOF

[James Almer]

On 10/16/2017 12:10 AM, Daniel Kučera wrote:
>> This changes:
>> ./ffmpeg -y -i 'concat:matrixbench_mpeg2.mpg|matrixbench_mpeg2.mpg'
> -vsync 0 -an file.avi
> 
> In case he doesn't have this sample:
> http://samples.ffmpeg.org/benchmark/testsuite1/matrixbench_mpeg2.mpg
> 
> Basically, with the patch applied the above command doesn't concatenate
> the input as requested.
> 
> 
> Is there a reason why this test is not included in fate?

Tests are added as regressions are found in some cases, and as features
are added in others. There's no concat protocol test in FATE currently,
which is obviously not ideal.

I see there's a cut version of this file in the fate samples suite,
mpeg2/matrixbench_mpeg2.lq1.mpg, which is enough to reproduce this
issue. A test could be created with it, but i'm not sure how to make a
working command line with it for fate that will not trip in some cases
depending on what's the path to the samples folder (Windows paths are
especially annoying in this regard).