Closed
Bug 178680
Opened 23 years ago
Closed 23 years ago
Improve charset check in SetMailCharacterSetToMsgWindow
Categories
(MailNews Core :: MIME, defect)
MailNews Core
MIME
Tracking
(Not tracked)
RESOLVED
FIXED
mozilla1.3alpha
People
(Reporter: nhottanscp, Assigned: nhottanscp)
Details
Attachments
(1 file)
|
6.68 KB,
patch
|
bugzilla
:
review+
Bienvenu
:
superreview+
|
Details | Diff | Splinter Review |
Since there is no menu item for us-ascii, we do special case handling to map it
to ISO-8859-1. Currently, the callers of SetMailCharacterSetToMsgWindow() do
that but we can do that inside the function instead.
|
Assignee | |
Updated•23 years ago
|
Status: NEW → ASSIGNED
QA Contact: yulian → nhotta
Target Milestone: --- → mozilla1.3alpha
|
|
Assignee | |
Comment 1•23 years ago
|
||
|
|
Assignee | |
Updated•23 years ago
|
Attachment #106013 -
Flags: review?(ducarroz)
|
||
Comment 2•23 years ago
|
||
Comment on attachment 106013 [details] [diff] [review]
Changed SetMailCharacterSet, moved us-ascii check from the callers to the function.
good cleanup. R=ducarroz
Attachment #106013 -
Flags: review?(ducarroz) → review+
|
|
Assignee | |
Updated•23 years ago
|
Attachment #106013 -
Flags: superreview?(bienvenu)
|
|
||
Comment 3•23 years ago
|
||
Comment on attachment 106013 [details] [diff] [review]
Changed SetMailCharacterSet, moved us-ascii check from the callers to the function.
looks good, sr=bienvenu
Attachment #106013 -
Flags: superreview?(bienvenu) → superreview+
|
|
Assignee | |
Comment 4•23 years ago
|
||
checked in to the trunk
Status: ASSIGNED → RESOLVED
Closed: 23 years ago
Resolution: --- → FIXED
|
||
Updated•21 years ago
|
Product: MailNews → Core
|
||
Updated•18 years ago
|
Product: Core → MailNews Core
You need to log in
before you can comment on or make changes to this bug.
Description
•