Ian Campbell writes ("Re: [Xen-devel] [PATCH 13/14] libxl: disks: allow
specification of "backendtype=phy|tap|qdisk""):
> On Thu, 2011-05-12 at 15:36 +0100, Ian Jackson wrote:
> > +/* Sets ->backend based on the string value. This ought to be an enum. */
>
> Is this an old comment? Backend is a libxl_disk_backend, which _is_ an
> enum.
What I mean, now, is that the IDL should provide a table of these
strings, and there should be a standard conversion function, rather
than this open-coded list of strcmps. I will update the comments.
Ian.
_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel
|