Closed
Bug 100328
Opened 23 years ago
Closed 23 years ago
[FIX][ps]support for printing of non-latin1 chars using of HTML->PS
Categories
(Core :: Printing: Output, defect)
Tracking
()
VERIFIED
FIXED
mozilla0.9.8
People
(Reporter: ggromov, Assigned: rods)
References
Details
Attachments
(1 file, 1 obsolete file)
(deleted),
patch
|
brendan
:
review+
brendan
:
superreview+
|
Details | Diff | Splinter Review |
Here is a patch to allow printing much more characters using HTML->PS
printing subsystem. It's a replacement for real_unicodeshow PS routine,
it that uses a dictionary mapping unicode codes to glyph names, and then
uses glyphname to show given glyph.
---
The patch is created by ALTlinux team (www.altlinux.ru) and is used
in their package of mozilla.
Their SRPMs with this patch included are publically distributed since
spring 2001,
from ftp://ftp.altlinux.ru/pub/distributions/ALTLinux/Sisyphus/SRPMS/
(NOTE: their ftp supports passive mode only!)
Please give credit to them for their wonderfull patches somewhere in
release notes and/or documentation, if possible.
Reporter | ||
Comment 1•23 years ago
|
||
Updated•23 years ago
|
Assignee | ||
Updated•23 years ago
|
Status: NEW → ASSIGNED
Comment 4•23 years ago
|
||
r=dcone
Comment 5•23 years ago
|
||
The patch is out of date (XP_FilePrintf is no more, good ol' stdio fprintf
suffices on all relevant platforms). Rod, can you apply the patch to the
version it was based on (one from around mid-September -- use cvs up -r to stick
to a rev or date near there that patches cleanly), then cvs up -A to bring the
changes forward. You'll want to change the calls such as XP_FilePrintf to fprintf.
/be
Reporter | ||
Comment 6•23 years ago
|
||
Guys, please also take a look at #100324 - it also contains a related patch.
Assignee | ||
Updated•23 years ago
|
Attachment #49757 -
Attachment is obsolete: true
Assignee | ||
Comment 7•23 years ago
|
||
Wow, that was painful. There was a conflict bringing it up to date. So it was
better to hand merge it.
Assignee | ||
Updated•23 years ago
|
Summary: (patch) support for printing of non-latin1 chars using of HTML->PS → [FIX]support for printing of non-latin1 chars using of HTML->PS
Target Milestone: --- → mozilla0.9.7
Updated•23 years ago
|
Summary: [FIX]support for printing of non-latin1 chars using of HTML->PS → [FIX][ps]support for printing of non-latin1 chars using of HTML->PS
Assignee | ||
Updated•23 years ago
|
Target Milestone: mozilla0.9.7 → mozilla0.9.8
Comment 8•23 years ago
|
||
Don, any chance of re-reviewing the merged patch?
Comment 9•23 years ago
|
||
r=dcone.. looks good. thanks.
Comment 10•23 years ago
|
||
Comment on attachment 59570 [details] [diff] [review]
updated patch
rs=brendan@mozilla.org if it compiles. Recording dcone's r= with the patch
manager (please use it, it helps queryability).
/be
Attachment #59570 -
Flags: superreview+
Attachment #59570 -
Flags: review+
Comment 11•23 years ago
|
||
Rod, could you check this in before it rots again?
Assignee | ||
Comment 12•23 years ago
|
||
fixed
Status: ASSIGNED → RESOLVED
Closed: 23 years ago
Resolution: --- → FIXED
Reporter | ||
Comment 13•23 years ago
|
||
Guys, please see #100324 too - it contains a small patch tha affects only cyrillic. Majority of Mozillas in russia and ExUSSR are running with this patch applied and it helps alot. In the worst case it will affect only cyrillic documents (in russian, ukrainian, bulgarian, etc). Could you please apply it too? Thanks in advance.
Comment 14•23 years ago
|
||
Gena, please verify and mark verified fixed.
Reporter | ||
Comment 15•23 years ago
|
||
It works fine - thank you very much! Please don't forget to apply #100324 for us cyrillic users.
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•