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

[libvirt PATCH 00/14] Various cleanups



Some reported by cppcheck, maybe even Coverity, the rest
I noticed while looking at the code.

Ján Tomko (14):
  tools: virshCheckpointListCollect: remove unused names
  vbox: remove VBoxCGlueTerm
  vbox: StartMachine: overwrite ret less often
  Remove pointless assignments
  Remove pointless initializations
  virsh: virshStreamSourceSkip: remove unused 'off'
  Do not check whether unsigned variables are negative
  libxl: remove unused 'bits' from struct guest_arch
  api: virDomainMemoryStats: use 'ret' variable
  storage: createFileDir: remove useless 'err' variable
  storage: createFileDir: use less ternary operators
  vbox: reduce variable scope in vboxDumpStorageControllers
  storage: storageBackendWipeLocal: reduce variable scope
  Reduce scope of some variables

 examples/c/admin/logging.c           |  2 +-
 src/libvirt-domain.c                 |  8 ++---
 src/libvirt-host.c                   |  4 +--
 src/libxl/libxl_capabilities.c       |  1 -
 src/libxl/libxl_driver.c             |  2 +-
 src/node_device/node_device_driver.c |  3 +-
 src/openvz/openvz_driver.c           |  2 +-
 src/storage/storage_backend_mpath.c  |  3 +-
 src/storage/storage_util.c           | 33 ++++++++++----------
 src/test/test_driver.c               |  2 +-
 src/vbox/vbox_XPCOMCGlue.c           | 19 ------------
 src/vbox/vbox_XPCOMCGlue.h           |  1 -
 src/vbox/vbox_common.c               | 46 +++++++++++++---------------
 tests/virnumamock.c                  |  2 +-
 tests/virrandommock.c                |  2 +-
 tools/virsh-checkpoint.c             |  5 ---
 tools/virsh-pool.c                   |  3 +-
 tools/virsh-util.c                   |  3 +-
 18 files changed, 53 insertions(+), 88 deletions(-)

-- 
2.26.2


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