Re: [PATCH] Wayland: Add missing extern "C"

2012-05-24 Thread Andreas Ericsson
On 05/23/2012 08:14 AM, yan.w...@linux.intel.com wrote: > From: Yan Wang > > Add extern "C" macro definition for C++. This plays merry hell with most auto-indentation programs and some editors though. May I suggest something like this instead? #ifdef __cplusplus #define DECL_START extern "C" { #

Re: [PATCH] Wayland: Add missing extern "C"

2012-05-23 Thread Kristian Høgsberg
On Wed, May 23, 2012 at 02:14:30PM +0800, yan.w...@linux.intel.com wrote: > From: Yan Wang > > Add extern "C" macro definition for C++. That's better, thanks. Kristian > --- > cursor/wayland-cursor.h |8 > 1 files changed, 8 insertions(+), 0 deletions(-) > > diff --git a/cursor/w

Re: [PATCH] Wayland: Add missing extern "C"

2012-05-23 Thread Juan Zhao
On 05/23/2012 02:14 PM, yan.w...@linux.intel.com wrote: From: Yan Wang Add extern "C" macro definition for C++. It would be helpful, when some C++ applications include this header file, like chromium. Thanks, Juan --- cursor/wayland-cursor.h |8 1 files changed, 8 inserti

[PATCH] Wayland: Add missing extern "C"

2012-05-22 Thread yan . wang
From: Yan Wang Add extern "C" macro definition for C++. --- cursor/wayland-cursor.h |8 1 files changed, 8 insertions(+), 0 deletions(-) diff --git a/cursor/wayland-cursor.h b/cursor/wayland-cursor.h index 154cd84..f8bf8d1 100644 --- a/cursor/wayland-cursor.h +++ b/cursor/wayland-c