From a15dac550918c55b08bbe0007f068cc5096ebaa1 Mon Sep 17 00:00:00 2001 From: Daniel Stone Date: Tue, 10 Jul 2012 02:03:28 +0100 Subject: [PATCH] Remove unused setupFunc from extensions setupFunc was used as an early callback for half-modular extensions such as Xv, XvMC and DGA to set up hooks between the core server and the modular component. Now we've rid ourselves of that, we can also bin setupFunc. Signed-off-by: Daniel Stone Reviewed-by: Jamey Sharp Signed-off-by: Keith Packard --- hw/xfree86/common/xf86Extensions.c | 12 +++---- hw/xfree86/dixmods/glxmodule.c | 3 +- hw/xfree86/doc/ddxDesign.xml | 1 - include/extension.h | 1 - mi/miinitext.c | 56 +++++++++++++----------------- 5 files changed, 30 insertions(+), 43 deletions(-) diff --git a/hw/xfree86/common/xf86Extensions.c b/hw/xfree86/common/xf86Extensions.c index 8cbeedf3b..d5309d558 100644 --- a/hw/xfree86/common/xf86Extensions.c +++ b/hw/xfree86/common/xf86Extensions.c @@ -58,32 +58,28 @@ static ExtensionModule extensionModules[] = { { XFree86VidModeExtensionInit, XF86VIDMODENAME, - &noXFree86VidModeExtension, - NULL + &noXFree86VidModeExtension }, #endif #ifdef XFreeXDGA { XFree86DGAExtensionInit, XF86DGANAME, - &noXFree86DGAExtension, - NULL + &noXFree86DGAExtension }, #endif #ifdef XF86DRI { XFree86DRIExtensionInit, "XFree86-DRI", - &noXFree86DRIExtension, - NULL + &noXFree86DRIExtension }, #endif #ifdef DRI2 { DRI2ExtensionInit, DRI2_NAME, - &noDRI2Extension, - NULL + &noDRI2Extension } #endif }; diff --git a/hw/xfree86/dixmods/glxmodule.c b/hw/xfree86/dixmods/glxmodule.c index 2f0a3dd2e..5a9e84529 100644 --- a/hw/xfree86/dixmods/glxmodule.c +++ b/hw/xfree86/dixmods/glxmodule.c @@ -49,8 +49,7 @@ static MODULESETUPPROTO(glxSetup); static ExtensionModule GLXExt = { GlxExtensionInit, "GLX", - &noGlxExtension, - NULL + &noGlxExtension }; static XF86ModuleVersionInfo VersRec = { diff --git a/hw/xfree86/doc/ddxDesign.xml b/hw/xfree86/doc/ddxDesign.xml index 118b58f95..ce81a9d5d 100644 --- a/hw/xfree86/doc/ddxDesign.xml +++ b/hw/xfree86/doc/ddxDesign.xml @@ -5973,7 +5973,6 @@ typedef struct { InitExtension initFunc; char * name; Bool *disablePtr; - InitExtension setupFunc; } ExtensionModule; diff --git a/include/extension.h b/include/extension.h index cc1dfa1fc..0f55d9038 100644 --- a/include/extension.h +++ b/include/extension.h @@ -84,7 +84,6 @@ typedef struct { InitExtension initFunc; const char *name; Bool *disablePtr; - InitExtension setupFunc; } ExtensionModule; extern _X_EXPORT unsigned short StandardMinorOpcode(ClientPtr /*client */ ); diff --git a/mi/miinitext.c b/mi/miinitext.c index a84d486e1..86fc31c99 100644 --- a/mi/miinitext.c +++ b/mi/miinitext.c @@ -235,65 +235,63 @@ EnableDisableExtensionError(const char *name, Bool enable) /* List of built-in (statically linked) extensions */ static ExtensionModule staticExtensions[] = { - {GEExtensionInit, "Generic Event Extension", &noGEExtension, NULL}, - {ShapeExtensionInit, "SHAPE", NULL, NULL}, + {GEExtensionInit, "Generic Event Extension", &noGEExtension}, + {ShapeExtensionInit, "SHAPE", NULL}, #ifdef MITSHM - {ShmExtensionInit, SHMNAME, &noMITShmExtension, NULL}, + {ShmExtensionInit, SHMNAME, &noMITShmExtension}, #endif - {XInputExtensionInit, "XInputExtension", NULL, NULL}, + {XInputExtensionInit, "XInputExtension", NULL}, #ifdef XTEST - {XTestExtensionInit, XTestExtensionName, &noTestExtensions, NULL}, + {XTestExtensionInit, XTestExtensionName, &noTestExtensions}, #endif - {BigReqExtensionInit, "BIG-REQUESTS", NULL, NULL}, - {SyncExtensionInit, "SYNC", NULL, NULL}, - {XkbExtensionInit, XkbName, NULL, NULL}, - {XCMiscExtensionInit, "XC-MISC", NULL, NULL}, + {BigReqExtensionInit, "BIG-REQUESTS", NULL}, + {SyncExtensionInit, "SYNC", NULL}, + {XkbExtensionInit, XkbName, NULL}, + {XCMiscExtensionInit, "XC-MISC", NULL}, #ifdef XCSECURITY - {SecurityExtensionInit, SECURITY_EXTENSION_NAME, &noSecurityExtension, NULL}, + {SecurityExtensionInit, SECURITY_EXTENSION_NAME, &noSecurityExtension}, #endif #ifdef PANORAMIX - {PanoramiXExtensionInit, PANORAMIX_PROTOCOL_NAME, &noPanoramiXExtension, - NULL}, + {PanoramiXExtensionInit, PANORAMIX_PROTOCOL_NAME, &noPanoramiXExtension}, #endif #ifdef XFIXES /* must be before Render to layer DisplayCursor correctly */ - {XFixesExtensionInit, "XFIXES", &noXFixesExtension, NULL}, + {XFixesExtensionInit, "XFIXES", &noXFixesExtension}, #endif #ifdef XF86BIGFONT - {XFree86BigfontExtensionInit, XF86BIGFONTNAME, &noXFree86BigfontExtension, - NULL}, + {XFree86BigfontExtensionInit, XF86BIGFONTNAME, &noXFree86BigfontExtension}, #endif - {RenderExtensionInit, "RENDER", &noRenderExtension, NULL}, + {RenderExtensionInit, "RENDER", &noRenderExtension}, #ifdef RANDR - {RRExtensionInit, "RANDR", &noRRExtension, NULL}, + {RRExtensionInit, "RANDR", &noRRExtension}, #endif #ifdef COMPOSITE - {CompositeExtensionInit, "COMPOSITE", &noCompositeExtension, NULL}, + {CompositeExtensionInit, "COMPOSITE", &noCompositeExtension}, #endif #ifdef DAMAGE - {DamageExtensionInit, "DAMAGE", &noDamageExtension, NULL}, + {DamageExtensionInit, "DAMAGE", &noDamageExtension}, #endif #ifdef SCREENSAVER - {ScreenSaverExtensionInit, ScreenSaverName, &noScreenSaverExtension, NULL}, + {ScreenSaverExtensionInit, ScreenSaverName, &noScreenSaverExtension}, #endif #ifdef DBE - {DbeExtensionInit, "DOUBLE-BUFFER", &noDbeExtension, NULL}, + {DbeExtensionInit, "DOUBLE-BUFFER", &noDbeExtension}, #endif #ifdef XRECORD - {RecordExtensionInit, "RECORD", &noTestExtensions, NULL}, + {RecordExtensionInit, "RECORD", &noTestExtensions}, #endif #ifdef DPMSExtension - {DPMSExtensionInit, DPMSExtensionName, &noDPMSExtension, NULL}, + {DPMSExtensionInit, DPMSExtensionName, &noDPMSExtension}, #endif #ifdef RES - {ResExtensionInit, XRES_NAME, &noResExtension, NULL}, + {ResExtensionInit, XRES_NAME, &noResExtension}, #endif #ifdef XV - {XvExtensionInit, XvName, &noXvExtension, NULL}, - {XvMCExtensionInit, XvMCName, &noXvExtension, NULL}, + {XvExtensionInit, XvName, &noXvExtension}, + {XvMCExtensionInit, XvMCName, &noXvExtension}, #endif #ifdef XSELINUX - {SELinuxExtensionInit, SELINUX_EXTENSION_NAME, &noSELinuxExtension, NULL}, + {SELinuxExtensionInit, SELINUX_EXTENSION_NAME, &noSELinuxExtension}, #endif }; @@ -379,8 +377,4 @@ LoadExtension(ExtensionModule * e, Bool builtin) newext->name = e->name; newext->initFunc = e->initFunc; newext->disablePtr = e->disablePtr; - newext->setupFunc = e->setupFunc; - - if (e->setupFunc != NULL) - e->setupFunc(); }