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

Re: [libvirt] [PATCH 5/5] src/xenconfig: move common parsing/formatting to xen_common

On 08/18/2014 10:10 AM, Eric Blake wrote:
On 08/15/2014 09:50 PM, Jim Fehlig wrote:
XM and XL config are very similar.  Disks are specified differently
in XL, but the old XM disk config is still supported by XL.  XL also
supports new config like spice that was never supported by XM.

This patch moves all the common parsing and formatting functions to
the new file xen_common.c and adapts the XM parser/formatter accordingly.
This restructuring paves way for introducing an XL parser/formatter in
the future.

While moving the code, fixup whitespace, comments, and style issues.

Signed-off-by: Jim Fehlig <jfehlig suse com>
  po/POTFILES.in             |    1 +
  src/Makefile.am            |    1 +
  src/xenconfig/xen_common.c | 2072 +++++++++++++++++++++++++++++++++++++++++
  src/xenconfig/xen_common.h |   46 +
  src/xenconfig/xen_xm.c     | 2187 ++------------------------------------------
  src/xenconfig/xen_xm.h     |   10 -
  6 files changed, 2205 insertions(+), 2112 deletions(-)
Big, but seems mostly mechanical.  I checked the diff via:

$ diff -u <(sed -n 's/^-//p' patch) <(sed -n 's/^\+//p' patch)

to make sure that most of the differences were indeed whitespace.  It
might have been a better idea to split into a couple of patches
(cleaning up style separately from renaming),

Heh, violated my own advice to David in his earlier patches. But I convinced myself this was fine since I was moving all the code to new files. Point taken though, and I'll do such style cleanup in separate patches in the future.

  but not worth the effort
to go back and redo it now.


Thanks for the review.  I've pushed the series now, minus 4/5.


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