[FFmpeg-devel] [PATCH 3/7] Move away x86 specific code from rgb2rgb part 1

Alexander Strange astrange
Thu Jan 20 02:36:56 CET 2011


On Jan 19, 2011, at 7:32 PM, Ronald S. Bultje wrote:

> Hi,
> 
> On Wed, Jan 19, 2011 at 6:46 PM, Luca Barbato <lu_zero at gentoo.org> wrote:
>> @@ -133,6 +133,7 @@ DECLARE_ASM_CONST(8, uint64_t, blue_15mask)  = 0x0000001f0000001fULL;
>> 
>>  void sws_rgb2rgb_init(int flags)
>>  {
>> +
>>  #if HAVE_MMX2 || HAVE_AMD3DNOW || HAVE_MMX
>>     if (flags & SWS_CPU_CAPS_SSE2)
>>         rgb2rgb_init_SSE2();
> 
> Cosmetics. :-p.
> 
> [..]
>> diff --git a/libswscale/x86/rgb2rgb_template.c b/libswscale/x86/rgb2rgb_template.c
>> new file mode 100644
>> index 0000000..7c557fd
>> --- /dev/null
>> +++ b/libswscale/x86/rgb2rgb_template.c
>> @@ -0,0 +1,2944 @@
>> +/*
>> + * software RGB to RGB converter
>> + * pluralize by software PAL8 to RGB converter
>> + *              software YUV to YUV converter
>> + *              software YUV to RGB converter
> [..]
> 
> This should be a git cp, so history is maintained, right? Other than
> that the contents look good to me so I'd say go for this. Duplication
> of the boilerplate code is fine for now, it allows us to optimize
> further in the future.

git doesn't have explicit cp; it just stores the patches and everything is detected later when you run the history-examining commands.




More information about the ffmpeg-devel mailing list