[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