[PATCH] Delete redundant GC initializations.
Alex Deucher
alexdeucher at gmail.com
Tue Jul 13 19:13:57 PDT 2010
On Tue, Jul 13, 2010 at 6:35 PM, Jamey Sharp <jamey at minilop.net> wrote:
> When a GC is allocated, it is zeroed, including all storage requested
> with dixRegisterPrivateKey. So CreateGC hooks don't need to initialize
> anything to zero.
>
> Signed-off-by: Jamey Sharp <jamey at minilop.net>
Reviewed-by: Alex Deucher <alexdeucher at gmail.com>
> ---
> fb/fbgc.c | 8 +-------
> hw/xfree86/xaa/xaaGC.c | 5 -----
> hw/xnest/GC.c | 3 ---
> hw/xwin/wingc.c | 5 -----
> miext/cw/cw.c | 1 -
> miext/damage/damage.c | 1 -
> 6 files changed, 1 insertions(+), 22 deletions(-)
>
> diff --git a/fb/fbgc.c b/fb/fbgc.c
> index 45e981b..b27a030 100644
> --- a/fb/fbgc.c
> +++ b/fb/fbgc.c
> @@ -64,19 +64,13 @@ const GCOps fbGCOps = {
> Bool
> fbCreateGC(GCPtr pGC)
> {
> - pGC->clientClip = NULL;
> - pGC->clientClipType = CT_NONE;
> -
> pGC->ops = (GCOps *) &fbGCOps;
> pGC->funcs = (GCFuncs *) &fbGCFuncs;
>
> /* fb wants to translate before scan conversion */
> pGC->miTranslate = 1;
> + pGC->fExpose = 1;
>
> - fbGetRotatedPixmap(pGC) = 0;
> - fbGetExpose(pGC) = 1;
> - fbGetFreeCompClip(pGC) = 0;
> - fbGetCompositeClip(pGC) = 0;
> fbGetGCPrivate(pGC)->bpp = BitsPerPixel (pGC->depth);
> return TRUE;
> }
> diff --git a/hw/xfree86/xaa/xaaGC.c b/hw/xfree86/xaa/xaaGC.c
> index fb24387..44d50e6 100644
> --- a/hw/xfree86/xaa/xaaGC.c
> +++ b/hw/xfree86/xaa/xaaGC.c
> @@ -48,11 +48,6 @@ XAACreateGC(GCPtr pGC)
> pGCPriv->wrapOps = NULL;
> pGCPriv->wrapFuncs = pGC->funcs;
> pGCPriv->XAAOps = &XAAFallbackOps;
> - pGCPriv->flags = 0;
> - pGCPriv->DashLength = 0;
> - pGCPriv->DashPattern = NULL;
> - pGCPriv->changes = 0;
> - /* initialize any other private fields here */
> pGC->funcs = &XAAGCFuncs;
> }
>
> diff --git a/hw/xnest/GC.c b/hw/xnest/GC.c
> index 407cc77..7968b4f 100644
> --- a/hw/xnest/GC.c
> +++ b/hw/xnest/GC.c
> @@ -73,9 +73,6 @@ static GCOps xnestOps = {
> Bool
> xnestCreateGC(GCPtr pGC)
> {
> - pGC->clientClipType = CT_NONE;
> - pGC->clientClip = NULL;
> -
> pGC->funcs = &xnestFuncs;
> pGC->ops = &xnestOps;
>
> diff --git a/hw/xwin/wingc.c b/hw/xwin/wingc.c
> index 196b5b5..e351c50 100644
> --- a/hw/xwin/wingc.c
> +++ b/hw/xwin/wingc.c
> @@ -137,11 +137,6 @@ winCreateGCNativeGDI (GCPtr pGC)
> pGC->depth);
> #endif
>
> - pGC->clientClip = NULL;
> - pGC->clientClipType = CT_NONE;
> - pGC->freeCompClip = FALSE;
> - pGC->pCompositeClip = 0;
> -
> pGC->ops = (GCOps *) &winGCOps;
> pGC->funcs = (GCFuncs *) &winGCFuncs;
>
> diff --git a/miext/cw/cw.c b/miext/cw/cw.c
> index 58816c9..3da3bc3 100644
> --- a/miext/cw/cw.c
> +++ b/miext/cw/cw.c
> @@ -325,7 +325,6 @@ cwCreateGC(GCPtr pGC)
> ScreenPtr pScreen = pGC->pScreen;
> Bool ret;
>
> - memset(pPriv, 0, sizeof(cwGCRec));
> SCREEN_PROLOGUE(pScreen, CreateGC);
>
> if ( (ret = (*pScreen->CreateGC)(pGC)) )
> diff --git a/miext/damage/damage.c b/miext/damage/damage.c
> index f5917ea..2ea5e63 100644
> --- a/miext/damage/damage.c
> +++ b/miext/damage/damage.c
> @@ -450,7 +450,6 @@ damageCreateGC(GCPtr pGC)
> damageGCPriv(pGC);
> Bool ret;
>
> - pGC->pCompositeClip = 0;
> unwrap (pScrPriv, pScreen, CreateGC);
> if((ret = (*pScreen->CreateGC) (pGC))) {
> pGCPriv->ops = NULL;
> --
> 1.7.0
>
> _______________________________________________
> xorg-devel at lists.x.org: X.Org development
> Archives: http://lists.x.org/archives/xorg-devel
> Info: http://lists.x.org/mailman/listinfo/xorg-devel
>
More information about the xorg-devel
mailing list