On 2013-04-27 8:20 PM, Diego Biurrun wrote:
> ---
>  libavcodec/x86/dsputil_mmx.c |   43 
> +++++++++++++++---------------------------
>  1 file changed, 15 insertions(+), 28 deletions(-)

[...]

>  #if HAVE_YASM
> +static void ff_avg_pixels16_mmxext(uint8_t *block, const uint8_t *pixels,
> +                                   int line_size, int h)
> +{
> +    ff_avg_pixels8_mmxext(block,     pixels,     line_size, h);
> +    ff_avg_pixels8_mmxext(block + 8, pixels + 8, line_size, h);
> +}
> +
> +
> +static void ff_put_pixels16_mmxext(uint8_t *block, const uint8_t *pixels,
> +                                   ptrdiff_t line_size, int h)
> +{
> +    ff_put_pixels8_mmxext(block,     pixels,     line_size, h);
> +    ff_put_pixels8_mmxext(block + 8, pixels + 8, line_size, h);
> +}

Any particular reason to put them here, or just "it's where HAVE_YASM starts"?

Barring anyone complaining about placement (not me), seems OK.

- Derek

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

Reply via email to