On Mon, Feb 02, 2015 at 11:49:30AM +0100, Peter Meerwald wrote:
> Signed-off-by: Peter Meerwald <[email protected]>
> 
> ---
> 
> v2:
> drop unnecessary constants (yes, they were for as asm version of s_zero which 
> turned out not worthwhile) (Martin)
> fix NEON register clobbering, use d16 instead of d8 as per AAPCS, §5.1.2.1 
> (Martin)
> remove trailing whitespace (Martin)

Since you are (probably) going to send another revision anyway..

> --- /dev/null
> +++ b/libavcodec/arm/g722dsp_init_arm.c
> @@ -0,0 +1,35 @@
> +
> +#include <stdint.h>
> +
> +#include "libavutil/attributes.h"
> +#include "libavutil/arm/cpu.h"
> +#include "libavcodec/g722dsp.h"

nit: empty line between libavutil and libavcodec headers.

> --- a/libavcodec/g722dsp.c
> +++ b/libavcodec/g722dsp.c
> @@ -65,4 +65,7 @@ static void g722_apply_qmf(const int16_t *prev_samples, int 
> xout[2])
>  void ff_g722dsp_init(struct G722DSPContext *c)
>  {
>      c->apply_qmf = g722_apply_qmf;
> +    

trailing whitespace on the empty line

Diego
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to