Closed
Bug 148468
Opened 22 years ago
Closed 22 years ago
Xprint-specific X font banning code is broken
Categories
(Core Graveyard :: Printing: Xprint, defect, P1)
Tracking
(Not tracked)
RESOLVED
FIXED
mozilla1.0.1
People
(Reporter: roland.mainz, Assigned: roland.mainz)
References
Details
Attachments
(1 file, 1 obsolete file)
|
1.51 KB,
patch
|
timeless
:
review+
scc
:
superreview+
jesup
:
approval+
|
Details | Diff | Splinter Review |
Xprint-specific X font banning code is broken Patch follows...
|
Assignee | |
Comment 1•22 years ago
|
||
Taking myself...
Assignee: katakai → Roland.Mainz
QA Contact: Roland.Mainz → katakai
|
|
Assignee | |
Updated•22 years ago
|
Status: NEW → ASSIGNED
Priority: -- → P1
Target Milestone: --- → mozilla1.0.1
|
|
Assignee | |
Comment 2•22 years ago
|
||
|
|
Assignee | |
Comment 4•22 years ago
|
||
Attachment #85865 -
Attachment is obsolete: true
Comment on attachment 85991 [details] [diff] [review] New patch for 2002-05-31-08-trunk per timeless's comments i didn't want the {}s around the non ifdef'd code, but sure.
Attachment #85991 -
Flags: review+
|
|
Assignee | |
Comment 6•22 years ago
|
||
timeless wrote:
> (From update of attachment 85991 [details] [diff] [review])
> i didn't want the {}s around the non ifdef'd code, but sure.
I added it that we can put "in" some *_FONT_PRINTF() debug macros at a later
point without touching the surrounding lines...
|
||
Comment 7•22 years ago
|
||
Comment on attachment 85991 [details] [diff] [review] New patch for 2002-05-31-08-trunk per timeless's comments sr=scc
Attachment #85991 -
Flags: superreview+
|
|
Assignee | |
Updated•22 years ago
|
Blocks: xprint_machv
|
||
Comment 8•22 years ago
|
||
Checked in to trunk for Roland
Status: ASSIGNED → RESOLVED
Closed: 22 years ago
Resolution: --- → FIXED
|
|
Assignee | |
Comment 9•22 years ago
|
||
Requesting a= for 1.0.1-branch...
|
||
Comment 10•22 years ago
|
||
Comment on attachment 85991 [details] [diff] [review] New patch for 2002-05-31-08-trunk per timeless's comments Approved for 1.0 branch. Please remove mozilla1.0.1+ when it's fixed and add fixed1.0.1
Attachment #85991 -
Flags: approval+
|
|
||
Updated•22 years ago
|
|
||
Updated•16 years ago
|
Product: Core → Core Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•