[Expat-bugs] [ expat-Bugs-676844 ] expat.h compile error: enum XML_Status

[SourceForge.net]

Bugs item #676844, was opened at 2003-01-29 07:37
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=110127&aid=676844&group_id=10127

Category: Build control
Group: None
Status: Open
Resolution: Fixed
Priority: 3
Submitted By: Nobody/Anonymous (nobody)
Assigned to: Fred L. Drake, Jr. (fdrake)
Summary: expat.h compile error: enum XML_Status

Initial Comment:
c++ -DHAVE_CONFIG_H -I. -I. -I../../autocfg -g -O2 -c
context.cpp  -fPIC -DPIC -o .libs/context.lo
In file included from parser.h:45,
                 from guard.h:143,
                 from context.cpp:45:
/usr/include/expat.h:657: use of enum `XML_Status'
without previous declaration
/usr/include/expat.h:736: multiple definition of `enum
XML_Status'


when building sablotron.

Hand editing the file to place the def of enum
XML_Status before any references to it fixes the problem.

----------------------------------------------------------------------

Comment By: Jacob Levy (jyljyljyl)
Date: 2003-02-28 17:35

Message:
Logged In: YES 
user_id=63723

This makes Expat 1.95.6 unusable for people who create
libraries that depend on Expat but don't include their own
copy of Expat. Sure, I can edit expat.h and fix it, but my
users should not be expected to do that.

For that reason I'm staying with Expat 1.95.5 until this
problem is fixed. It'd be really nice if you could make
Expat 1.95.7 soon..

In my case GCC 2.95.2 and VC++ 6.0 are complaining.


----------------------------------------------------------------------

Comment By: Melvyn Sopacua (nyvlem)
Date: 2003-02-13 23:56

Message:
Logged In: YES 
user_id=212431

Yes, that works.

----------------------------------------------------------------------

Comment By: Karl Waclawek (kwaclaw)
Date: 2003-02-06 12:47

Message:
Logged In: YES 
user_id=290026

But current CVS works for you, right?

----------------------------------------------------------------------

Comment By: Melvyn Sopacua (nyvlem)
Date: 2003-02-06 12:17

Message:
Logged In: YES 
user_id=212431

> So far only gcc3.2 has complained.

Nope:
/usr/local/include/expat.h:657: use of enum `XML_Status' 
without previous declaration
/usr/local/include/expat.h:736: multiple definition of `enum 
XML_Status'
gmake[2]: *** [context.lo] Error 1
gmake[2]: Leaving directory 
`/home/mdev/cvs/sablot/src/engine'
gmake[1]: *** [all-recursive] Error 1
gmake[1]: Leaving directory `/home/mdev/cvs/sablot/src'
gmake: *** [all-recursive] Error 1

$ gcc --version
2.95.3

----------------------------------------------------------------------

Comment By: Karl Waclawek (kwaclaw)
Date: 2003-01-31 06:43

Message:
Logged In: YES 
user_id=290026

It *is* fixed in CVS. Are you sure you checked out the 
right version, which is expat.h 1.51?

----------------------------------------------------------------------

Comment By: Nobody/Anonymous (nobody)
Date: 2003-01-31 02:34

Message:
Logged In: NO 

I just got the same error, already fixed it. But don't 
understand why it isn't fixed in CVS ?

----------------------------------------------------------------------

Comment By: Fred L. Drake, Jr. (fdrake)
Date: 2003-01-29 10:44

Message:
Logged In: YES 
user_id=3066

I've not checked the C89 standard yet, but Expat 1.95.6 is 
certainly dodgy in this case.  ;-(  The first draft of the C spec I 
found online certainly seemed to imply that any use of an 
incomplete enum is not allowed; I'm not likely to go out and 
buy a copy of the final spec to check further.  ;-)

As noted, this has been fixed in CVS.  Fixed the summary to 
better indicate what this report is about, and lowered the 
priority to get it out of the way for maintainers.

----------------------------------------------------------------------

Comment By: Karl Waclawek (kwaclaw)
Date: 2003-01-29 07:51

Message:
Logged In: YES 
user_id=290026

So far only gcc3.2 has complained.
Not sure if this is a bug, since most compilers accept it,
but it has been fixed in CVS already anyway.

Set resolution status to fixed, but leave open.
There may be other users who would report this as a bug
and I want them to see the open report instead of 
having duplicates created.

Assigned to Fred - since he may know more about
whether this truly is a bug.

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=110127&aid=676844&group_id=10127