On Sun, Apr 01, 2012 at 03:51:17PM -0400, Derek Buitenhuis wrote:
> From: Themaister <[email protected]>

Ugh, anonymous submission, try to find out who this is, please.

> --- a/libswscale/swscale_unscaled.c
> +++ b/libswscale/swscale_unscaled.c
> @@ -344,6 +344,107 @@ static int palToRgbWrapper(SwsContext *c, const uint8_t 
> *src[], int srcStride[],
>  
> +static int planarRgbToRgbWrapper(SwsContext *c, const uint8_t *src[],
> +                                 int srcStride[], int srcSliceY, int 
> srcSliceH,
> +                                 uint8_t *dst[], int dstStride[])
> +{
> +    int alpha_first = 0;
> +    if (c->srcFormat != PIX_FMT_GBRP) {
> +        av_log(c, AV_LOG_ERROR, "unsupported planar RGB conversion %s -> 
> %s\n",
> +              av_get_pix_fmt_name(c->srcFormat), 
> av_get_pix_fmt_name(c->dstFormat));

Indentation is off and long lines could be broken.

> +    switch (c->dstFormat) {
> +        case PIX_FMT_BGR24:

Indent case labels at the same depth as switch.

> +            gbr24ptopacked24((const uint8_t* []) {src[1], src[0], src[2]},
> +                             (int []) {srcStride[1], srcStride[0], 
> srcStride[2]},
> +                             dst[0] + srcSliceY * dstStride[0], dstStride[0],
> +                             srcSliceH, c->srcW);

space inside {}

> +        default:
> +            av_log(c, AV_LOG_ERROR,
> +                   "unsupported planar RGB conversion %s -> %s\n",
> +                    av_get_pix_fmt_name(c->srcFormat),
> +                    av_get_pix_fmt_name(c->dstFormat));

Indentation is off again.

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

Reply via email to