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

Clément Bœsch u at pkh.me
Mon Sep 19 15:03:30 EEST 2016


On Mon, Sep 19, 2016 at 02:01:17PM +0200, Carl Eugen Hoyos wrote:
> 2016-09-19 14:00 GMT+02:00 Clément Bœsch <u at pkh.me>:
> > On Mon, Sep 19, 2016 at 01:37:10PM +0200, Paul B Mahol wrote:
> >> 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.
> >
> > Please don't use such aggressive statements. It might just be a ticket
> > number error here.
> 
> Why do you believe that there is an error?
> 

Because I'm assuming you made a mistake some way or another. It does look
like wording instead of ticket number.

> > From the code of conduct: "Do not assume malice for things that can be
> > attributed to incompetence. Even if it is malice, it’s rarely good to
> > start with that as initial assumption."
> 
> I thought this is not relevant anymore?
> 

Why do you believe that?

-- 
Clément B.


More information about the ffmpeg-devel mailing list