Closed
Bug 114222
Opened 23 years ago
Closed 23 years ago
cleanup nsBlockFrame::RemoveFrame
Categories
(Core :: Layout, defect, P2)
Core
Layout
Tracking
()
RESOLVED
FIXED
mozilla0.9.8
People
(Reporter: dbaron, Assigned: dbaron)
Details
Attachments
(1 file)
|
2.21 KB,
patch
|
waterson
:
review+
kinmoz
:
superreview+
|
Details | Diff | Splinter Review |
I just ran into nsBlockFrame::RemoveFrame, which was a little ugly because it had an unneeded extra test that could have been fixed by an early return, and because the most common case was at the end. Cleanup patch to be attached.
|
Assignee | |
Updated•23 years ago
|
Status: NEW → ASSIGNED
Priority: -- → P2
Target Milestone: --- → mozilla0.9.8
|
|
Assignee | |
Comment 1•23 years ago
|
||
Comment on attachment 60947 [details] [diff] [review] patch r/sr=kin@netscape.com (whichever one you need)
Attachment #60947 -
Flags: superreview+
|
||
Comment 3•23 years ago
|
||
Comment on attachment 60947 [details] [diff] [review] patch r=waterson
Attachment #60947 -
Flags: review+
|
|
Assignee | |
Comment 4•23 years ago
|
||
Fix checked in 2002-01-06 10:17 PDT.
Status: ASSIGNED → RESOLVED
Closed: 23 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•