On Fri, Sep 11, 2015 at 6:30 AM, Luca Barbato <[email protected]> wrote:
> That loop is completely stand-alone.
> ---
>  libavcodec/jpeg2000dec.c | 23 +++++++++++++++++------
>  1 file changed, 17 insertions(+), 6 deletions(-)
>
> diff --git a/libavcodec/jpeg2000dec.c b/libavcodec/jpeg2000dec.c
> index f5d5daf..4395dcd 100644
> --- a/libavcodec/jpeg2000dec.c
> +++ b/libavcodec/jpeg2000dec.c
> @@ -1093,7 +1091,9 @@ static int jpeg2000_decode_tile(Jpeg2000DecoderContext 
> *s, Jpeg2000Tile *tile,
>                      Jpeg2000Prec *prec = band->prec + precno;
>
>                      /* Loop on codeblocks */
> -                    for (cblkno = 0; cblkno < prec->nb_codeblocks_width * 
> prec->nb_codeblocks_height; cblkno++) {
> +                    for (cblkno = 0;
> +                         cblkno < prec->nb_codeblocks_width * 
> prec->nb_codeblocks_height;
> +                         cblkno++) {
>                          int x, y;
>                          Jpeg2000Cblk *cblk = prec->cblk + cblkno;
>                          decode_cblk(s, codsty, &t1, cblk,

this change seems unneeded

> @@ -1116,6 +1116,17 @@ static int jpeg2000_decode_tile(Jpeg2000DecoderContext 
> *s, Jpeg2000Tile *tile,
>          /* inverse DWT */
>          ff_dwt_decode(&comp->dwt, codsty->transform == FF_DWT97 ? 
> (void*)comp->f_data : (void*)comp->i_data);
>      } /*end comp */
> +}
> +
> +static int jpeg2000_decode_tile(Jpeg2000DecoderContext *s, Jpeg2000Tile 
> *tile,
> +                                AVFrame *picture)
> +{
> +    int compno;
> +    int x, y;
> +
> +    uint8_t *line;
> +
> +    tile_codeblocks(s, tile);
>
>      /* inverse MCT transformation */
>      if (tile->codsty[0].mct)

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

Reply via email to