[Libvirt-cim] [PATCH 1 of 3] Fix issue with VSMC enum

Kaitlin Rupert kaitlin at linux.vnet.ibm.com
Fri Nov 30 16:11:12 UTC 2007


Dan Smith wrote:
> KR> +        s = get_host_cs(_BROKER, ref, &inst);
> KR> +        if (s.rc != CMPI_RC_OK)
> KR> +                goto out;
> KR> +
> KR> +        if (cu_get_str_prop(inst, "Name", &hostname) != CMPI_RC_OK)
> KR> +                goto out;
>
> It's even cleaner this way, right? :)
>
> With the recent API change, you don't even have to free something,
> which I think is pretty nifty.
>   
Definitely much cleaner, and also the right thing to do. =)

-- 
Kaitlin Rupert
IBM Linux Technology Center
karupert at us.ibm.com





More information about the Libvirt-cim mailing list