Closed Bug 8623 Opened 25 years ago Closed 23 years ago

API: UnregisterComponent shouldn't return nsresult

Categories

(Core :: XPCOM, defect, P3)

All
Windows NT
defect

Tracking

()

RESOLVED WONTFIX
mozilla1.0

People

(Reporter: warrensomebody, Assigned: dougt)

References

Details

UnregisterComponent shouldn't return nsresult. It should never fail, causing the NSUnregisterSelf to bail and not finish unregistering stuff.
Status: NEW → ASSIGNED
Summary: UnregisterComponent shouldn't return nsresult → API: UnregisterComponent shouldn't return nsresult
Target Milestone: M10
You might want to make this change at the same time you split out NSRegisterSelf and NSUnregisterSelf into an interface. While you're at it, you should clean up everything to use GenericFactory -- actually a subclass of GenericFactory that requires callbacks for RegisterSelf and UnregisterSelf (along with the DLL version number, name, etc.)
Target Milestone: M10 → M14
Target Milestone: M14 → M17
Target Milestone: M17 → M20
Moving all current open XPCOM and XPCOM Registry bugs to rayw since dp is on sabbatical. rayw is now default assignee for these components.
Assignee: dp → rayw
Status: ASSIGNED → NEW
This is part of a large number of issues related to "Registering Components in XPCOM". The interface that already contains registration and unregistration is nsIModule, and does not return anything. The problem is that lots of DLLs are not using it as they should.
Status: NEW → ASSIGNED
Depends on: 46768
Assigning rest of xpcom stuff to myself (from Ray).
Assignee: rayw → warren
Status: ASSIGNED → NEW
Component: XPCOM Registry → XPCOM
Target Milestone: M20 → mozilla1.0
dp is no longer @netscape.com. changing qa contact to default for this product
QA Contact: dp → kandrot
reasigning warren bugs to default component owners.
Assignee: warren → kandrot
QA Contact: kandrot → scc
Target Milestone: mozilla1.0 → ---
reassign all kandrot xpcom bug.
Assignee: kandrot → dougt
conrad, is unregistration implied w/ "replace" in the component manager?
Blocks: 70229
Blocks: 98278
Target Milestone: --- → mozilla1.0
I disagree. everything can fail. I think that interface methods should return nsresult's and code should deal with it. marking invalid. reopen if opinions differ.
Keywords: mozilla1.0
Marking wont fix.
Status: NEW → RESOLVED
Closed: 23 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.