Re: [libav-devel] [PATCH v2 1/5] g722: Split out g722_qmf_apply() function into g722dsp.c

2015-02-02 Thread Peter Meerwald
> > --- > > > > v2: > > license statement in g722dsp.c (Timothy) > > Not quite, you copy and pasted only part of the standard license > header. will address these and other nitpicks in v3 (which I plan to send out after a couple of days to allow further review) thanks, p. > > --- a/libavcode

Re: [libav-devel] [PATCH v2 1/5] g722: Split out g722_qmf_apply() function into g722dsp.c

2015-02-02 Thread Diego Biurrun
On Mon, Feb 02, 2015 at 11:49:26AM +0100, Peter Meerwald wrote: > Signed-off-by: Peter Meerwald > > --- > > v2: > license statement in g722dsp.c (Timothy) Not quite, you copy and pasted only part of the standard license header. > --- a/libavcodec/g722dec.c > +++ b/libavcodec/g722dec.c > @@ -12

[libav-devel] [PATCH v2 1/5] g722: Split out g722_qmf_apply() function into g722dsp.c

2015-02-02 Thread Peter Meerwald
Signed-off-by: Peter Meerwald --- v2: license statement in g722dsp.c (Timothy) turn doxygen comment into regular comment (Timothy) opening braces on separate line (Martin) --- libavcodec/Makefile | 4 ++-- libavcodec/g722.c| 22 -- libavcodec/g722.h| 5 +++-- liba