[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH 08/14] libxl: disks: new xlu_disk_parse function

Ian Campbell writes ("Re: [Xen-devel] [PATCH 08/14] libxl: disks: new 
xlu_disk_parse function"):
> On Thu, 2011-05-12 at 15:36 +0100, Ian Jackson wrote:
> > [...]
> Something seems to be up with the leading whitespace here.


> > +   * string containing a collection of parameters (but they all refer
> > +   * to of the configuration for a single disk).
>            ^^ what is that doing there?


> This is the third time this week I've wondered if a patch ought to come
> along with a library of test inputs and expected outputs. I'll be
> proposing we switch to the waterfall model before you know it...

My series repost will include one, since it seemed useful.


Xen-devel mailing list



Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.