Closed
Bug 12376
Opened 25 years ago
Closed 25 years ago
Fix warnings in expat
Categories
(Core :: XML, defect, P3)
Tracking
()
VERIFIED
FIXED
M14
People
(Reporter: nisheeth_mozilla, Assigned: nisheeth_mozilla)
Details
(Whiteboard: 11/16: requested verification by reporter)
Attachments
(1 file)
|
(deleted),
patch
|
Details | Diff | Splinter Review |
Here's what I know of warnings in mozilla/expat/* as of today:
Windows: no warnings
Linux: 2 warnings:
1:
xmlparse.c: In function `initializeEncoding':
xmlparse.c:2013: warning: comparison is always 0 due to limited range of data
type
2:
xmlparse.c: In function `initializeEncoding':
xmlparse.c:2013: warning: comparison is always 0 due to limited range of data
type
xmlparse.c: In function `processXmlDecl':
xmlparse.c:2069: warning: declaration of `s' shadows a parameter
Mac: Unknown.
I'm going to compile on the Mac and add whatever new warnings I find to this bug
report. There were some changes that went into xmldef.h that might have gotten
rid of some of the warnings that were being spewed earlier.
Once I've gotten a handle on all the warnings, I'll use Kathy's patch for fixing
them. Hang in there, Kathy! :-)
|
Assignee | |
Comment 1•25 years ago
|
||
|
|
Assignee | |
Updated•25 years ago
|
Status: NEW → ASSIGNED
Target Milestone: M14
|
|
Assignee | |
Comment 2•25 years ago
|
||
Accepting bug and setting milestone to M14 as this isn't a beta blocker...
|
|
Assignee | |
Updated•25 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 25 years ago
Resolution: --- → FIXED
|
|
Assignee | |
Comment 3•25 years ago
|
||
These warnings are fixed now.
|
||
Comment 4•25 years ago
|
||
Nisheeth: This looks like internal code fixes. Can you verify it fixed for me?
Thanks
|
|
||
Updated•25 years ago
|
Whiteboard: 11/16: requested verification by reporter
|
|
Assignee | |
Updated•25 years ago
|
Status: RESOLVED → VERIFIED
|
|
Assignee | |
Comment 5•25 years ago
|
||
Marking bug verified.
You need to log in
before you can comment on or make changes to this bug.
Description
•