[libvirt] [PATCH 0/2] virsh: Further improve handling of integer options

Andrea Bolognani abologna at redhat.com
Thu May 21 09:40:25 UTC 2015


As suggested by Michal: now that we have a generic error message for
failures related to the parsing of integer options, it makes sense to
perform the corresponding check in a single spot instead of replicating
it every time vshCommandOpt*() is used.

Andrea Bolognani (2):
  virsh: Pass vshControl to all vshCommandOpt*() calls.
  virsh: Move error messages inside vshCommandOpt*() functions.

 tests/vcpupin                |   4 +-
 tools/virsh-domain-monitor.c |  97 +++---
 tools/virsh-domain.c         | 696 +++++++++++++++++++------------------------
 tools/virsh-host.c           |  87 ++----
 tools/virsh-interface.c      |  18 +-
 tools/virsh-network.c        |  38 ++-
 tools/virsh-nodedev.c        |   6 +-
 tools/virsh-pool.c           |  26 +-
 tools/virsh-secret.c         |   8 +-
 tools/virsh-snapshot.c       |  88 +++---
 tools/virsh-volume.c         |  50 ++--
 tools/virsh.c                | 195 ++++++------
 tools/virsh.h                |  77 ++---
 13 files changed, 624 insertions(+), 766 deletions(-)

-- 
2.1.0




More information about the libvir-list mailing list