On Tue, Nov 15, 2011 at 3:18 PM, Christophe Fergeau <cfergeau@xxxxxxxxxx> wrote: > What do you mean exactly? Having a warning in genum_get_nick and drop the > != NULL test? Or only to add the warning in genum_get_nick, and not add a > warning here? I can easily add a g_warn_if_reached() in genum_get_nick Yeah, I meant putting g_warn_if_reached() in genum_get_nick(). 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. 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); cheers -- Marc-André Lureau -- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list