On Tue, Nov 15, 2011 at 2:37 PM, Christophe Fergeau <cferg...@redhat.com> wrote:
> g_return_if_fail(klock != NULL);

I think better is:
g_return_if_fail(GVIR_IS_CONFIG_CLOCK(klock));

> libxml2 provides http://xmlsoft.org/html/libxml-xmlstring.html#BAD_CAST
> which isn't really easy on the eyes. I've got more patches which are not
> ready for review yet, but which move these casts in only a few helper
> functions instead of having them everywhere. If you don't mind, I'll
> address this when all these patches have been sent, is it ok?

sure

-- 
Marc-André Lureau

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

Reply via email to