Closed
Bug 410358
Opened 17 years ago
Closed 17 years ago
use nsTArray in nsTreeBodyFrame
Categories
(Core :: XUL, defect)
Core
XUL
Tracking
()
RESOLVED
FIXED
People
(Reporter: dwitte, Assigned: dwitte)
References
Details
(Keywords: memory-footprint)
Attachments
(1 file)
4.92 KB,
patch
|
neil
:
review+
neil
:
superreview+
pavlov
:
approval1.9+
|
Details | Diff | Splinter Review |
nsValueArray has only two users in the tree, so after this is done i'd like to remove it. (nsTArray has numerous advantages - smaller footprint being one of them, especially when it's not being used, which seems the common case here.) NeilAway, would you like to review this? ;)
Attachment #294982 -
Flags: superreview?(neil)
Attachment #294982 -
Flags: review?(neil)
Assignee | ||
Updated•17 years ago
|
Assignee: nobody → dwitte
Updated•17 years ago
|
Status: NEW → ASSIGNED
Updated•17 years ago
|
Attachment #294982 -
Flags: superreview?(neil)
Attachment #294982 -
Flags: superreview+
Attachment #294982 -
Flags: review?(neil)
Attachment #294982 -
Flags: review+
Assignee | ||
Comment 1•17 years ago
|
||
Comment on attachment 294982 [details] [diff] [review] v1 requesting approval - pretty simple patch that sets the stage for removing old, superceded code.
Attachment #294982 -
Flags: approval1.9?
Updated•17 years ago
|
Attachment #294982 -
Flags: approval1.9? → approval1.9+
Assignee | ||
Comment 2•17 years ago
|
||
checked in.
Status: ASSIGNED → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
Component: XP Toolkit/Widgets: Trees → XUL
QA Contact: xptoolkit.trees → xptoolkit.widgets
You need to log in
before you can comment on or make changes to this bug.
Description
•