Re: [PATCH 0/2] Simplify hash table data free callbacks

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

 



On 11/22/19 1:00 PM, Daniel P. Berrangé wrote:
See the patch commit message for the interesting details.

Daniel P. Berrangé (2):
   conf: stop using hash key when free'ing hash entries
   util: consolidate on one free callback for hash data

  src/conf/domain_addr.c              |  4 +--
  src/conf/nwfilter_params.c          |  2 +-
  src/conf/virchrdev.c                | 43 +++++++++++++++++++----------
  src/conf/virdomainmomentobjlist.c   |  3 +-
  src/conf/virnetworkobj.c            |  2 +-
  src/libxl/libxl_logger.c            |  2 +-
  src/locking/lock_daemon.c           |  3 +-
  src/nwfilter/nwfilter_dhcpsnoop.c   |  4 +--
  src/nwfilter/nwfilter_learnipaddr.c |  2 +-
  src/qemu/qemu_block.c               |  3 +-
  src/qemu/qemu_conf.c                |  4 +--
  src/qemu/qemu_conf.h                |  2 +-
  src/qemu/qemu_domain.c              |  3 +-
  src/qemu/qemu_migration.c           |  3 +-
  src/qemu/qemu_monitor.c             |  5 ++--
  src/qemu/qemu_monitor.h             |  2 +-
  src/qemu/qemu_monitor_json.c        |  2 +-
  src/util/vircgroup.c                |  2 +-
  src/util/virhash.c                  | 28 +++++--------------
  src/util/virhash.h                  | 16 ++---------
  src/util/virjson.c                  |  3 +-
  src/util/virjson.h                  |  2 +-
  src/util/virkeyfile.c               |  2 +-
  src/util/virlockspace.c             |  2 +-
  src/util/virobject.c                |  3 +-
  src/util/virobject.h                |  3 +-
  src/util/virsystemd.c               |  8 +++---
  27 files changed, 70 insertions(+), 88 deletions(-)


Reviewed-by: Michal Privoznik <mprivozn@xxxxxxxxxx>

Michal

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux