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

Benoit Fouet benoit.fouet
Tue Jun 26 20:18:52 CEST 2007


Mauricio Alvarez 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.
>
> Mauricio
> ------------------------------------------------------------------------
>
> Index: libavcodec/ppc/h264_template_altivec.c
> ===================================================================
> --- libavcodec/ppc/h264_template_altivec.c	(revision 9431)
> +++ libavcodec/ppc/h264_template_altivec.c	(working copy)
> @@ -1,4 +1,4 @@
> -/*
> +	/*
>   

oops ? :)

Ben




More information about the ffmpeg-devel mailing list