On Mon, Feb 19, 2007 at 02:46:43PM +0000, Mark McLoughlin wrote:
> Unfortunately I don't see a good thing to do in
> this case except to change the semantics of virConnectGetType, which
> arguably breaks ABI compatibility.
>
> Suggestions from anyone?
Okay, suggest:
1) Add a new function with a different name which returns an
allocated string. Mark the old function as deprecated.
2) Retain the old function, and have a list of likely static strings
which you can return if it matches the string you got from the
remote side, and if not return (and leak) an allocated string
I would go from 2) but keep an array of existing returned entries, so
we only keep one list of the set of OS type handled. That list will be finite
and relatively short, that should not be a problem.
Daniel
--
Red Hat Virtualization group
http://redhat.com/virtualization/
Daniel Veillard | virtualization library
http://libvirt.org/
veillard(a)redhat.com | libxml GNOME XML XSLT toolkit
http://xmlsoft.org/
http://veillard.com/ | Rpmfind RPM search engine
http://rpmfind.net/