Closed Bug 16602 Opened 20 years ago Closed 20 years ago

Need to reduce directory enumeration costs via suffix-dispatch

Categories

(Core :: XPCOM, defect, P3)

x86
Linux
defect

Tracking

()

RESOLVED WONTFIX

People

(Reporter: shaver, Assigned: shaver)

References

Details

(Keywords: perf)

We should enumerate the components directory only once (_possibly_ more if we're
doing initial autoregistration, but maybe the installer should have to worry
about that), dispatching to prefs and the typelib loader and whatnot based on
file suffix.
Status: NEW → ASSIGNED
The current list of places in the codebase that do enumeration of the components
directory are:
- autoregistration
  = nsNativeComponentLoader.cpp
  = mozJSComponentLoader.cpp
- default pref loading
  = nsPref.cpp
- typelib file loading
  = nsInterfaceInfoManager.cpp

Coming soon, I think:
- chrome fragment registry

Question for Simon: how do we dispatch on the Mac?  Do we require use of file
suffixes, or do I need to map from file type to ``virtual suffix''?

Question for the crowd: do we care about making autoregistration participate in
this?  It'd be a lot of work to get the ordering work, and I want
autoregistration to die anyway.  If we don't, we still reduce the number of
directory iterations by one and can probably make that iteration faster.
Pref no longer enumerats components directory. Neeti fixed that one.
Keywords: perf
Summary: [perf] Need to reduce directory enumeration costs via suffix-dispatch → Need to reduce directory enumeration costs via suffix-dispatch
This appears not to matter.
Status: ASSIGNED → RESOLVED
Closed: 20 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.