Closed Bug 16602 Opened 20 years ago Closed 20 years ago
Need to reduce directory enumeration costs via suffix-dispatch
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.
20 years ago
20 years ago
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.
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.