On 06/24/2013 01:37 AM, Derek Buitenhuis wrote:
> Signed-off-by: Derek Buitenhuis <[email protected]>
> ---
> For Kostya.
> ---
> libavcodec/cllc.c | 104
> +++++++++++++++++++++++++++++++++++++++++++++++++++++-
> 1 file changed, 103 insertions(+), 1 deletion(-)
>
> diff --git a/libavcodec/cllc.c b/libavcodec/cllc.c
> index a6b51b3..c371c5c 100644
> --- a/libavcodec/cllc.c
> +++ b/libavcodec/cllc.c
> @@ -1,7 +1,7 @@
> /*
> * Canopus Lossless Codec decoder
> *
> - * Copyright (c) 2012 Derek Buitenhuis
> + * Copyright (c) 2012-2013 Derek Buitenhuis
> *
> * This file is part of Libav.
> *
> @@ -179,6 +179,40 @@ static int read_rgb24_component_line(CLLCContext *ctx,
> GetBitContext *gb,
> return 0;
> }
>
> +static int read_yuv_component_line(CLLCContext *ctx, GetBitContext *gb,
> + int *top_left, VLC *vlc, uint8_t *outbuf,
> + int luma)
> +{
> + uint8_t *dst;
> + int pred, code;
> + int i;
> + int step;
> +
> + OPEN_READER(bits, gb);
> +
> + dst = outbuf;
> + pred = *top_left;
> +
> + step = luma ? 4 : 2;
> +
> + /* Simultaneously read and restore the line */
> + for (i = 0; i < ctx->avctx->width / (step / 2); i++) {
> + UPDATE_CACHE(bits, gb);
> + GET_VLC(code, bits, gb, vlc->table, 7, 2);
> +
> + pred += code;
> + dst[0] = pred;
> + dst += step;
> + }
> +
> + CLOSE_READER(bits, gb);
> +
> + /* Stash the first pixel */
> + *top_left = dst[-2 * ctx->avctx->width];
Wouldn't be simpler referring to outbuf ?
> +
> + return 0;
> +}
> +
> static int decode_argb_frame(CLLCContext *ctx, GetBitContext *gb, AVFrame
> *pic)
> {
> AVCodecContext *avctx = ctx->avctx;
> @@ -267,6 +301,59 @@ static int decode_rgb24_frame(CLLCContext *ctx,
> GetBitContext *gb, AVFrame *pic)
> return 0;
> }
>
> +static int decode_yuv_frame(CLLCContext *ctx, GetBitContext *gb, AVFrame
> *pic)
> +{
> + AVCodecContext *avctx = ctx->avctx;
> + uint8_t block;
> + uint8_t *dst;
> + int pred[3];
> + int ret;
> + int i, j;
> + VLC vlc[2];
> +
> + pred[0] = 0x80;
> + pred[1] = 0x80;
> + pred[2] = 0x80;
> +
> + dst = pic->data[0];
> +
> + skip_bits(gb, 8);
> +
> + block = get_bits(gb, 8);
> + if (block) {
> + av_log(ctx->avctx, AV_LOG_ERROR,
> + "Blocked YUV support is not implemented.");
Use avpriv_request_sample or avpriv_report_missing_feature.
The rest looks fine for me.
lu
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel
