On Sun, Apr 28, 2013 at 04:30:45PM -0400, Derek Buitenhuis wrote:
> 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"?

The latter; it seemed like a half decent place to move the functions to.

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

Reply via email to