Doug Turner wrote:

>> Who needs the service manager but doesn't get it through their 
>> nsIModule thing?  
> 
> It will be a requirement of the 'glue' helper library that we are 
> creating.  A similar API will be NS_GetComponentManager().


Why is it a requirement?  (Where can I find these requirements?)

I thought the glue library was going to prevent people from having to 
link to libxpcom, by letting them pull in some things (nsGenericFactory, 
nsCOMPtr, etc.) that they are currently linking for, and therefore at 
risk from changes to.

It looks like the stuff in your patch doesn't do that, because it still 
calls through to NS_GetServiceManager and therefore has a link-time 
dependency on some symbol in libxpcom.

What am I missing here?

(I also don't like the addition of NS_CreateServicesFromCategory, least 
of all in an IDL file!  I'll comment in the bug in a bit.)

>> document and support?
> 
> I don't think that we have a choice.  We need public methods to get to 
> the service manager and component manager.


Why?  Who needs them?  If there are people who don't get the component 
manager-cum-service manager through their nsIModule calls, I agree 
completely.  But I can't think of who those would be.

Mike


Reply via email to