[FFmpeg-devel] [PATCH] OpenEXR decoder rev-9

[Diego Biurrun]

On Wed, Jul 08, 2009 at 08:56:39AM +0200, Jimmy Christensen wrote:
> --- libavcodec/exr.c	(revision 0)
> +++ libavcodec/exr.c	(revision 0)
> @@ -0,0 +1,405 @@
> +                            // Process other channels
> +                            if(ptr_tmp < buf) {

nit: if (

> +                                while(ptr_tmp[0] != 0x0) {
> +                                    ptr_tmp++;
> +                                }

pointless {}

> +                    if (*buf) {
> +                        av_log(avctx, AV_LOG_ERROR, "Doesn't support this line order : %d\n", *buf);
> +                    }

ditto

Everything else OK from my side..

Diego