[libav-devel] [PATCH] mov: support the Color Parameter Atoms 'colr'

2014-10-06 Thread Vittorio Giovara
--- Simplified return check. Vittorio libavformat/mov.c | 66 +++ 1 file changed, 66 insertions(+) diff --git a/libavformat/mov.c b/libavformat/mov.c index 3734689..acd3406 100644 --- a/libavformat/mov.c +++ b/libavformat/mov.c @@ -887,6

Re: [libav-devel] [PATCH] mov: support the Color Parameter Atoms 'colr'

2014-10-06 Thread Luca Barbato
On 06/10/14 15:59, Vittorio Giovara wrote: --- Simplified return check. Vittorio libavformat/mov.c | 66 +++ 1 file changed, 66 insertions(+) Fine for me, might be nice to print the unexpected values though. lu

Re: [libav-devel] [PATCH] mov: support the Color Parameter Atoms 'colr'

2014-10-06 Thread Vittorio Giovara
On Mon, Oct 6, 2014 at 3:14 PM, Luca Barbato lu_z...@gentoo.org wrote: On 06/10/14 15:59, Vittorio Giovara wrote: --- Simplified return check. Vittorio libavformat/mov.c | 66 +++ 1 file changed, 66 insertions(+) Fine for me, might

Re: [libav-devel] [PATCH] mov: support the Color Parameter Atoms 'colr'

2014-10-06 Thread Luca Barbato
On 06/10/14 18:42, Vittorio Giovara wrote: On Mon, Oct 6, 2014 at 3:14 PM, Luca Barbato lu_z...@gentoo.org wrote: On 06/10/14 15:59, Vittorio Giovara wrote: --- Simplified return check. Vittorio libavformat/mov.c | 66 +++ 1 file

Re: [libav-devel] [PATCH] mov: support the Color Parameter Atoms 'colr'

2014-10-03 Thread Luca Barbato
On 03/10/14 01:19, Vittorio Giovara wrote: On Thu, Oct 2, 2014 at 8:06 PM, Luca Barbato lu_z...@gentoo.org wrote: On 02/10/14 20:53, Vittorio Giovara wrote: --- libavformat/mov.c | 80 +++ 1 file changed, 80 insertions(+) Do you have

Re: [libav-devel] [PATCH] mov: support the Color Parameter Atoms 'colr'

2014-10-03 Thread Anton Khirnov
Quoting Vittorio Giovara (2014-10-02 20:53:40) --- libavformat/mov.c | 80 +++ 1 file changed, 80 insertions(+) diff --git a/libavformat/mov.c b/libavformat/mov.c index 3734689..fd79e6f 100644 --- a/libavformat/mov.c +++

Re: [libav-devel] [PATCH] mov: support the Color Parameter Atoms 'colr'

2014-10-03 Thread Vittorio Giovara
On Fri, Oct 3, 2014 at 12:56 PM, Anton Khirnov an...@khirnov.net wrote: Quoting Vittorio Giovara (2014-10-02 20:53:40) --- libavformat/mov.c | 80 +++ 1 file changed, 80 insertions(+) diff --git a/libavformat/mov.c b/libavformat/mov.c

[libav-devel] [PATCH] mov: support the Color Parameter Atoms 'colr'

2014-10-02 Thread Vittorio Giovara
--- libavformat/mov.c | 80 +++ 1 file changed, 80 insertions(+) diff --git a/libavformat/mov.c b/libavformat/mov.c index 3734689..fd79e6f 100644 --- a/libavformat/mov.c +++ b/libavformat/mov.c @@ -887,6 +887,85 @@ static int

Re: [libav-devel] [PATCH] mov: support the Color Parameter Atoms 'colr'

2014-10-02 Thread Luca Barbato
On 02/10/14 20:53, Vittorio Giovara wrote: --- libavformat/mov.c | 80 +++ 1 file changed, 80 insertions(+) Do you have any sample? lu ___ libav-devel mailing list libav-devel@libav.org

Re: [libav-devel] [PATCH] mov: support the Color Parameter Atoms 'colr'

2014-10-02 Thread Vittorio Giovara
On Thu, Oct 2, 2014 at 8:06 PM, Luca Barbato lu_z...@gentoo.org wrote: On 02/10/14 20:53, Vittorio Giovara wrote: --- libavformat/mov.c | 80 +++ 1 file changed, 80 insertions(+) Do you have any sample? I used