Closed Bug 55228 Opened 24 years ago Closed 18 years ago

nsObserverService::Notify could be better

Categories

(Core :: XPCOM, defect, P3)

defect

Tracking

()

RESOLVED WORKSFORME
Future

People

(Reporter: jonsmirl, Unassigned)

References

Details

(Keywords: helpwanted, perf)

Attachments

(1 file)

nsObserverService::Notify does not check for the case of an empty list. Instead 
if calls nsObserverService::GetObserverList which creates a list if none 
exists.  This causes a lot of needless work during the shutdown process if the 
Observer list was never created in the first place.
setting Jon Smirl's bugs to New
Status: UNCONFIRMED → NEW
Ever confirmed: true
Transferring to default XPCOM owner.
Assignee: rayw → warren
reasigning warren bugs to default component owners.
Assignee: warren → kandrot
QA Contact: rayw → scc
reassign all kandrot xpcom bug.
Assignee: kandrot → dougt
Keywords: perf
Blocks: 98275
*** Bug 101448 has been marked as a duplicate of this bug. ***
Keywords: helpwanted
Target Milestone: --- → Future
Keywords: mozilla1.0+
Keywords: mozilla1.0+mozilla1.0-
Attached patch patchSplinter Review
this will break a patch i have.
Assignee: dougt → nobody
QA Contact: scc → xpcom
This is significantly better now.
Status: NEW → RESOLVED
Closed: 18 years ago
Resolution: --- → WORKSFORME
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: