Closed Bug 41180 Opened 25 years ago Closed 25 years ago

unnecessary allocation from BufferContainsHTML

Categories

(Core :: DOM: HTML Parser, defect, P3)

Product:
Core
Component:
DOM: HTML Parser
Platform:
x86
Linux
Type:
defect

Tracking

()

Status:
VERIFIED FIXED

People

(Reporter: waterson, Assigned: rickg)

References

(
URL
)

Details

(Keywords: arch, Whiteboard: fix in hand)

Bringing up an empty web page in viewer causes this to allocate ~3Kb of data, at 200 bytes a pop from nsString::Left(). Seems like this could be done more efficiently without allocation, esp. using iterator APIs in the new string stuff.
Keywords: arch
Trivial fix in hand. If you folks only knew how much time we spend trying to eliminate string allocations in the parsing engine. :)
Status: NEW → ASSIGNED
Marking fix in hand
Whiteboard: fix in hand
Rewrote the routine to use more efficient string searches, and eliminated buffer allocations.
Status: ASSIGNED → RESOLVED
Closed: 25 years ago
Resolution: --- → FIXED
updated qa contact.
QA Contact: janc → bsharma
Verified on 2001-07-19-branch on WinNT Marking verified as per above developer comments.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.