Re: [PATCH OSSTEST v2 3/4] Add support for selecting resources based on their properties.

[Ian Campbell <ian.campbell@citrix.com>]

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.jackson@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.