[FFmpeg-devel] [PATCH 2/6] libopenjpegenc: Simplify initialization of sub_dx/y.

Michael Bradshaw mbradshaw at sorensonmedia.com
Fri Nov 18 22:56:25 CET 2011


On Thu, Nov 17, 2011 at 6:32 PM, Michael Niedermayer <michaelni at gmx.at> wrote:
> Signed-off-by: Michael Niedermayer <michaelni at gmx.at>
> ---
>  libavcodec/libopenjpegenc.c |   23 +++++------------------
>  1 files changed, 5 insertions(+), 18 deletions(-)
>
> diff --git a/libavcodec/libopenjpegenc.c b/libavcodec/libopenjpegenc.c
> index 1a92d60..31d87e7 100644
> --- a/libavcodec/libopenjpegenc.c
> +++ b/libavcodec/libopenjpegenc.c
> @@ -59,59 +59,46 @@ static opj_image_t *mj2_create_image(AVCodecContext *avctx, opj_cparameters_t *p
>     int numcomps = 0;
>     OPJ_COLOR_SPACE color_space = CLRSPC_UNKNOWN;
>
> +    sub_dx[0] = sub_dx[3] = 1;
> +    sub_dy[0] = sub_dy[3] = 1;
> +    sub_dx[1] = sub_dx[2] = 1<<av_pix_fmt_descriptors[avctx->pix_fmt].log2_chroma_w;
> +    sub_dy[1] = sub_dy[2] = 1<<av_pix_fmt_descriptors[avctx->pix_fmt].log2_chroma_h;
> +
>     switch (avctx->pix_fmt) {
>     case PIX_FMT_GRAY8:
>         color_space = CLRSPC_GRAY;
>         numcomps = 1;
>         bpp = 8;
> -        sub_dx[0] = 1;
> -        sub_dy[0] = 1;
>         break;
>     case PIX_FMT_RGB24:
>         color_space = CLRSPC_SRGB;
>         numcomps = 3;
>         bpp = 24;
> -        sub_dx[0] = sub_dx[1] = sub_dx[2] = 1;
> -        sub_dy[0] = sub_dy[1] = sub_dy[2] = 1;
>         break;
>     case PIX_FMT_RGBA:
>         color_space = CLRSPC_SRGB;
>         numcomps = 4;
>         bpp = 32;
> -        sub_dx[0] = sub_dx[1] = sub_dx[2] = sub_dx[3] = 1;
> -        sub_dy[0] = sub_dy[1] = sub_dy[2] = sub_dy[3] = 1;
>         break;
>     case PIX_FMT_YUV420P:
>         color_space = CLRSPC_SYCC;
>         numcomps = 3;
>         bpp = 12;
> -        sub_dx[0] = 1;
> -        sub_dx[1] = sub_dx[2] = 2;
> -        sub_dy[0] = 1;
> -        sub_dy[1] = sub_dy[2] = 2;
>         break;
>     case PIX_FMT_YUV422P:
>         color_space = CLRSPC_SYCC;
>         numcomps = 3;
>         bpp = 16;
> -        sub_dx[0] = 1;
> -        sub_dx[1] = sub_dx[2] = 2;
> -        sub_dy[0] = sub_dy[1] = sub_dy[2] = 1;
>         break;
>     case PIX_FMT_YUV440P:
>         color_space = CLRSPC_SYCC;
>         numcomps = 3;
>         bpp = 16;
> -        sub_dx[0] = sub_dx[1] = sub_dx[2] = 1;
> -        sub_dy[0] = 1;
> -        sub_dy[1] = sub_dy[2] = 2;
>             break;
>     case PIX_FMT_YUV444P:
>         color_space = CLRSPC_SYCC;
>         numcomps = 3;
>         bpp = 24;
> -        sub_dx[0] = sub_dx[1] = sub_dx[2] = 1;
> -        sub_dy[0] = sub_dy[1] = sub_dy[2] = 1;
>         break;
>     default:
>         av_log(avctx, AV_LOG_ERROR, "The requested pixel format '%s' is not supported\n", av_get_pix_fmt_name(avctx->pix_fmt));
> --
> 1.7.4.1
>
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>

Applied.


More information about the ffmpeg-devel mailing list