On Mon, Apr 13, 2015 at 07:41:12AM +0200, Peter Krempa wrote: > On Fri, Apr 10, 2015 at 14:58:55 +0200, Ján Tomko wrote: > > diff --git a/src/conf/storage_feature_conf.c > > b/src/conf/storage_feature_conf.c > > new file mode 100644 > > index 0000000..77e6406 > > --- /dev/null > > +++ b/src/conf/storage_feature_conf.c > > @@ -0,0 +1,62 @@ > > +/* > > + * storage_feature_conf.c: config handling for storage file features > > + * > > + * Copyright: Red Hat, Inc > > + * > > + * LGLPv2.1+ > > + */ > > I like this compact header, but I'm not sure if the rest of upstream has > the same opinion. > >
It was just a placeholder and I forgot to update it before sending the
series.
If we want to switch to a more compact header, I think it should be done
consistently all over the codebase.
I have squashed in a copy of the header from another conf file.
> > + const char *xpath,
> > + char **compat,
> > + virBitmapPtr *features)
> > +{
> > + xmlNodePtr *nodes = NULL;
> > + char *feat_xpath = NULL;
> > + size_t i;
> > + int n;
> > + int ret = -1;
> > +
> > + if (!virXPathNode(xpath, ctxt))
> > + return 0;
> > +
> > + if (!*compat && VIR_STRDUP(*compat, "1.1") < 0)
> > + return -1;
>
> Is there a specific reason that you check whether the compat string is
> not assigned previously?
>
Yes, the user might have specified a different compat level.
Jan
signature.asc
Description: Digital signature
-- libvir-list mailing list [email protected] https://www.redhat.com/mailman/listinfo/libvir-list
