[FFmpeg-devel] [PATCH] h264 luma interpolation 8x8 for altivec

Guillaume POIRIER poirierg
Tue Jun 26 20:53:24 CEST 2007


Hi,

On 6/26/07, Mauricio Alvarez <lokifo at gmail.com> wrote:
> On 6/26/07, Guillaume Poirier <gpoirier at mplayerhq.hu> wrote:
> >
> > Hi,
> >
> > Mauricio Alvarez wrote:
> >
> > > OK, I divided the patch in different parts.
> > >
> > > part1: cosmetics: replace all the variable definitions in h264_altivec.c
> > > and
> > > h264_template_altivec.c with the data types defined in types_altivec.h
> > > including the use of the LOAD_ZERO and the corresponding zero_xxxxv
> > > definitions.
> >
> >
> > That part is OK to me, except that you don't keep the end of the lines
> > ('\') of macros properly aligned.
> >
>
> Fixed, this is a cosmetic patch, and such kind of details should be OK.

You missed a few. Attached patch should have them all these issues fixed.

Luca, are you OK with this patch?

Guillaume
-- 
Subversion has been the most pointless project ever started
  -- Linus Torvalds
-------------- next part --------------
A non-text attachment was scrubbed...
Name: part1_h264_altivec_data_types.2.diff
Type: application/octet-stream
Size: 50686 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20070626/3069be73/attachment.obj>



More information about the ffmpeg-devel mailing list