Uwe Bonnes <[EMAIL PROTECTED]> writes:

> +    if ((init & CBM_INIT) && (IsBadReadPtr(bits,width*height*bpp/8)))
> +     {
> +       WARN("unreadable bits\n");
> +       return 0;
> +     }

It would be better to do this with an exception handler. Using the
IsBad* functions is usually a bad idea, as they hurt performance and
do not offer any guarantee in a multi-threaded environment anyway.

-- 
Alexandre Julliard
[EMAIL PROTECTED]

Reply via email to