GObject sounds like it has its benefits, and I'm definitely open to the idea of using it for implementation. But for now, I'd like to make sure the programmer interface itself is acceptable, since we can always change out the implementation of the library itself once the API is decided upon.
I'd go so far as to say that let's figure out API and packaging with the code as is for now, and once that's settled the implementation can be modified whenever convenient. -Arjun ----- Original Message ----- From: "Daniel P. Berrange" <[email protected]> To: "Arjun Roy" <[email protected]> Cc: "virt-tools-list" <[email protected]>, "Cole Robinson" <[email protected]>, "Hugh O. Brock" <[email protected]> Sent: Monday, February 1, 2010 4:48:09 AM GMT -08:00 US/Canada Pacific Subject: Re: libosinfo - implemented. I've only had a quick look at the API / code so far, but my first impression is that the API would majorly benefit by being ported to use GLib / GObject rather than being built from scratch. In particular this would let you use a good object system, which your code is clearly trying to achieve. The use of the structure list / hash table data types & error reporting in GLib would also bring benefits to this. Finally, perhaps the most compelling reason is that with GObject Introspection, we will get the ability to access libosinfo from multiple other languages for free, without needing to hand write or auto generate any extra code. The only minor downside is the OOM handling in GLib, but for the scenarios in which libosinfo is to be used I don't think that this is a problem worth worrying about & is easily outweighed by the advantages. Regards, Daniel -- |: Red Hat, Engineering, London -o- http://people.redhat.com/berrange/ :| |: http://libvirt.org -o- http://virt-manager.org -o- http://ovirt.org :| |: http://autobuild.org -o- http://search.cpan.org/~danberr/ :| |: GnuPG: 7D3B9505 -o- F3C9 553F A1DA 4AC2 5648 23C1 B3DF F742 7D3B 9505 :| _______________________________________________ virt-tools-list mailing list [email protected] https://www.redhat.com/mailman/listinfo/virt-tools-list
