On Mon, Apr 11, 2011 at 11:07:37AM +0200, Luca Barbato wrote:
> ---
>  libswscale/rgb2rgb.c          |    9 +-
>  libswscale/rgb2rgb_template.c |  174 ++++++++++++++++++++--------------------
>  2 files changed, 91 insertions(+), 92 deletions(-)

I'd be happy to see you explain the reason for this change in the log
message.

> --- a/libswscale/rgb2rgb.c
> +++ b/libswscale/rgb2rgb.c
> @@ -248,7 +248,7 @@ static inline void RENAME(rgb24to15)(const uint8_t *src, 
> uint8_t *dst, long src_
> -static inline void RENAME(rgb15tobgr24)(const uint8_t *src, uint8_t *dst, 
> long src_size)
> +static inline void rgb15tobgr24_c(const uint8_t *src, uint8_t *dst, long 
> src_size)

nit: In cases such as these those long lines could possibly be shortened.

> @@ -367,7 +367,7 @@ static inline void RENAME(rgb24tobgr24)(const uint8_t 
> *src, uint8_t *dst, long s
>  
> -static inline void RENAME(yuvPlanartoyuy2)(const uint8_t *ysrc, const 
> uint8_t *usrc, const uint8_t *vsrc, uint8_t *dst,
> +static inline void yuvPlanartoyuy2_c(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, long vertLumPerChroma)

Indentation is messed up - just merge the fixup into this commit IMO;
more instances below.

Diego
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to