On 01/08/14 01:01, Diego Biurrun wrote:
> This is how all the other init functions operate.
> ---
> 
> Now without embarassing typo ...
> 
>  libavcodec/ppc/idctdsp.c | 17 +++++++++--------
>  1 file changed, 9 insertions(+), 8 deletions(-)
> 
> diff --git a/libavcodec/ppc/idctdsp.c b/libavcodec/ppc/idctdsp.c
> index e1dde82..17f7dbb 100644
> --- a/libavcodec/ppc/idctdsp.c
> +++ b/libavcodec/ppc/idctdsp.c
> @@ -231,14 +231,15 @@ av_cold void ff_idctdsp_init_ppc(IDCTDSPContext *c, 
> AVCodecContext *avctx,
>                                   unsigned high_bit_depth)
>  {
>  #if HAVE_ALTIVEC
> -    if (PPC_ALTIVEC(av_get_cpu_flags())) {
> -        if (!high_bit_depth) {
> -            if ((avctx->idct_algo == FF_IDCT_AUTO) ||
> -                (avctx->idct_algo == FF_IDCT_ALTIVEC)) {
> -                c->idct_add  = idct_add_altivec;
> -                c->idct_put  = idct_put_altivec;
> -                c->perm_type = FF_IDCT_PERM_TRANSPOSE;
> -            }
> +    if (!PPC_ALTIVEC(av_get_cpu_flags()))
> +        return;
> +
> +    if (!high_bit_depth) {
> +        if ((avctx->idct_algo == FF_IDCT_AUTO) ||
> +            (avctx->idct_algo == FF_IDCT_ALTIVEC)) {
> +            c->idct_add  = idct_add_altivec;
> +            c->idct_put  = idct_put_altivec;
> +            c->perm_type = FF_IDCT_PERM_TRANSPOSE;
>          }
>      }
>  #endif /* HAVE_ALTIVEC */
> 

Fine for me, nag me once I claim to have spare time to add some more
altivec code.

lu

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

Reply via email to