Closed
Bug 3211
Opened 26 years ago
Closed 26 years ago
RX: Service Manager and Repository
Categories
(Core :: XPCOM, defect, P2)
Tracking
()
VERIFIED
FIXED
M4
People
(Reporter: chofmann, Assigned: dp)
Details
(Whiteboard: 3/25/99 -- awaiting reporter to verify)
May be safe, but bug warren and dp:
Much of our non-static/global data needs to be partitioned by window,
not reachable by other windows except through statics/globals.
Therefore we're starting the hunt with statics and globals.
Please have a look at the file(s) below to see if we can improve reentrantcy
across
the code base by making modifications to:
xpcom/public/nsIServiceManager.h: static nsIServiceManager*
globalServiceManager;
xpcom/public/nsMacRepository.h:static MacLibrary libraries[] = {
xpcom/public/nsRepository.h: static nsHashtable *factories;
xpcom/public/nsRepository.h: static PRMonitor *monitor;
xpcom/public/nsRepository.h: static nsDllStore *dllStore;
Comment 1•26 years ago
|
||
I think globalServiceManager is the one true global. It's access is already
synchronized.
Reporter | ||
Comment 2•26 years ago
|
||
RX tasks aren't going to make M3.
close the reentrantcy tracking tasks if this specific area has been looked at
and doesn't seem to be a problem. Otherwise each of these areas
still need some looking at.
Updated•26 years ago
|
Status: NEW → RESOLVED
Closed: 26 years ago
Resolution: --- → FIXED
Comment 3•26 years ago
|
||
I think this is now a non-issue. (a) the statics have become instance variables,
(b) you can create per-window service managers
Updated•26 years ago
|
QA Contact: 3849
Whiteboard: awaiting reporter to verify
Updated•26 years ago
|
Whiteboard: awaiting reporter to verify → 3/25/99 -- awaiting reporter to verify
Reporter | ||
Updated•26 years ago
|
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•