Closed Bug 3934 Opened 26 years ago Closed 26 years ago

FileSpec performance needs to get better for serious use

Categories

(Core Graveyard :: Tracking, defect, P3)

x86
All

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: dp, Assigned: mcmullen)

Details

Too many copies and allocs are happeing. We should figure out usage patterns and improve performance.
Target Milestone: M4
Set target milestone to M4. John, do we need to fix this soon?
I have already made some changes that will help this problem: reference counting, plus some new path concatenation operators on nsFileURL and nsFilePath (previously only on nsFileSpec), which will mean that it won't be so necessary for people living in the URL world to convert back and forth between URL and spec. I don't think it's urgent, though dp might disagree. As long as I promise to check this stuff in at some point, I hope dp's worries are allayed.
Status: ASSIGNED → RESOLVED
Closed: 26 years ago
Resolution: --- → FIXED
DP, please try it again now. Lots of performance improvements. You can use the code I suggested in my email now, concatenating directly to the nsFileURL. Marking fixed, but if you see more problems, I know you'll reopen it.
Status: RESOLVED → VERIFIED
Win and Mac holding their own against IE (even without Necko in yet). Linux has gotten better from a user perspective (no auto performance tests yet). Marking Verified. Performance is monitored weekly.
Moving all Apprunner bugs past and present to Other component temporarily whilst don and I set correct component. Apprunner component will be deleted/retired shortly.
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.