nsPersistentProperties reads one char at a time from the input stream. This should be change to read and process a block at a time.
Summary: nsPersistentProperties read 1 byte at a time. → nsPersistentProperties reads 1 byte at a time.
ack. bad patch. I am reallocating when I don't have too...
hey chris, can you look at this patch?
Wow, so...how many times do we buffer this data now? (I'd have thought the whole point of nsIUnicharInputStream would've been to buffer the data.) Would it be better to do the buffering there? I notice that |ConverterInputStream| (in nsUnicharInputStream.cpp) has an |nsIUnicharBuffer| member: is this not working right?
I should have marked this bug dependent on http://bugzilla.mozilla.org/show_bug.cgi?id=92477. ignore for now... I need to get results.
Depends on: 92477
Target Milestone: mozilla0.9.4 → mozilla0.9.5
Doug ? Seems wrong to make this mozilla1.0 This looks like it can give a sizable perf improvement.
It was marked m1.0 because 93055 was. I see that Darin moved it up. Darin, do you want to see if you can fix up these two callers (92477 and this)
to darin. This should be changed along with the changes to support partial reads
Assignee: dougt → darin
is this really a big performance problem? NS_NewConvertStream returns a input stream that has a buffer. therefore, reading one byte at a time is not really all that costly. i also don't believe that this change would be blocked by bug 93055. resolving WONTFIX unless someone can show that this will actually help. IMO this is only bloat.
Status: NEW → RESOLVED
Last Resolved: 17 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.