[libvirt] [PATCH] xen_xs: name xendConfigVersion magic numbers

Philipp Hahn hahn at univention.de
Thu Feb 2 05:38:39 UTC 2012

Hello Eric,

Am Donnerstag 02 Februar 2012 00:28:37 schrieben Sie:
> On 01/31/2012 10:08 AM, Philipp Hahn wrote:
> > +#define XEND_CONFIG_VERSION_3_0_2 1
> > +#define XEND_CONFIG_VERSION_3_0_3 2
> > +#define XEND_CONFIG_VERSION_3_0_4 3
> > +#define XEND_CONFIG_VERSION_3_1_0 4
> This fails 'make syntax-check' if you have cppi installed, due to
> inconsistent indentation.

Looks like a new tool I have to learn about.

> Also, I would have done this as an enum 
> rather than as #defines (it's slightly easier to debug enums in gdb).

I choose not not use an enum, because that value is read diretly from externel 
XenD and can have other values. It's initialized to 0 and to -1 on errors, so 
your enum is not complete.
But if your version still compiles, I'm also fine with that.

Philipp Hahn           Open Source Software Engineer      hahn at univention.de
Univention GmbH        Linux for Your Business        fon: +49 421 22 232- 0
Mary-Somerville-Str.1  D-28359 Bremen                 fax: +49 421 22 232-99
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 197 bytes
Desc: This is a digitally signed message part.
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20120202/b72ee0e2/attachment-0001.sig>

More information about the libvir-list mailing list