[FFmpeg-devel] [PATCH]lavf/movenc: Put correct display aspect ratio in ARES atom

[Carl Eugen Hoyos]

2016-09-20 18:47 GMT+02:00 Paul B Mahol <onemda at gmail.com>:
> On 9/20/16, Carl Eugen Hoyos <ceffmpeg at gmail.com> wrote:
>> 2016-09-20 17:42 GMT+02:00 Paul B Mahol <onemda at gmail.com>:
>>> On 9/20/16, Carl Eugen Hoyos <ceffmpeg at gmail.com> wrote:
>>>> 2016-09-20 17:26 GMT+02:00 Paul B Mahol <onemda at gmail.com>:
>>>>> On 9/20/16, Carl Eugen Hoyos <ceffmpeg at gmail.com> wrote:
>>>>>> 2016-09-19 13:28 GMT+02:00 Carl Eugen Hoyos <cehoyos at ag.or.at>:
>>>>>>> Hi!
>>>>>>>
>>>>>>> We read a display aspect ratio from ARES atom because of several
>>>>>>> user-provided samples, I believe it makes sense to also write the
>>>>>>> width value into the ares atom depending on the aspect ratio.
>>>>>>
>>>>>> Ping.
>>>>>
>>>>> What about height?
>>>>
>>>> I thought height is already set correctly, is that wrong?
>>>
>>> So, height is not rescaled, but width is?
>>
>> It is rescaled for interlaced video iirc.
>>
>>> This does not make sense.
>>
>> What do you suggest?
>
> Please stop sending incorrect patches.

Please be slightly more constructive:
How can I reproduce the issue you see with the patch I sent?

Carl Eugen