On Tue, May 24, 2011 at 04:07:50PM -0400, Ronald S. Bultje wrote:
> ---
>  libswscale/x86/swscale_template.c |   58 ++++++++++++++++++++++++++----------
>  1 files changed, 42 insertions(+), 16 deletions(-)

The log message tells me what you are doing, but not why.

> --- a/libswscale/x86/swscale_template.c
> +++ b/libswscale/x86/swscale_template.c
> @@ -827,11 +827,10 @@
>  
>  
> -static inline void RENAME(yuv2yuvX)(SwsContext *c, const int16_t *lumFilter, 
> const int16_t **lumSrc, int lumFilterSize,
> +static inline void RENAME(yuv2yuvX_ar)(SwsContext *c, const int16_t 
> *lumFilter, const int16_t **lumSrc, int lumFilterSize,
>                                      const int16_t *chrFilter, const int16_t 
> **chrSrc, int chrFilterSize, const int16_t **alpSrc,
>                                      uint8_t *dest, uint8_t *uDest, uint8_t 
> *vDest, uint8_t *aDest, long dstW, long chrDstW)

Here and below: Please do not leave indentation messed up, possibly
even shorten those long lines.

> @@ -873,7 +875,16 @@ static inline void RENAME(yuv2yuv1)(SwsContext *c, const 
> int16_t *lumSrc, const
> +
> +static inline void RENAME(yuv2yuv1)(SwsContext *c, const int16_t *lumSrc, 
> const int16_t *chrSrc, const int16_t *alpSrc,
> +                                    uint8_t *dest, uint8_t *uDest, uint8_t 
> *vDest, uint8_t *aDest, long dstW, long chrDstW)
> +{
> +    long p= 4;
> +    const uint8_t *src[4]= {alpSrc + dstW, lumSrc + dstW, chrSrc + chrDstW, 
> chrSrc + VOFW + chrDstW};
> +    uint8_t *dst[4]= {aDest, dest, uDest, vDest};
> +    x86_reg counter[4]= {dstW, dstW, chrDstW, chrDstW};

What person does that coding style remind me of? ;)

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

Reply via email to