[PATCH 2/4] xf86: migrate to using xf86ScreenToScrn wrapper

Keith Packard keithp at keithp.com
Thu May 17 16:02:23 PDT 2012


Dave Airlie <airlied at gmail.com> writes:

> migrate to new helper API.

>      SCREEN_PROLOG(GetImage);
> -//    if (xf86Screens[pScreen->myNum]->vtSema) {
> +//    if (xf86ScreenToScrn(pScreen)->vtSema) {
>      VGAGet(pScreen);
>  //    }

Maybe just remove the commented out lines?

> -      xf86Screens[pmap->pScreen->myNum]->vtSema || pScreenPriv->isDGAmode))
> +      xf86ScreenToScrn(pmap->pScreen)->vtSema || pScreenPriv->isDGAmode))

(I note that a bunch of these are just checking for vtSema; might be
nice to have a helper function for that?)

Reviewed-by: Keith Packard <keithp at keithp.com>

-- 
keith.packard at intel.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 827 bytes
Desc: not available
URL: <http://lists.x.org/archives/xorg-devel/attachments/20120517/203afefb/attachment.pgp>


More information about the xorg-devel mailing list