[FFmpeg-devel] [PATCH] lavf/isom: add "dvhe" fourCC for HEVC

Jan Ekström jeebjp at gmail.com
Thu Nov 8 09:25:05 EET 2018


On Thu, Nov 8, 2018, 03:38 Carl Eugen Hoyos <ceffmpeg at gmail.com wrote:

>
> Apart from "please see archives":
> Why can't we do it like the last decade and fix files that don't work
> instead of creating patches that apparently are impossible to test?
>
> Carl Eugen
>

We have something that:
a) has a specification (albeit by Dolby, which we all can have our opinions
on, but still a specification).
b) has the identifiers registered to itself by mpeg-4 ra

When looking into something that is specified, wouldn't you go through it
and enable what it mentions if you notice that there were actually more
things than you originally thought?! I'm pretty sure this way of going
through it has nothing special regarding it compared to the "last decade".

Generally picking up values from samples happens when there are no
specifications, or if there's a specific agenda from the side of FFmpeg to
prefer one identifier to the other. Or if someone just forgot to look at a
specification during initial implementation. Or the specification changed.

Jan


More information about the ffmpeg-devel mailing list