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

[Carl Eugen Hoyos]

2016-09-19 15:27 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 14:57 GMT+02:00 Paul B Mahol <onemda at gmail.com>:
>>
>>> title should mentioned what kind of aspect ratio this is about.
>>> As I thought it is about sar.
>>
>> I don't understand: FFmpeg only knows about sample aspect
>> ratio (also known as pixel aspect ratio), display aspect ratio is
>> only shown on the console for convenience.
>> Without this patch, sar is 1:1 and (I suspect) the sample is not
>> decoded correctly, with my patch sar is 4:3 and the sample
>> looks better.
>>
>> What do I miss?
>
> ARES atom does not store sar. And this patch sets dar for AVdn.

Which is converted to sar (the only thing libavcodec cares
about and the only thing FFmpeg exports) in utils.c.

Carl Eugen