As part of the work in http://bugzilla.mozilla.org/show_bug.cgi?id=67699 I'm revamping some of the generic module code to be a little cleaner and help the nsIClassInfo cause. One thing I'd like to do is change the signature of nsModuleComponentInfo::mRegisterSelfProc (and mUnregisterSelfProc) to take an additional |nsModuleComponentInfo *| argument. So NSRegisterSelfProcPtr would become: typedef NS_CALLBACK(NSRegisterSelfProcPtr) (nsIComponentManager *aCompMgr, nsIFile *aPath, const char *registryLocation, const char *componentType, const nsModuleComponentInfo *info); and NSUnregisterSelfProcPtr would become: typedef NS_CALLBACK(NSUnregistrSelfProcPtr) (nsIComponentManager *aCompMgr, nsIFile *aPath, const char *registryLocation, const nsModuleComponentInfo *info); Are there any objections to that? It'll let things like nsUConvModule use the generic factory and nsIClassInfo infrastructure much more easily, and it seems like it's generally the right thing. (And while I'm at it, are there any objections to the other changes to nsModuleComponentInfo in my latest patch to 67699?) Mike
