Closed Bug 12376 Opened 20 years ago Closed 20 years ago

Fix warnings in expat

Categories

(Core :: XML, defect, P3)

x86
Windows NT
defect

Tracking

()

VERIFIED FIXED

People

(Reporter: nisheeth_mozilla, Assigned: nisheeth_mozilla)

Details

(Whiteboard: 11/16: requested verification by reporter)

Attachments

(1 file)

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!  :-)
Status: NEW → ASSIGNED
Target Milestone: M14
Accepting bug and setting milestone to M14 as this isn't a beta blocker...
Status: ASSIGNED → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED
These warnings are fixed now.
Nisheeth: This looks like internal code fixes. Can you verify it fixed for me?
Thanks
Whiteboard: 11/16: requested verification by reporter
Status: RESOLVED → VERIFIED
Marking bug verified.
You need to log in before you can comment on or make changes to this bug.