On Fri, Nov 11, 2011 at 04:52:11PM +0100, Marc-André Lureau wrote: > Having gvir_config_object_add_child() and > gvir_config_object_replace_child() would be more obvious (even the > internal code should be splitted) Thanks for the naming suggestions :) I'm not fully satisfied with how most of the helpers are named, so don't hesitate to suggest better names. One thing that is missing from the _replace_node name is that it's not obvious that this will create a new node if needed. But I made this change anyway, thanks! Christophe
Attachment:
pgp9jfMP2Cahd.pgp
Description: PGP signature
-- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list