Closed
Bug 88735
Opened 24 years ago
Closed 24 years ago
misc cleanup of block code
Categories
(Core :: Layout, defect, P1)
Core
Layout
Tracking
()
VERIFIED
FIXED
mozilla0.9.3
People
(Reporter: dbaron, Assigned: dbaron)
Details
Attachments
(1 file)
|
29.97 KB,
patch
|
Details | Diff | Splinter Review |
I've been reading the block code to try and learn how it works. So far I've
read through nsBlock{Frame,ReflowContext,ReflowState}.cpp from beginning to end
(with various attention to detail). I made a lot of comments that I'd like to
investigate later (some of which should probably be filed as bugs ASAP), removed
some unused code, renamed some unnecessarily-overloaded functions, and corrected
some typos in comments.
|
Assignee | |
Comment 1•24 years ago
|
||
|
|
Assignee | |
Updated•24 years ago
|
Status: NEW → ASSIGNED
Priority: -- → P1
Target Milestone: --- → mozilla0.9.3
|
||
Comment 2•24 years ago
|
||
I think that the method renaming and dead code elimincation is valuable, and
would actually appreciate having some of these comments in my tree, too. I say
check it in...r=waterson.
|
|
||
Comment 3•24 years ago
|
||
Looks good. sr=attinasi
|
|
Assignee | |
Comment 4•24 years ago
|
||
Checked in 2001-07-03 19:00 PDT.
Status: ASSIGNED → RESOLVED
Closed: 24 years ago
Resolution: --- → FIXED
|
||
Comment 5•24 years ago
|
||
Since I 'm not exactly sure how to test this, I'm marking verified based on the
last comments.
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•