Re: [libav-devel] [PATCH] movenc: Don't wrap whole extradata atoms in glbl.

2011-10-14 Thread Tomas Härdin
On Wed, 2011-10-12 at 15:29 -0700, Alex Converse wrote: > On Fri, Sep 30, 2011 at 5:11 AM, Tomas Härdin > wrote: > > On Fri, 2011-09-30 at 13:44 +0200, Luca Barbato wrote: > >> On 9/30/11 10:59 AM, Benjamin Larsson wrote: > >> > > >> >> > >> >> At work we added an "int interlacing;" field to AVCo

Re: [libav-devel] [PATCH] movenc: Don't wrap whole extradata atoms in glbl.

2011-10-12 Thread Alex Converse
On Fri, Sep 30, 2011 at 5:11 AM, Tomas Härdin wrote: > On Fri, 2011-09-30 at 13:44 +0200, Luca Barbato wrote: >> On 9/30/11 10:59 AM, Benjamin Larsson wrote: >> > >> >> >> >> At work we added an "int interlacing;" field to AVCodecContext. It's >> >> only set by a few decoders though (no demuxer se

Re: [libav-devel] [PATCH] movenc: Don't wrap whole extradata atoms in glbl.

2011-09-30 Thread Luca Barbato
On 9/30/11 2:11 PM, Tomas Härdin wrote: On Fri, 2011-09-30 at 13:44 +0200, Luca Barbato wrote: On 9/30/11 10:59 AM, Benjamin Larsson wrote: At work we added an "int interlacing;" field to AVCodecContext. It's only set by a few decoders though (no demuxer sets it), and doesn't provide field o

Re: [libav-devel] [PATCH] movenc: Don't wrap whole extradata atoms in glbl.

2011-09-30 Thread Tomas Härdin
On Fri, 2011-09-30 at 13:44 +0200, Luca Barbato wrote: > On 9/30/11 10:59 AM, Benjamin Larsson wrote: > > > >> > >> At work we added an "int interlacing;" field to AVCodecContext. It's > >> only set by a few decoders though (no demuxer sets it), and doesn't > >> provide field order information. > >

Re: [libav-devel] [PATCH] movenc: Don't wrap whole extradata atoms in glbl.

2011-09-30 Thread Luca Barbato
On 9/30/11 10:59 AM, Benjamin Larsson wrote: At work we added an "int interlacing;" field to AVCodecContext. It's only set by a few decoders though (no demuxer sets it), and doesn't provide field order information. To support field order, "interlacing" could have the value of 0/1/2 (progressiv

Re: [libav-devel] [PATCH] movenc: Don't wrap whole extradata atoms in glbl.

2011-09-30 Thread Benjamin Larsson
> > At work we added an "int interlacing;" field to AVCodecContext. It's > only set by a few decoders though (no demuxer sets it), and doesn't > provide field order information. > To support field order, "interlacing" could have the value of 0/1/2 > (progressive, TFF, BFF) or a separate field cou

Re: [libav-devel] [PATCH] movenc: Don't wrap whole extradata atoms in glbl.

2011-09-30 Thread Tomas Härdin
On Fri, 2011-09-30 at 01:15 -0700, Alex Converse wrote: > On Fri, Sep 30, 2011 at 12:19 AM, Tomas Härdin > wrote: > > On Thu, 2011-09-29 at 13:44 -0700, Alex Converse wrote: > >> This should result in more idiomatic output that is less confusing to > >> marginal implementations. > >> --- > >> li

Re: [libav-devel] [PATCH] movenc: Don't wrap whole extradata atoms in glbl.

2011-09-30 Thread Alex Converse
On Fri, Sep 30, 2011 at 12:19 AM, Tomas Härdin wrote: > On Thu, 2011-09-29 at 13:44 -0700, Alex Converse wrote: >> This should result in more idiomatic output that is less confusing to >> marginal implementations. >> --- >>  libavformat/movenc.c |   16 ++-- >>  1 files changed, 14 inse

Re: [libav-devel] [PATCH] movenc: Don't wrap whole extradata atoms in glbl.

2011-09-30 Thread Tomas Härdin
On Thu, 2011-09-29 at 13:44 -0700, Alex Converse wrote: > This should result in more idiomatic output that is less confusing to > marginal implementations. > --- > libavformat/movenc.c | 16 ++-- > 1 files changed, 14 insertions(+), 2 deletions(-) > > diff --git a/libavformat/movenc

[libav-devel] [PATCH] movenc: Don't wrap whole extradata atoms in glbl.

2011-09-29 Thread Alex Converse
This should result in more idiomatic output that is less confusing to marginal implementations. --- libavformat/movenc.c | 16 ++-- 1 files changed, 14 insertions(+), 2 deletions(-) diff --git a/libavformat/movenc.c b/libavformat/movenc.c index b79bbe8..066dae0 100644 --- a/libavfor