On Sun, Sep 23, 2012 at 07:00:18PM +0200, Jordi Ortiz wrote:
> 
> --- /dev/null
> +++ b/libavcodec/dirac_dwt.c
> @@ -0,0 +1,571 @@
> +#include "dirac_dwt.h"

license boilerplate

> +static av_always_inline void horizontal_compose_haari(IDWTELEM *b,
> +                                                      IDWTELEM *temp, int w,
> +                                                      int shift)
> +{
> +}
> +
> +static void horizontal_compose_haar0i(IDWTELEM *b, IDWTELEM *temp, int w)
> +{
> +    horizontal_compose_haari(b, temp, w, 0);
> +}
> +
> +static void horizontal_compose_haar1i(IDWTELEM *b, IDWTELEM *temp, int w)
> +{
> +    horizontal_compose_haari(b, temp, w, 1);
> +}

Will that inlining strategy work as expected?  I forget...

> --- /dev/null
> +++ b/libavcodec/dirac_dwt.h
> @@ -0,0 +1,74 @@
> +#include "dwt.h"

license boilerplate

> --- a/libavcodec/dwt.h
> +++ b/libavcodec/dwt.h
> @@ -26,7 +26,12 @@
>  typedef int DWTELEM;
>  typedef short IDWTELEM;
>  
> +#define MAX_DWT_SUPPORT 8
> +#define MAX_DECOMPOSITIONS 8

nit: align the numbers

Diego
_______________________________________________
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to