On 2012-01-11 09:20:08 +0100, Vladimir Pantelic wrote:
> Paul B Mahol wrote:
> >---
> > libswscale/rgb2rgb.c | 15 +++++++++++++++
> > libswscale/rgb2rgb.h | 1 +
> > libswscale/swscale_unscaled.c | 1 +
> > 3 files changed, 17 insertions(+), 0 deletions(-)
> >
> >diff --git a/libswscale/rgb2rgb.c b/libswscale/rgb2rgb.c
> >index 0f07e57..5458d26 100644
> >--- a/libswscale/rgb2rgb.c
> >+++ b/libswscale/rgb2rgb.c
> >@@ -282,6 +282,21 @@ void rgb15tobgr15(const uint8_t *src, uint8_t
> >*dst, int src_size)
> > }
> > }
> >
> >+void rgb12tobgr12(const uint8_t *src, uint8_t *dst, int src_size)
> >+{
> >+ uint16_t *dest = (uint16_t*)dst;
> >+ uint16_t *sour = (uint16_t*)src;
>
> looks a bit ugly to me, we don't want the code to go "sour", do we?
>
> >+ int i;
> >+ int num_pixels = src_size>> 1;
> >+
> >+ for (i = 0; i< num_pixels; i++) {
>
> give operators room to breathe and don't do double spaces:
don't be fooled by the misquoting of your email client
> "i < num_pixels"
exactly as in the original patch:
'+ for (i = 0; i < num_pixels; i++) {'
same below
Janne
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel
