[FFmpeg-devel] [PATCH] swscale: add generic yuv2yuv wrapper function

Ramiro Polla ramiro.polla
Sat Sep 11 07:15:39 CEST 2010


This is a patch series that cleans the yuv2yuv functions inside
rgb2rgb.c a bit (and removes their global function pointers).

Now I'm stuck in a bikeshed issue. There is one rgb2yuv function in
there (rgb24toyv12). So "yuv2yuv_func" and "Yuv2YuvFunc" don't sound
too great anymore. How should it be named?

I also plan on having another function pointer in the context to
simplify rgb2rgb (they'll take another set of parameters though). I'm
thinking of adding a "void *priv_data" to the context so that at some
point later on it will be possible to split the fields in the context
and only allocate/initialize them depending on the scaler chosen.
Would this be a good place to put the function pointers?
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0006-swscale-add-generic-yuv2yuv-wrapper-function-curre.patch
Type: application/octet-stream
Size: 4342 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20100911/582b2b4a/attachment.obj>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0008-swscale-move-some-wrapper-functions-to-rgb2rgb.patch
Type: application/octet-stream
Size: 10449 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20100911/582b2b4a/attachment-0001.obj>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0009-rgb2rgb-rename-some-variables-to-simplify-next-patc.patch
Type: application/octet-stream
Size: 13060 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20100911/582b2b4a/attachment-0002.obj>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0010-yuv2yuv-make-a-standard-interface-for-yuv2yuv-funct.patch
Type: application/octet-stream
Size: 28270 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20100911/582b2b4a/attachment-0003.obj>



More information about the ffmpeg-devel mailing list