Re: mscms: quiet a noisy fixme (1/3)

[Detlef Riekenberg]

On Do, 2008-10-16 at 19:44 -0500, Austin English wrote:

> +        if (quietfixme == 0)
> +        {
> +            FIXME("unhandled bitmap format\n");
Such a FIXME has no useful information.
You should also dump the unhandled bitmap format in the FIXME

> +            quietfixme = 1;
> +            return TYPE_RGB_8;
> +        }
When you swap both lines above, ...

>      }
> +    return 0;
... you should no longer need the "return 0" above.
>  }
>  



-- 
 
By by ... Detlef