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

Reply via email to