Closed Bug 403280 Opened 18 years ago Closed 18 years ago

nsLayoutUtils::DoCompareTreePosition and nsContentUtils::ComparePoints should use nsAutoTArray

Categories

(Core :: DOM: Core & HTML, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: pavlov, Assigned: pavlov)

Details

(Keywords: memory-footprint, perf)

Attachments

(1 file)

Attached patch fixSplinter Review
nsAutoVoidArray only uses 8 buckets which causes lots and lots and lots of allocations when going through this code for no reason. Let us increase that number. also changing nsContentUtils::ComparePosition to 32 per jonas
Attachment #288092 - Flags: review?(jonas)
Attachment #288092 - Flags: superreview+
Attachment #288092 - Flags: review?(jonas)
Attachment #288092 - Flags: review+
checked in.
Status: NEW → RESOLVED
Closed: 18 years ago
Resolution: --- → FIXED
Why not use nsAutoTPtrArray throughout?
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: