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

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



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 (5):
  virsh: Use standard error message in vshCommandOptTimeoutToMs().
  virsh: Improve vshCommandOptTimeoutToMs().
  virsh: Make vshCommandOptScaledInt() use vshCommandOpt().
  virsh: Pass vshControl to all vshCommandOpt*() calls.
  virsh: Move error messages inside vshCommandOpt*() functions.

 tests/vcpupin                |   4 +-
 tools/virsh-domain-monitor.c |  17 +--
 tools/virsh-domain.c         | 226 ++++++++++++---------------------------
 tools/virsh-host.c           |  67 +++---------
 tools/virsh-interface.c      |   6 +-
 tools/virsh-network.c        |  10 +-
 tools/virsh-nodedev.c        |   4 +-
 tools/virsh-snapshot.c       |   2 +-
 tools/virsh-volume.c         |  26 +----
 tools/virsh.c                | 248 +++++++++++++++++++++++++------------------
 tools/virsh.h                |  66 ++++++------
 11 files changed, 278 insertions(+), 398 deletions(-)

-- 
2.1.0


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