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 ffi.h in link.cc.

From: Andrew Pinski <pinskia at physics dot uc dot edu>
To: David Daney <ddaney at avtrex dot com>
Cc: Java Patch List <java-patches at gcc dot gnu dot org>
Date: Mon, 6 Mar 2006 17:02:54 -0500
Subject: Re: Patch: Include ffi.h in link.cc.
References: <440CAE9C.7060503@avtrex.com>

On Mar 6, 2006, at 4:50 PM, David Daney wrote:

+#ifdef USE_LIBFFI
+extern "C" {
+#include <ffi.h>
+}
+#endif
+

Is there a reason why you are wrapping the include of ffi.h with 'extern "C"' as that seems wrong and problematic. Now if ffi.h is not C++ includable, it should be fixed instead of working about it each time it is included.

-- Pinski

Follow-Ups:
- Re: Patch: Include ffi.h in link.cc.
  - From: David Daney

References:
- Patch: Include ffi.h in link.cc.
  - From: David Daney

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