bugzilla.mozilla.org has resumed normal operation. Attachments prior to 2014 will be unavailable for a few days. This is tracked in Bug 1475801.
Please report any other irregularities here.

Use |nsISaveAsCharset::attr_EntityAfterCharsetConv| instead of |nsISaveAsCharset::attr_EntityBeforeCharsetConv| in GTK+/Xlib toolkits

RESOLVED FIXED in mozilla1.0

Status

()

Core
XUL
--
enhancement
RESOLVED FIXED
17 years ago
17 years ago

People

(Reporter: Roland Mainz, Assigned: Roland Mainz)

Tracking

Trunk
mozilla1.0
All
Linux
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

(Assignee)

Description

17 years ago
[rbs wrote in bug 33498 comment #9]
> In the process, I noted that other platforms are using 
> nsISaveAsCharset::attr_EntityBeforeCharsetConv. I think what people want is
> nsISaveAsCharset::attr_EntityAfterCharsetConv. 
>
> The former will convert to entities _before_ doing the transliteration, 
> thereby possibly converting some glyphs that could be transliterated. The 
> latter will transliterate, and then the glyphs that couldn't be transliterated 
> will be converted to their entity representation if possible or to question 
> marks if not possible. Finally, just in case, I also made sure that the Win32 
> routines that expect the converted result are 0-length safe. Not sure about 
> other platforms here.

RFE: Do it!
(Assignee)

Comment 1

17 years ago
Created attachment 72147 [details] [diff] [review]
Patch for 2002-02-28-08-trunk
(Assignee)

Comment 2

17 years ago
Requesting r=/sr=/a= ...
Status: NEW → ASSIGNED
Keywords: patch, review
Target Milestone: --- → mozilla1.0
(Assignee)

Updated

17 years ago
Depends on: 85417
(Assignee)

Updated

17 years ago
No longer depends on: 85417
(Assignee)

Updated

17 years ago
Depends on: 85417

Comment 3

17 years ago
Comment on attachment 72147 [details] [diff] [review]
Patch for 2002-02-28-08-trunk

r=rbs
Attachment #72147 - Flags: review+
(Assignee)

Comment 4

17 years ago
rbs:
Do we have any testcase to see the "before patch <--> after patch"-effect ?

Comment 5

17 years ago
I would like to (strongly) recommend we have Frank Tang, the converter expert,
look at this.

Comment 6

17 years ago
>rbs:
>Do we have any testcase to see the "before patch <--> after patch"-effect ?

No. But you can easily make one that will exercise the code as per my above 
description. See transliterate.properties for a sample of characters that can be
transliterated. To test if the code 0-length safe, you could try some of the
empty characters, eg., <span>&zwsp;&InvisibleComma;...etc</span>  (while making
sure that you have no fonts with glyphs for them...).

Comment 7

17 years ago
I think rbs rbs is right, but I don't really think this change will have any
effect since probably no characters which can be covered by "ISO-8859-1" will
ever hit the code here unless there are no ISO-8859-1 font installed on the
system, which is very very very unlikely.

I think we should not check in this patch now since:
1. there are no obvious gain (no functionality gain or loss with or without this
patch)
and 2. there are risk (tiny but not zero) if we take this patch. 

I think there are no reason to take a zero gain tiny risk patch in this stage.
why don'w we save it till post m1.0?

Comment 8

17 years ago
Because it frees your mind, reduce bug counts, and allows concentrating fully on
other important things.
(Assignee)

Comment 9

17 years ago
Just wondering - why are we using "ISO-8859-1" and not $LANG or $LC_CTYPE (which
ever one is the correct one) - and fall back to "ISO-8859-1" on demand ?

Comment 10

17 years ago
waterson asked similarly in bug 33498 comment #18, reply: bug 33498 comment #18
(Assignee)

Comment 11

17 years ago
ftang:
Windows works with this since eternity... I think Windows and X11(=GTK,Xlib)
should do the same.

Unless you see major problems I would like to get sr= and a= for trunk, please
...
(Assignee)

Comment 12

17 years ago
Requesting sr= ...

Comment 13

17 years ago
Roland: could you point out some specific cases for this?

Without a strong case for doing this I would recommend we avoid the risk
till after m1.0.
(Assignee)

Comment 14

17 years ago
Brian Stell wrote:
> Roland: could you point out some specific cases for this?

Mhhh... not really... 

... rbs - do you have one ?

Comment 15

17 years ago
I just happen to see a checkin comment on a related issue:
http://bonsai.mozilla.org/cvsquery.cgi?module=MozillaTinderboxAll&branch=HEAD&cvsroot=/cvsroot&date=explicit&mindate=1018051860&maxdate=1018052880&who=nhotta%25netscape.com

(The trivial patch for this bug has been sitting here with a r= all along, it is
not really the best use of my time to drag over such intelligible matters on and
on.)

Comment 16

17 years ago
Comment on attachment 72147 [details] [diff] [review]
Patch for 2002-02-28-08-trunk

rs=attinasi
Attachment #72147 - Flags: superreview+
(Assignee)

Comment 17

17 years ago
Created attachment 79100 [details] [diff] [review]
Patch for 2002-04-09-08-trunk (merge to tip only)
Attachment #72147 - Attachment is obsolete: true
(Assignee)

Comment 18

17 years ago
Checked into "trunk" by timeless
(http://bonsai.mozilla.org/cvsquery.cgi?module=MozillaTinderboxAll&branch=HEAD&cvsroot=/cvsroot&date=explicit&mindate=1018746780&maxdate=1018747800&who=timeless%25mac.com)
...

----

rbs:
Should I keep this bug open and wait a while and then request a= for the
1.0-branch again (assuming noone files a bug again this... :) ?

Comment 19

17 years ago
Why bother that much for something that minor? Just close the bug and move on,
and the patch will eventually make its way to another Mozilla release.
(Assignee)

Comment 20

17 years ago
OK, marking bug as FIXED...
Status: ASSIGNED → RESOLVED
Last Resolved: 17 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.