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

Re: [Xen-devel] [PATCH 0/5] Towards a restartable oxenstored



On Mon, Apr 10, 2017 at 09:10:15AM +0100, Christian Lindig wrote:
> 
> > On 7. Apr 2017, at 14:27, Jonathan Davies <Jonathan.Davies@xxxxxxxxxx> 
> > wrote:
> > 
> > tools/ocaml/xenstored/domain.ml    |  4 ++--
> > tools/ocaml/xenstored/store.ml     |  8 +++++++-
> > tools/ocaml/xenstored/xenstored.ml | 10 ++++++----
> 
> The OCaml code is looking good and I’d be happy to take it as it is. 

Can I translate that to reviewed-by's?

> 
> I noticed that the (existing) code for handling the path for the
> database file uses (^) to construct it. It would be better to use
> Filename.concat and this could have been done as part of this patch.
> Admittedly, this would be mostly cosmetic as it would help mainly
> portability to Windows.
> 

Feel free to submit patches to fix that when the development window
opens.

Wei.

> — Christian

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
https://lists.xen.org/xen-devel

 


Rackspace

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