mscms: quiet a noisy fixme (1/3)
Austin English
austinenglish at gmail.com
Fri Oct 17 15:12:56 CDT 2008
On Fri, Oct 17, 2008 at 12:09 PM, Detlef Riekenberg <wine.dev at web.de> wrote:
> 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
>
There's a trace above that does that, though the FIXME should as well.
>> + quietfixme = 1;
>> + return TYPE_RGB_8;
>> + }
> When you swap both lines above, ...
>
>> }
>> + return 0;
> ... you should no longer need the "return 0" above.
>> }
Nope, still fails.
--
-Austin
More information about the wine-devel
mailing list