[libvirt PATCH 0/8] More VIR_FREE removals

Laine Stump posted 8 patches 3 years, 2 months ago
Test syntax-check failed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/libvirt tags/patchew/20210212052818.684878-1-laine@redhat.com
src/conf/capabilities.c           | 38 +++++++++++++++----------------
src/esx/esx_vi.c                  | 18 +++++++--------
src/esx/esx_vi_types.c            | 24 +++++++++----------
src/locking/lock_manager.c        |  4 ++--
src/openvz/openvz_conf.c          |  2 +-
src/qemu/qemu_domain.c            |  2 +-
src/qemu/qemu_driver.c            |  8 +++----
src/remote/remote_daemon_stream.c |  2 +-
src/util/virconf.c                |  4 ++--
src/util/virerror.c               |  4 ++--
src/util/virobject.c              |  2 +-
src/util/virstring.c              |  4 ++--
src/vmware/vmware_conf.c          |  4 ++--
src/vmware/vmware_driver.c        |  4 ++--
14 files changed, 60 insertions(+), 60 deletions(-)
[libvirt PATCH 0/8] More VIR_FREE removals
Posted by Laine Stump 3 years, 2 months ago
Only 90 this time. These are all functions that behave similar to the
*Free() functions, but their names don't end in "Free" so I missed
them last time.

Laine Stump (8):
  esx: replace VIR_FREE with g_free in any ESX_VI__TEMPLATE__FREE
  conf: convert VIR_FREE to g_free in other functions that free their
    arg
  locking: convert VIR_FREE to g_free in other functions that free their
    arg
  openvz: convert VIR_FREE to g_free in other functions that free their
    arg
  remote: convert VIR_FREE to g_free in other functions that free their
    arg
  qemu: convert VIR_FREE to g_free in other functions that free their
    arg
  util: convert VIR_FREE to g_free in other functions that free their
    arg
  vmware: convert VIR_FREE to g_free in other functions that free their
    arg

 src/conf/capabilities.c           | 38 +++++++++++++++----------------
 src/esx/esx_vi.c                  | 18 +++++++--------
 src/esx/esx_vi_types.c            | 24 +++++++++----------
 src/locking/lock_manager.c        |  4 ++--
 src/openvz/openvz_conf.c          |  2 +-
 src/qemu/qemu_domain.c            |  2 +-
 src/qemu/qemu_driver.c            |  8 +++----
 src/remote/remote_daemon_stream.c |  2 +-
 src/util/virconf.c                |  4 ++--
 src/util/virerror.c               |  4 ++--
 src/util/virobject.c              |  2 +-
 src/util/virstring.c              |  4 ++--
 src/vmware/vmware_conf.c          |  4 ++--
 src/vmware/vmware_driver.c        |  4 ++--
 14 files changed, 60 insertions(+), 60 deletions(-)

-- 
2.29.2

Re: [libvirt PATCH 0/8] More VIR_FREE removals
Posted by Michal Privoznik 3 years, 2 months ago
On 2/12/21 6:28 AM, Laine Stump wrote:
> Only 90 this time. These are all functions that behave similar to the
> *Free() functions, but their names don't end in "Free" so I missed
> them last time.
> 
> Laine Stump (8):
>    esx: replace VIR_FREE with g_free in any ESX_VI__TEMPLATE__FREE
>    conf: convert VIR_FREE to g_free in other functions that free their
>      arg
>    locking: convert VIR_FREE to g_free in other functions that free their
>      arg
>    openvz: convert VIR_FREE to g_free in other functions that free their
>      arg
>    remote: convert VIR_FREE to g_free in other functions that free their
>      arg
>    qemu: convert VIR_FREE to g_free in other functions that free their
>      arg
>    util: convert VIR_FREE to g_free in other functions that free their
>      arg
>    vmware: convert VIR_FREE to g_free in other functions that free their
>      arg
> 
>   src/conf/capabilities.c           | 38 +++++++++++++++----------------
>   src/esx/esx_vi.c                  | 18 +++++++--------
>   src/esx/esx_vi_types.c            | 24 +++++++++----------
>   src/locking/lock_manager.c        |  4 ++--
>   src/openvz/openvz_conf.c          |  2 +-
>   src/qemu/qemu_domain.c            |  2 +-
>   src/qemu/qemu_driver.c            |  8 +++----
>   src/remote/remote_daemon_stream.c |  2 +-
>   src/util/virconf.c                |  4 ++--
>   src/util/virerror.c               |  4 ++--
>   src/util/virobject.c              |  2 +-
>   src/util/virstring.c              |  4 ++--
>   src/vmware/vmware_conf.c          |  4 ++--
>   src/vmware/vmware_driver.c        |  4 ++--
>   14 files changed, 60 insertions(+), 60 deletions(-)
> 

Reviewed-by: Michal Privoznik <mprivozn@redhat.com>

Michal