[FFmpeg-devel] [PATCH]lavf/mov: Read aspect ratio from ares atom for dnxhd

[Carl Eugen Hoyos]

2016-09-19 13:57 GMT+02:00 Paul B Mahol <onemda at gmail.com>:
> On 9/19/16, Carl Eugen Hoyos <ceffmpeg at gmail.com> wrote:
>> 2016-09-19 13:37 GMT+02:00 Paul B Mahol <onemda at gmail.com>:
>>> On 9/19/16, Carl Eugen Hoyos <cehoyos at ag.or.at> wrote:
>>>> Hi!
>>>>
>>>> Attached patch fixes the aspect ratio for the sample from
>>>> ticket #5325 / #2125.
>>>>
>>>> The fate test changes because the fate sample  was written
>>>> by FFmpeg with an incorrect aspect ratio.
>>>>
>>>> Please comment, Carl Eugen
>>>>
>>>
>>>> From 0553b0adfee87401854f0313dbcf386f2fb7ae68 Mon Sep 17 00:00:00 2001
>>>> From: Carl Eugen Hoyos <cehoyos at ag.or.at>
>>>> Date: Mon, 19 Sep 2016 13:30:06 +0200
>>>> Subject: [PATCH] lavf/mov: Read aspect ratio from ares atom for dnxhd.
>>>>
>>>> Fixes ticket #2125.
>>>> Fixes ticket #5325.
>>>
>>> Feel free to spread lies to logs, it that helps your ego.
>>
>> I don't think I can follow you, sorry.
>
> How you can claim that you fixed those two tickets?

Afaict, the file is not decoded correctly by current FFmpeg.
It looks better with the patch in this thread.
Do you disagree?

Carl Eugen