[virt-tools-list] [PATCH V5 1/2] Add comparison function for NodeDevice

Cole Robinson crobinso at redhat.com
Tue Sep 16 10:54:06 UTC 2014


On 09/16/2014 05:42 AM, Lin Ma wrote:
> PCIDevice and USBDevice include respective comparison logic.
> 
> Signed-off-by: Lin Ma <lma at suse.com>
> ---
>  virtManager/details.py | 47 ++---------------------------------------
>  virtinst/nodedev.py    | 57 ++++++++++++++++++++++++++++++++++++++++++++++++++
>  2 files changed, 59 insertions(+), 45 deletions(-)
> 

You fixed the initify bit, but you missed my suggestion about

- Renaming the function to compare_to_hostdev
- Making the function nonstatic, and instead make it def
compare_to_hostdev(self, hostdev)

Please fix those bits as well

Thanks,
Cole




More information about the virt-tools-list mailing list