[libvirt] [PATCH v3 02/13] Rename all domain list APIs to have virDomainObjList prefix
Jiri Denemark
jdenemar at redhat.com
Fri Feb 1 20:42:07 UTC 2013
On Fri, Feb 01, 2013 at 11:18:24 +0000, Daniel P. Berrange wrote:
> From: "Daniel P. Berrange" <berrange at redhat.com>
>
> The APIs names for accessing the domain list object are
> very inconsistent. Rename them all to have a standard
> virDomainObjList prefix.
> ---
> src/conf/domain_conf.c | 156 +++++++++++++++++---------------
> src/conf/domain_conf.h | 56 ++++++------
> src/libvirt_private.syms | 20 ++---
> src/libxl/libxl_conf.h | 2 +-
> src/libxl/libxl_driver.c | 172 ++++++++++++++++++-----------------
> src/lxc/lxc_conf.h | 2 +-
> src/lxc/lxc_driver.c | 144 ++++++++++++++---------------
> src/lxc/lxc_process.c | 12 +--
> src/openvz/openvz_conf.c | 6 +-
> src/openvz/openvz_conf.h | 2 +-
> src/openvz/openvz_driver.c | 67 +++++++-------
> src/parallels/parallels_driver.c | 52 +++++------
> src/parallels/parallels_storage.c | 2 +-
> src/parallels/parallels_utils.h | 2 +-
> src/qemu/qemu_conf.c | 2 +-
> src/qemu/qemu_conf.h | 2 +-
> src/qemu/qemu_domain.c | 2 +-
> src/qemu/qemu_driver.c | 184 +++++++++++++++++++-------------------
> src/qemu/qemu_migration.c | 8 +-
> src/qemu/qemu_process.c | 2 +-
> src/test/test_driver.c | 179 ++++++++++++++++++------------------
> src/uml/uml_conf.h | 2 +-
> src/uml/uml_driver.c | 140 ++++++++++++++---------------
> src/vmware/vmware_conf.c | 7 +-
> src/vmware/vmware_conf.h | 2 +-
> src/vmware/vmware_driver.c | 72 +++++++--------
> 26 files changed, 665 insertions(+), 632 deletions(-)
Pretty big but quite mechanical. You could have changed
type function(...)
into
type
function(...)
in all places rather than in just a few places but don't bother doing
that now. Nice thing is that all the renames and reordering of function
parameters can be checked by the compiler. And it doesn't complain about
anything, even make check and syntax-check passes.
ACK
Jirka
More information about the libvir-list
mailing list