On Sat, 10 Apr 2010 19:16:40 +1000, [email protected] wrote:

> +             if (! pDstPix) /* Usually implies display driver rejected 
> width, height, depth, etc... */
> +             {
> +                 err = BadValue;
> +                 goto bail;
> +             }
> +

The only error you can legitimately return here is BadAlloc.

Otherwise, this looks like a fine patch, please be sure to fix it and
add a Signed-off-by: line so that I can merge it into the server.

-- 
[email protected]

Attachment: pgpIg1OtD8Ss6.pgp
Description: PGP signature

_______________________________________________
[email protected]: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: http://lists.x.org/mailman/listinfo/xorg-devel

Reply via email to