Closed
Bug 863025
Opened 12 years ago
Closed 12 years ago
nsUniversalDetector overlooks the UTF-16 BOM if the input contains only the BOM
Categories
(Core :: Internationalization, defect)
Core
Internationalization
Tracking
()
RESOLVED
FIXED
mozilla23
People
(Reporter: emk, Assigned: emk)
References
Details
Attachments
(2 files)
2.14 KB,
patch
|
smontagu
:
review+
bajaj
:
approval-mozilla-aurora+
bajaj
:
approval-mozilla-beta+
|
Details | Diff | Splinter Review |
961 bytes,
patch
|
standard8
:
review+
|
Details | Diff | Splinter Review |
See bug 862757.
Assignee | ||
Comment 1•12 years ago
|
||
Also fixed the crazy indent.
Updated•12 years ago
|
Attachment #738720 -
Flags: review?(smontagu) → review+
Assignee | ||
Comment 2•12 years ago
|
||
Flags: in-testsuite?
Whiteboard: [leave open]
Target Milestone: --- → mozilla23
Assignee | ||
Comment 3•12 years ago
|
||
Attachment #738998 -
Flags: review?(mbanner)
Comment 4•12 years ago
|
||
Assignee | ||
Comment 5•12 years ago
|
||
Comment on attachment 738998 [details] [diff] [review]
Reenable the UTF-16BE test for bug 862757
[Approval Request Comment]
Bug caused by (feature/regressing bug #): bug 860180
User impact if declined: Thunderbird will not read some attachments correctly.
Testing completed (on m-c, etc.): on c-c.
Risk to taking this patch (and alternatives if risky): low
String or IDL/UUID changes made by this patch: none
Attachment #738998 -
Flags: approval-mozilla-beta?
Attachment #738998 -
Flags: approval-mozilla-aurora?
Assignee | ||
Comment 6•12 years ago
|
||
Comment on attachment 738998 [details] [diff] [review]
Reenable the UTF-16BE test for bug 862757
Sorry, wrong patch.
Attachment #738998 -
Flags: approval-mozilla-beta?
Attachment #738998 -
Flags: approval-mozilla-aurora?
Assignee | ||
Comment 7•12 years ago
|
||
Comment on attachment 738720 [details] [diff] [review]
patch
[Approval Request Comment]
Bug caused by (feature/regressing bug #): bug 860180
User impact if declined: Thunderbird will not read some attachments correctly.
Testing completed (on m-c, etc.): on c-c.
Risk to taking this patch (and alternatives if risky): low
String or IDL/UUID changes made by this patch: none
Attachment #738720 -
Flags: approval-mozilla-beta?
Attachment #738720 -
Flags: approval-mozilla-aurora?
Updated•12 years ago
|
Attachment #738720 -
Flags: approval-mozilla-beta?
Attachment #738720 -
Flags: approval-mozilla-beta+
Attachment #738720 -
Flags: approval-mozilla-aurora?
Attachment #738720 -
Flags: approval-mozilla-aurora+
Comment 8•12 years ago
|
||
Comment 9•12 years ago
|
||
Comment on attachment 738998 [details] [diff] [review]
Reenable the UTF-16BE test for bug 862757
Sorry for the delay, looks good, thanks.
Attachment #738998 -
Flags: review?(mbanner) → review+
Assignee | ||
Comment 10•12 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Whiteboard: [leave open]
You need to log in
before you can comment on or make changes to this bug.
Description
•