Closed
Bug 274964
Opened 21 years ago
Closed 21 years ago
Switch to Expat's xml declaration parsing
Categories
(Core :: XML, defect)
Core
XML
Tracking
()
RESOLVED
FIXED
People
(Reporter: peterv, Assigned: peterv)
Details
Attachments
(1 file)
|
16.78 KB,
patch
|
jst
:
review+
jst
:
superreview+
|
Details | Diff | Splinter Review |
|
Assignee | |
Comment 1•21 years ago
|
||
Attachment #168885 -
Flags: superreview?(jst)
Attachment #168885 -
Flags: review?(jst)
|
||
Comment 2•21 years ago
|
||
Comment on attachment 168885 [details] [diff] [review]
v1
Nice! r+sr=jst
Attachment #168885 -
Flags: superreview?(jst)
Attachment #168885 -
Flags: superreview+
Attachment #168885 -
Flags: review?(jst)
Attachment #168885 -
Flags: review+
|
|
Assignee | |
Updated•21 years ago
|
Status: NEW → RESOLVED
Closed: 21 years ago
Resolution: --- → FIXED
This checkin breaks Solaris build, because the 3rd argument of
nsXMLContentSink::HandleXMLDeclaration is a |const PRInt32|
(http://lxr.mozilla.org/seamonkey/source/content/xml/document/src/nsXMLContentSink.cpp#1235)
but in idl-generated nsIExpatSink.h, its just a |PRInt32|. This mismatch causes
libgklayout module failed to load on Solaris, non-debug build.
You need to log in
before you can comment on or make changes to this bug.
Description
•