This is the mail archive of the
java-patches@gcc.gnu.org
mailing list for the Java project.
Re: [Patch] boehm-gc/win32_threads.c: Get configuration defines
- From: Bryce McKinlay <mckinlay at redhat dot com>
- To: Danny Smith <danny_smith_0000 at yahoo dot co dot nz>
- Cc: GCC Patches <gcc-patches at gcc dot gnu dot org>, java-patches <java-patches at gcc dot gnu dot org>
- Date: Mon, 23 Aug 2004 11:06:32 -0400
- Subject: Re: [Patch] boehm-gc/win32_threads.c: Get configuration defines
- References: <20040822212416.29306.qmail@web50903.mail.yahoo.com>
Thanks Danny. I'm going to check in this variant which should have the
same effect but work in the upstream boehm-gc as well. Let me know if
you still see any problems.
Bryce
2004-08-23 Bryce McKinlay <mckinlay@redhat.com>
* win32_threads.c: Move GC_WIN32_THREADS check after gc_priv.h include.
--- win32_threads.c 13 Aug 2004 23:05:30 -0000 1.17
+++ win32_threads.c 23 Aug 2004 15:03:50 -0000
@@ -1,6 +1,7 @@
+#include "private/gc_priv.h"
+
#if defined(GC_WIN32_THREADS)
-#include "private/gc_priv.h"
#include <windows.h>
#ifdef CYGWIN32
Danny Smith wrote:
Build of libgcj fails on mingw32 because of missing symbols from
boehm-gc/win32_threads.c. In fact, win32_threads.o is an empty file,
because the necessary configuration defines are no longer in the
makefile but in gc_config.h.
This fixes:
2004-08-23 Danny Smith <dannysith@users.sourceforge.net>
* win32_threads.c: Include "gc_config.h"
Index: win32_threads.c
===================================================================
RCS file: /cvs/gcc/gcc/boehm-gc/win32_threads.c,v
retrieving revision 1.17
diff -c -3 -p -r1.17 win32_threads.c
*** win32_threads.c 13 Aug 2004 23:05:30 -0000 1.17
--- win32_threads.c 22 Aug 2004 21:14:24 -0000
***************
*** 1,3 ****
--- 1,4 ----
+ #include "gc_config.h"
#if defined(GC_WIN32_THREADS)
#include "private/gc_priv.h".
Find local movie times and trailers on Yahoo! Movies.
http://au.movies.yahoo.com