encapsulate printf's in BasicTableLayoutStrategy.cpp

VERIFIED FIXED in mozilla0.9.2

Status

()

Core
Layout: Tables
P3
normal
VERIFIED FIXED
17 years ago
17 years ago

People

(Reporter: Bernd, Assigned: Bernd)

Tracking

Trunk
mozilla0.9.2
x86
Windows 98
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(URL)

Attachments

(1 attachment)

(Assignee)

Description

17 years ago
we could significantly better encapsulate the printf's in 
BasicTableLayoutStrategy.cpp by using an #ifdef mechanism instead of global's. 
This would help with bug 78674.
(Assignee)

Comment 1

17 years ago
Created attachment 33091 [details] [diff] [review]
patch
(Assignee)

Updated

17 years ago
Blocks: 78674
Keywords: patch

Comment 2

17 years ago
Bernd, the change looks good to me. As long as Chris approves, I'll gladly give
my sr=attinasi

Comment 3

17 years ago
r=karnaze
Target Milestone: --- → mozilla0.9.1

Comment 4

17 years ago
looks like this should get checked-in in the next couple of days
for wait for the next milestone.   moving non-critical items off
the 0.9.1 list.

Updated

17 years ago
Target Milestone: mozilla0.9.1 → ---

Comment 5

17 years ago
0.9.2 since that's what the bug this blocks is set to.
Priority: -- → P3
Target Milestone: --- → mozilla0.9.2
(Assignee)

Comment 6

17 years ago
fix checked in
Status: NEW → RESOLVED
Last Resolved: 17 years ago
Resolution: --- → FIXED

Comment 7

17 years ago
 Marking Verified
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.