This is the mail archive of the java-patches@gcc.gnu.org mailing list for the Java project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

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

From: Tom Tromey <tromey at redhat dot com>
To: Ranjit Mathew <rmathew at hotmail dot com>
Cc: java-patches at gcc dot gnu dot org
Date: 25 Jan 2003 21:26:34 -0700
Subject: Re: Patch: Include platform.h in gnu/gcj/runtime/natStackTrace.cc
References: <b0uipm$svi$2@main.gmane.org><15922.58111.160019.861642@cuddles.cambridge.redhat.com><87ptql6nrz.fsf@fleche.redhat.com><15922.59359.864092.528020@cuddles.cambridge.redhat.com><b0vljc$ut8$1@main.gmane.org>
Reply-to: tromey at redhat dot com

>>>>> "Ranjit" == Ranjit Mathew <rmathew@hotmail.com> writes:

Ranjit> I agree totally with what you say and the configure.in thingy is what
Ranjit> I had done on my first attempt. However, you must realise that we also
Ranjit> need the prototype for backtrace( ) which doesn't belong to config.h
Ranjit> AFAICT.

We can do both: move the definition of HAVE_BACKTRACE into
configure.in, and also include platform.h in natStackTrace.cc.  I know
the former isn't really touched by your original patch, but it would
be helpful if you could do it anyway.

Tom

References:
- Patch: Include platform.h in gnu/gcj/runtime/natStackTrace.cc
  - From: Ranjit Mathew
- Patch: Include platform.h in gnu/gcj/runtime/natStackTrace.cc
  - From: Andrew Haley
- Re: Patch: Include platform.h in gnu/gcj/runtime/natStackTrace.cc
  - From: Tom Tromey
- Re: Patch: Include platform.h in gnu/gcj/runtime/natStackTrace.cc
  - From: Andrew Haley
- Re: Patch: Include platform.h in gnu/gcj/runtime/natStackTrace.cc
  - From: Ranjit Mathew

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]