Closed
Bug 204073
Opened 21 years ago
Closed 21 years ago
more data could be const in layout
Categories
(Core :: Layout, defect)
Core
Layout
Tracking
()
RESOLVED
FIXED
People
(Reporter: ataylor, Assigned: ataylor)
Details
(Keywords: memory-footprint)
Attachments
(1 file)
|
9.80 KB,
patch
|
bzbarsky
:
review+
alecf
:
superreview+
|
Details | Diff | Splinter Review |
There are a number of variables in mozilla/layout that could be made const. Making them const allows some optimizations not otherwise available, such as sharing more data between multiple running instances.
|
Assignee | |
Comment 1•21 years ago
|
||
Here is a patch that constifies some variables in layout. It makes an additional ~1k of data read-only.
|
||
Comment 2•21 years ago
|
||
Comment on attachment 122208 [details] [diff] [review] constifies some variables in layout nice work! sr=alecf
Attachment #122208 -
Flags: superreview+
|
|
Assignee | |
Updated•21 years ago
|
Attachment #122208 -
Flags: review?(ian)
|
||
Comment 3•21 years ago
|
||
Comment on attachment 122208 [details] [diff] [review] constifies some variables in layout I'm not sure that hixie would be the best r= for this... :) r=me; let me know when you need a checkin (when you have a= or when we reopen for 1.5a).
Attachment #122208 -
Flags: review?(ian) → review+
|
|
||
Comment 4•21 years ago
|
||
Checked in.
Status: NEW → RESOLVED
Closed: 21 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•