HE> +static CMPIStatus _get_devices(const CMPIBroker *broker, HE> + const CMPIObjectPath *reference, HE> + const virDomainPtr dom, HE> + const uint16_t type, HE> + struct inst_list *list) HE> +{ HE> + CMPIStatus s = {CMPI_RC_OK, NULL}; HE> + struct virt_device *devs = NULL; HE> int count; HE> int i; HE> - struct virt_device *devs = NULL; Please don't rearrange lines as part of a patch that does more than just cleanups. HE> count = get_devices(dom, &devs, type); HE> if (count <= 0) HE> goto out; HE> - HE> + You're adding leading whitespace here. HE> for (i = 0; i < count; i++) { HE> CMPIInstance *dev = NULL; HE> - HE> - dev = device_instance(broker, &devs[i], dom, ns); HE> + HE> + dev = device_instance(broker, &devs[i], dom, NAMESPACE(reference)); HE> if (dev) HE> inst_list_add(list, dev); HE> - HE> + ...and here. -- Dan Smith IBM Linux Technology Center Open Hypervisor Team email: danms@us.ibm.com