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: [RFC] Fix PR38892: "--enable-libgcj-debug" breaks bootstrap.

From: Tom Tromey <tromey at redhat dot com>
To: Dave Korn <dave dot korn dot cygwin at googlemail dot com>
Cc: java-patches at gcc dot gnu dot org
Date: Wed, 22 Apr 2009 19:04:59 -0600
Subject: Re: [RFC] Fix PR38892: "--enable-libgcj-debug" breaks bootstrap.
References: <49EF6F29.2000601@gmail.com> <m3prf4ti8i.fsf@fleche.redhat.com> <49EFA476.5080305@gmail.com>
Reply-to: Tom Tromey <tromey at redhat dot com>

>>>>> "Dave" == Dave Korn <dave.korn.cygwin@googlemail.com> writes:

Tom> Yeah, just GCJ_DEBUG would be fine.

Dave>   Do you care about the "__" prefix?  I wasn't sure if maybe it
Dave> might end up in the user's namespace (do any of the
Dave> libjava/include files end up getting installed?), so I already
Dave> did it the "__GCJ_DEBUG" way; I'll respin it if it matters.

A couple of the headers are installed, but this symbol isn't used by
any of them.  So, without the underscores is ok.  It doesn't matter
hugely to me, though, so if you already have the patch, the underscore
variant is fine too.

Tom

Follow-Ups:
- Re: [RFC] Fix PR38892: "--enable-libgcj-debug" breaks bootstrap.
  - From: Dave Korn

References:
- [RFC] Fix PR38892: "--enable-libgcj-debug" breaks bootstrap.
  - From: Dave Korn
- Re: [RFC] Fix PR38892: "--enable-libgcj-debug" breaks bootstrap.
  - From: Tom Tromey
- Re: [RFC] Fix PR38892: "--enable-libgcj-debug" breaks bootstrap.
  - From: Dave Korn

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