Re: [Xen-devel] [PATCH 1 of 2] libxl_pci: rename is_assigned to is_pcidev_in_array

[Ian Jackson <Ian.Jackson@xxxxxxxxxxxxx>]

Doug Magee writes ("[PATCH 1 of 2] libxl_pci: rename is_assigned to 
is_pcidev_in_array"):
> All this function does is check to see if a device is in an array of pcidevs 
> passed by the caller.  The function name can be misleading if ever used to 
> check against a list of devices other than those assigned to a domain.

This is a good change and I have committed it, but I think it would be
good to change the name of the formal parameter to is_pcidev_in_array,
too, to "array" or "devices" or "haystack" or something.  And a
corresponding change to num_assigned.

Thanks,
Ian.

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel