[PATCH 05/36] xf86: cleanup helper code to use a pointer.

Aaron Plattner aplattner at nvidia.com
Tue Jul 3 17:04:04 PDT 2012


On 07/02/2012 03:12 AM, Dave Airlie wrote:
> From: Dave Airlie <airlied at redhat.com>
>
> This is in preparation for gpu screens in here, just use
> a pScrn pointer to point at the new screen.
>
> suggested by Keith.
>
> Reviewed-by: Keith Packard <keithp at keithp.com>
> Signed-off-by: Dave Airlie <airlied at redhat.com>
> ---
>   hw/xfree86/common/xf86Helper.c |   18 ++++++++++--------
>   1 file changed, 10 insertions(+), 8 deletions(-)
>
> diff --git a/hw/xfree86/common/xf86Helper.c b/hw/xfree86/common/xf86Helper.c
> index 5ef1dab..4f3315b 100644
> --- a/hw/xfree86/common/xf86Helper.c
> +++ b/hw/xfree86/common/xf86Helper.c
> @@ -167,6 +167,7 @@ ScrnInfoPtr
>   xf86AllocateScreen(DriverPtr drv, int flags)
>   {
>       int i;
> +    ScrnInfoPtr pScrn;
>
>       if (xf86Screens == NULL)
>           xf86NumScreens = 0;
> @@ -174,22 +175,23 @@ xf86AllocateScreen(DriverPtr drv, int flags)
>       i = xf86NumScreens++;
>       xf86Screens = xnfrealloc(xf86Screens, xf86NumScreens * sizeof(ScrnInfoPtr));
>       xf86Screens[i] = xnfcalloc(sizeof(ScrnInfoRec), 1);
> -    xf86Screens[i]->scrnIndex = i;      /* Changes when a screen is removed */
> -    xf86Screens[i]->origIndex = i;      /* This never changes */
> -    xf86Screens[i]->privates = xnfcalloc(sizeof(DevUnion),
> +    pScrn = xf86Screens[i];
> +    pScrn->scrnIndex = i;      /* Changes when a screen is removed */
> +    pScrn->origIndex = i;      /* This never changes */
> +    pScrn->privates = xnfcalloc(sizeof(DevUnion),
>                                            xf86ScrnInfoPrivateCount);

Spacing got messed up here.  Otherwise,

Reviewed-by: Aaron Plattner <aplattner at nvidia.com>

-- Aaron

>       /*
>        * EnableDisableFBAccess now gets initialized in InitOutput()
> -     * xf86Screens[i]->EnableDisableFBAccess = xf86EnableDisableFBAccess;
> +     * pScrn->EnableDisableFBAccess = xf86EnableDisableFBAccess;
>        */
>
> -    xf86Screens[i]->drv = drv;
> +    pScrn->drv = drv;
>       drv->refCount++;
> -    xf86Screens[i]->module = DuplicateModule(drv->module, NULL);
> +    pScrn->module = DuplicateModule(drv->module, NULL);
>
> -    xf86Screens[i]->DriverFunc = drv->driverFunc;
> +    pScrn->DriverFunc = drv->driverFunc;
>
> -    return xf86Screens[i];
> +    return pScrn;
>   }
>
>   /*
>




More information about the xorg-devel mailing list