From mboxrd@z Thu Jan  1 00:00:00 1970
From: Razvan Cojocaru <rzvncj@gmail.com>
Subject: Xen tools (libxs, xenstore) and C++
Date: Tue, 22 Jan 2013 17:16:56 +0200
Message-ID: <50FEAD68.1000604@gmail.com>
Mime-Version: 1.0
Content-Type: text/plain; charset="windows-1252"; Format="flowed"
Content-Transfer-Encoding: quoted-printable
Return-path: <xen-devel-bounces@lists.xen.org>
List-Unsubscribe: <http://lists.xen.org/cgi-bin/mailman/options/xen-devel>,
	<mailto:xen-devel-request@lists.xen.org?subject=unsubscribe>
List-Post: <mailto:xen-devel@lists.xen.org>
List-Help: <mailto:xen-devel-request@lists.xen.org?subject=help>
List-Subscribe: <http://lists.xen.org/cgi-bin/mailman/listinfo/xen-devel>,
	<mailto:xen-devel-request@lists.xen.org?subject=subscribe>
Sender: xen-devel-bounces@lists.xen.org
Errors-To: xen-devel-bounces@lists.xen.org
To: "xen-devel@lists.xen.org" <xen-devel@lists.xen.org>
List-Id: xen-devel@lists.xenproject.org

Hello,

I'd like to use xenctrl.h and a handful of other headers in a C++ =

application. The headers I'm interested now are xenctrl.h, =

xen/hvm/save.h, xen/mem_event.h and xenstore.h. Unfortunately, they're =

not C++-friendly.

Here's why:

1. None of the headers enclose their contents in:

#ifdef __cplusplus
extern "C" {
#endif

/* contents here */

#ifdef __cplusplus
}
#endif

2. xen/arch-x86/hvm/save.h uses the C++ keyword 'new' as a parameter =

name (quite a few times).

3. C++ (being type-safer) frowns upon such assignments as:

struct hvm_hw_cpu *newcpu=3Dh; // h is void*

which should be explicitly written as:

struct hvm_hw_cpu *newcpu=3D(struct hvm_hw_cpu *)h;

4. xenctrl.h typedefs "enum xc_error_code xc_error_code;" _before_ "enum =

xc_error_code { /* ... */ }" has been defined, which C++ doesn't allow.

I'm also getting:

/usr/include/xen/mem_event.h:71:1: error: expected =91;=92 after union =

definition
/usr/include/xen/mem_event.h:71:1: error: expected =91:=92 before =91;=92 t=
oken

where the DEFINE_RING_TYPES(mem_event, mem_event_request_t, =

mem_event_response_t); macro is being expanded in mem_event.h.

Are there plans to have the userspace libraries be friendlier to C++?

Thanks,
Razvan Cojocaru