Hey, On Tue, Nov 15, 2011 at 03:33:45PM +0100, Marc-André Lureau wrote: > However, thinking a bit more about it, genum_get_nick() shouldn't warn > if given a value that has no associated nick, because enums shouldn't > be limited to their associated name, and those value would be > perfectly valid, but should return NULL and that's it. For now I have added a warning to genum_get_nick since we shouldn't get an enum with no associated strings. If this happens, this means something unexpected happened, so it's better to warn about it. We can change this when we have a valid use case for this. > > However, siliently not doing anything is misleading. So instead of > > if (type_str != NULL) > xmlNodeSetContent(node, (xmlChar*)type_str); > > I would do (or similar): > > g_return_if_fail (type_str != NULL); > xmlNodeSetContent(node, (xmlChar*)type_str); And I did that change too. Christophe
Attachment:
pgpUNP2RcJF5h.pgp
Description: PGP signature
-- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list