On 09/16/2014 05:42 AM, Lin Ma wrote:
> PCIDevice and USBDevice include respective comparison logic.
> 
> Signed-off-by: Lin Ma <[email protected]>
> ---
>  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

_______________________________________________
virt-tools-list mailing list
[email protected]
https://www.redhat.com/mailman/listinfo/virt-tools-list

Reply via email to