[virt-tools-list] [virt-manager PATCH] gfxdetails: check settype in set_dev

Cole Robinson crobinso at redhat.com
Thu Jan 30 13:33:22 UTC 2014


On 01/29/2014 11:43 PM, Chen Hanxiao wrote:
> From: Chen Hanxiao <chenhanxiao at cn.fujitsu.com>
> 
> commit  64b2d0c6b7804f3501bfd8b657346776a8849879
> forget to check settype and pylint complains this
> as Unused variable.
> This patch will fix this.
> 
> Signed-off-by: Chen Hanxiao <chenhanxiao at cn.fujitsu.com>
> ---
>  virtManager/gfxdetails.py | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/virtManager/gfxdetails.py b/virtManager/gfxdetails.py
> index 28ca332..2bc6c3a 100644
> --- a/virtManager/gfxdetails.py
> +++ b/virtManager/gfxdetails.py
> @@ -199,7 +199,8 @@ class vmmGraphicsDetails(vmmGObjectUI):
>          if is_other:
>              settype = gfx.pretty_type_simple(gtype)
>  
> -        uiutil.set_combo_entry(self.widget("graphics-type"), gtype, 0)
> +        if settype:
> +            uiutil.set_combo_entry(self.widget("graphics-type"), gtype, 0)
>          return title
>  
>  
> 

I pushed a different fix yesterday

- Cole




More information about the virt-tools-list mailing list