Closed
Bug 250119
Opened 21 years ago
Closed 21 years ago
xmlextras: ParseFromString/ParseFromStream should use AUTF8Strings
Categories
(Core :: XML, defect)
Tracking
()
RESOLVED
FIXED
People
(Reporter: vlad, Assigned: vlad)
Details
Attachments
(2 files)
|
8.51 KB,
patch
|
shaver
:
review+
shaver
:
superreview+
|
Details | Diff | Splinter Review |
|
10.31 KB,
patch
|
Details | Diff | Splinter Review |
the parse bits should use AUTF8String to avoid high-bit string lossage from js;
also adds ParseFromUTF8String.
|
Assignee | |
Comment 1•21 years ago
|
||
|
|
Assignee | |
Comment 2•21 years ago
|
||
01:59 <jst> r=jst
01:59 <jst> you want that in a bug?
02:00 <vlad> sure, 250119
02:00 <jst> hmm, no patch yet... feel free to add r=jst yourself once you've got
a patch
|
||
Comment 3•21 years ago
|
||
Comment on attachment 152483 [details] [diff] [review]
xmlextras-stringparse-1.patch
Marking my and jst's approval.
Attachment #152483 -
Flags: superreview+
Attachment #152483 -
Flags: review+
|
|
Assignee | |
Comment 4•21 years ago
|
||
in on trunk and aviary.
Status: NEW → RESOLVED
Closed: 21 years ago
Resolution: --- → FIXED
|
|
Assignee | |
Comment 5•21 years ago
|
||
Attaching patch that went in for posterity's sake; combo of this patch and
first iteration of the patch in bug 244078.
|
|
Assignee | |
Comment 6•20 years ago
|
||
landed/synced on 1.7
You need to log in
before you can comment on or make changes to this bug.
Description
•