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

Ronald S. Bultje rsbultje
Thu Jan 20 01:26:14 CET 2011

Hi,

2011/1/19 M?ns Rullg?rd :

Diego Biurrun 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/swscalefunctions.c @@ -0,0 +1,67 @@ +#include <inttypes.h> stdint.h is enough and please add the standard license header. +void (*yv12toyuy2)(const uint8t *ysrc, const uint8t *usrc, const uint8t *vsrc, uint8t *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.

I agree for this particular case. If this is going away, I think this is fine.

Ronald

More information about the ffmpeg-devel mailing list