[FFmpeg-devel] [PATCH 1/7] Thin rgb2rgb a little part 1 of many

Måns Rullgård mans
Thu Jan 20 01:21:41 CET 2011


Diego Biurrun <diego at biurrun.de> writes:

> On Thu, Jan 20, 2011 at 12:46:20AM +0100, Luca Barbato wrote:
>> Move the function pointers in a separate file.
>> It will removed once the pointers will be factored in a more structured
>> form
>> 
>> --- /dev/null
>> +++ b/libswscale/swscale_functions.c
>> @@ -0,0 +1,67 @@
>> +#include <inttypes.h>
>
> stdint.h is enough and please add the standard license header.
>
>> +void (*yv12toyuy2)(const uint8_t *ysrc, const uint8_t *usrc, const uint8_t *vsrc, uint8_t *dst,
>> +                   long width, long height,
>> +                   long lumStride, long chromStride, long dstStride);
>
> Shorten lines while you're at it?

I got the impression this file was going away entirely before it's
done.  No need to spend time on cosmetics in that case.

-- 
M?ns Rullg?rd
mans at mansr.com



More information about the ffmpeg-devel mailing list