[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH v2] RFC: Automatically check xen's public headers for C++ pitfalls.



On Thu, 2015-03-05 at 12:25 +0100, Tim Deegan wrote:
> At 17:28 +0100 on 26 Feb (1424968122), Tim Deegan wrote:
> > BTW, ring.h is the only instance of that, so the extra diff to clear
> > that up too is pretty small (see below).
> > 
> > Not sure what people think about that though - it might be
> > quite a PITA for downstream users of it, though they ought really to
> > be using local copies so they can update in a controlled way.
> 
> So I've seen four responses in favour of just renaming the field
> (Andrew Cooper, Razvan Cojocaru, Don Slutz and David Vrabel) and one
> in favour of #ifdeffing it so it's only renamed in C++ (Jan Beulich).
> I really don't like adding more #ifdefs to an already hard-to-read
> file; I'd rather just rename the field, or else leaving it alone and
> letting C++ users carry the fixup in their own code.
> 
> CC'ing the other "THE REST" maintainers for their opinions.

Rather than ifdefs for C++, don't we need them based on
__XEN_INTERFACE_VERSION__?

I don't much like that (I'd rather just change the name) but I think
that's what we are supposed to do here.



_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.