[FFmpeg-devel] [PATCH v2 3/3] fate: Add MXF D10/DV25 probe tests

Tobias Rapp t.rapp at noa-archive.com
Thu Oct 20 10:36:42 EEST 2016


On 19.10.2016 23:06, Michael Niedermayer wrote:
> On Wed, Oct 19, 2016 at 02:35:22PM +0200, Tobias Rapp wrote:
>> Signed-off-by: Tobias Rapp <t.rapp at noa-archive.com>
>> ---
>>  tests/fate/mxf.mak            | 19 ++++++++++++++++++-
>>  tests/ref/fate/mxf-probe-d10  |  3 +++
>>  tests/ref/fate/mxf-probe-dv25 |  4 ++++
>>  3 files changed, 25 insertions(+), 1 deletion(-)
>>  create mode 100644 tests/ref/fate/mxf-probe-d10
>>  create mode 100644 tests/ref/fate/mxf-probe-dv25
>
> Applying: fate: Add MXF D10/DV25 probe tests
> fatal: corrupt patch at line 60
> didnt check deeply but some lines looks quite long, maybe something in
> the mail chain didnt like that
>
> [...]

Took the chance to change ffprobe output format from "compact" to 
"default". The reference files are only <80 bytes larger but it avoids 
the long lines and improves reading future diffs (git blame).

Also added a DNXHD probe test using an existing fate sample so 
progressive field order is covered now, too.

Updated patch attached as a file.

Regards,
Tobias
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0003-fate-Add-MXF-D10-DNXHD-DV25-probe-tests.patch
Type: text/x-patch
Size: 10480 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20161020/fd381313/attachment.bin>


More information about the ffmpeg-devel mailing list