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

[Diego Biurrun]

On Tue, Sep 08, 2009 at 03:30:26PM +0200, Jimmy Christensen wrote:
> 
> --- libavcodec/exr.c	(revision 0)
> +++ libavcodec/exr.c	(revision 0)
> @@ -0,0 +1,469 @@
> +static const unsigned int bytes_per_color_table[3] = {
> +        1, 2, 4

Indentation is off

> +
> +    if (channel_iter == s->red_channel ||
> +        channel_iter == s->green_channel ||
> +        channel_iter == s->blue_channel) {

nit: align the ||

> +                switch(*buf) {

switch (

> +    if (s->red_channel != -1 &&
> +        s->blue_channel != -1 &&
> +        s->green_channel != -1) {

nit: align

Diego