[libvirt] [libvirt-glib 4/4] Implement gvir_config_domain_get_devices

Christophe Fergeau cfergeau at redhat.com
Wed Jan 4 06:50:18 UTC 2012


On Tue, Jan 03, 2012 at 10:08:10PM +0000, Daniel P. Berrange wrote:
> On Tue, Jan 03, 2012 at 11:38:15PM +0200, Zeeshan Ali (Khattak) wrote:
> > On Tue, Jan 3, 2012 at 12:11 PM, Christophe Fergeau <cfergeau at redhat.com> wrote:
> > > There are some devices which we are not able to parse yet, in these
> > > cases a runtime warning is printed to tell what's going on.
> > 
> >   Actually I see that most of the device types are not being handled.
> > I would suggest we implement all the cases before merging this or
> > otherwise we'll easily forget about this. Yes the warning won't let us
> > forget but I don't think emitting a warning for each unimplemented
> > type (by the library itself) is a good idea since these warnings are
> > visible to users/app developers and they are useless for them.
> 
> Agreed, the way we deal with the XML docs ensures we preserve
> any parts of the DOM we don't know about. IMHO we should only
> emit a warning if there is a bit of the DOM we don't expect
> and we are loosing that data during the requested manipulation.

I'll change these warnings to g_debug then

Christophe
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20120104/22379b6f/attachment-0001.sig>


More information about the libvir-list mailing list