Re: [libvirt-glib 12/37] Add more GVirConfigOs setters

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi

Please merge with previous patch.

> +    if (enable)
> +        xmlNewProp(node, (xmlChar*)"enable", (xmlChar*)"yes");
> +    else
> +        xmlNewProp(node, (xmlChar*)"enable", (xmlChar*)"no");
> +}

might be useful to have use a gchar * gvir_config_yes_no (bool condition)

> +        if (strcmp("boot", (char *)node->name) == 0) {

It could use the safer g_strcmp0.


-- 
Marc-André Lureau

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list



[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]