On Wed, 2015-09-16 at 14:50 +0100, Ian Jackson wrote:
> Ian Campbell writes ("[PATCH OSSTEST v2 3/4] Add support for selecting
> resources based on their properties."):
> > In particular for allocating hosts based on host properties.
>
> Acked-by: Ian Jackson <ian.jack...@eu.citrix.com>
>
>
> > + die "propname: $prop?" unless propname_check($prop);
> ...
> > +sub propname_check ($) {
> > + # ensure propname is valid and in the canonical WordWordWord form
> > + # (rather than, say, one of the legacy forms described in
> > + # propname_massage).
>
> As a matter of taste, I would have had propname_check throw rather
> than making it be a predicate.
I did wonder about that. I'm happy to change if you want me to.
Ian.
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel
