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

Tobias Rapp t.rapp at noa-archive.com
Fri Oct 21 12:58:49 EEST 2016


On 20.10.2016 09:36, Tobias Rapp wrote:
> 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.

The DNXHD probe test reference file has changed after the recent commits 
to mxfdec.c . Find updated patch attached.

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: 12862 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20161021/62fb241c/attachment.bin>


More information about the ffmpeg-devel mailing list