[Libvirt-cim] [PATCH] [TEST] Fixing the 02_hostsystem_to_rasd.py tc

Kaitlin Rupert kaitlin at linux.vnet.ibm.com
Tue Apr 8 20:03:15 UTC 2008


>  def main():
>      options = main.options
>      log_param()
> +    global vsxml 

It's probably a good idea to pass vsxml in as an argument than to 
declare it as a global.  It can be unclear when the value gets set; 
passing the value in as an argument helps cleanup up this confusion.

-- 
Kaitlin Rupert
IBM Linux Technology Center
kaitlin at linux.vnet.ibm.com




More information about the Libvirt-cim mailing list