2 Nov
2007
2 Nov
'07
7:43 p.m.
KR> This looks good. Only minor issue is that you don't return the KR> status, but I the other cu_<> functions behave the same way. +1 I don't return a CMPIStatus, but I do return a pointer to the caller's cu_property struct that didn't pass the check, which they can use to set a CMPIStatus if they wish (see my subsequent patches to libvirt-cim). I did it this way to avoid having to pass around a CMPIBroker, and so that the caller has more control over the status value. -- Dan Smith IBM Linux Technology Center Open Hypervisor Team email: danms@us.ibm.com