On Mon, Nov 27, 2017 at 03:20:10PM -0500, Vittorio Giovara wrote:
> Signed-off-by: Vittorio Giovara
> ---
> libavformat/mov.c | 71
> +++
> 1 file changed, 71 insertions(+)
>
> diff --git a/libavformat/mov.c
>* On 11/27/2017 5:20 PM, James Almer wrote:*> Pointless duplicate atoms :/ At
>least these don't use fixed point
> values, so they are nicer.
indeed
> I assume no file will have both smdm and mdcv, or coll and clli, so
> reusing the MOVStreamContext fields should be ok, but maybe free the
>
> On 11/27/2017 5:20 PM, James Almer wrote:
> Where are these two defined? Because
> https://github.com/webmproject/vp9-dash/blob/master/VPCodecISOMediaFileFormatBinding.md
> describes coll and smdm, and those are already supported.
these are unrelated to vp9 (or mp4), they are the mov-only atoms
On 11/27/2017 5:20 PM, Vittorio Giovara wrote:
> Signed-off-by: Vittorio Giovara
> ---
> libavformat/mov.c | 71
> +++
> 1 file changed, 71 insertions(+)
>
> diff --git a/libavformat/mov.c b/libavformat/mov.c
>
Signed-off-by: Vittorio Giovara
---
libavformat/mov.c | 71 +++
1 file changed, 71 insertions(+)
diff --git a/libavformat/mov.c b/libavformat/mov.c
index 79023ef369..bb463017a3 100644
--- a/libavformat/mov.c
+++