Patch: Include platform.h in gnu/gcj/runtime/natStackTrace.cc

Andrew Haley aph@redhat.com
Sun Jan 26 15:31:00 GMT 2003


Ranjit Mathew writes:
 > Andrew Haley wrote:
 > > For example: say you read your program source and discover that some
 > > code is conditional on HAVE_BACKTRACE, so you look in include/config.h
 > > and discover that variable isn't set.  However, it *is* set, whatever
 > > config.h says, just somewhere else.  And because you have two
 > > contradictory defintions of HAVE_BACKTRACE the result depends on what
 > > order platform.h and config.h are included.  ARgh!
 > 
 > I agree totally with what you say and the configure.in thingy is what
 > I had done on my first attempt. However, you must realise that we also
 > need the prototype for backtrace( ) which doesn't belong to config.h
 > AFAICT.
 > 
 > The other choices with me were to either include platform.h or to
 > have conditional (on WIN32 being defined) code in natStackTrace.cc
 > that declares the prototype for backtrace( ).

I have no object at all do #including platform.h, but don't define
HAVE_BACKTRACE there, please!  platform.h is a good place for the
backtrace() prototype.

Andrew.



More information about the Java-patches mailing list