[Libguestfs] [PATCH 1/2] v2v: -i disk: Move code that creates name from disk to separate module.
Richard W.M. Jones
rjones at redhat.com
Fri Sep 9 14:33:55 UTC 2016
On Fri, Sep 09, 2016 at 04:26:01PM +0200, Pino Toscano wrote:
> On Friday, 9 September 2016 15:09:56 CEST Richard W.M. Jones wrote:
> > Simple code motion.
> > ---
> > v2v/Makefile.am | 2 ++
> > v2v/input_disk.ml | 21 ++-------------------
> > v2v/name_from_disk.ml | 41 +++++++++++++++++++++++++++++++++++++++++
> > v2v/name_from_disk.mli | 24 ++++++++++++++++++++++++
> > 4 files changed, 69 insertions(+), 19 deletions(-)
> > create mode 100644 v2v/name_from_disk.ml
> > create mode 100644 v2v/name_from_disk.mli
>
> Looks good -- I'd have put it in Utils though, instead of a new module
> with just a single simple function.
That's because you don't have a 16 core computer :-)
Rich.
--
Richard Jones, Virtualization Group, Red Hat http://people.redhat.com/~rjones
Read my programming and virtualization blog: http://rwmj.wordpress.com
virt-top is 'top' for virtual machines. Tiny program with many
powerful monitoring features, net stats, disk stats, logging, etc.
http://people.redhat.com/~rjones/virt-top
More information about the Libguestfs
mailing list