delete VIR_ALLOC and VIR_ALLOC_N (glib chronicles)

[libvirt PATCH 0/8] delete VIR_ALLOC and VIR_ALLOC_N (glib chronicles)

Ján Tomko posted 8 patches 3 years, 6 months ago

Download series mbox

Test syntax-check failed

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/libvirt tags/patchew/cover.1602158901.git.jtomko@redhat.com

src/access/viraccessmanager.c       |   3 +-
src/admin/admin_server_dispatch.c   |   6 +-
src/hypervisor/domain_driver.c      |   3 +-
src/hypervisor/virclosecallbacks.c  |   7 +-
src/libvirt_private.syms            |   2 -
src/libxl/xen_xm.c                  |   9 +-
src/remote/remote_daemon_config.c   |   3 +-
src/remote/remote_daemon_dispatch.c | 279 +++++++++-------------------
src/remote/remote_daemon_stream.c   |   6 +-
src/remote/remote_driver.c          | 102 +++-------
src/test/test_driver.c              |  66 +++----
src/util/viralloc.c                 |  39 ----
src/util/viralloc.h                 |  33 ----
tests/viralloctest.c                |  89 +--------
14 files changed, 157 insertions(+), 490 deletions(-)

Expand all Fold all

[libvirt PATCH 0/8] delete VIR_ALLOC and VIR_ALLOC_N (glib chronicles)

Posted by Ján Tomko 3 years, 6 months ago

Ján Tomko (8):
  remote: refactor remoteSerializeDHCPLease
  libxl: xenParseXMOS: separate VIR_ALLOC call
  remote: use g_new0 instead of VIR_ALLOC
  remote: remoteDispatchAuthList: remove useless 'rv'
  src: use g_new0 instead of VIR_ALLOC
  tests: use g_new0 instead of VIR_ALLOC
  tests: delete VIR_ALLOC tests cases
  util: delete VIR_ALLOC and VIR_ALLOC_N

 src/access/viraccessmanager.c       |   3 +-
 src/admin/admin_server_dispatch.c   |   6 +-
 src/hypervisor/domain_driver.c      |   3 +-
 src/hypervisor/virclosecallbacks.c  |   7 +-
 src/libvirt_private.syms            |   2 -
 src/libxl/xen_xm.c                  |   9 +-
 src/remote/remote_daemon_config.c   |   3 +-
 src/remote/remote_daemon_dispatch.c | 279 +++++++++-------------------
 src/remote/remote_daemon_stream.c   |   6 +-
 src/remote/remote_driver.c          | 102 +++-------
 src/test/test_driver.c              |  66 +++----
 src/util/viralloc.c                 |  39 ----
 src/util/viralloc.h                 |  33 ----
 tests/viralloctest.c                |  89 +--------
 14 files changed, 157 insertions(+), 490 deletions(-)

-- 
2.26.2

Re: [libvirt PATCH 0/8] delete VIR_ALLOC and VIR_ALLOC_N (glib chronicles)

Posted by Pavel Hrdina 3 years, 6 months ago

On Thu, Oct 08, 2020 at 02:12:06PM +0200, Ján Tomko wrote:
> Ján Tomko (8):
>   remote: refactor remoteSerializeDHCPLease
>   libxl: xenParseXMOS: separate VIR_ALLOC call
>   remote: use g_new0 instead of VIR_ALLOC
>   remote: remoteDispatchAuthList: remove useless 'rv'
>   src: use g_new0 instead of VIR_ALLOC
>   tests: use g_new0 instead of VIR_ALLOC
>   tests: delete VIR_ALLOC tests cases
>   util: delete VIR_ALLOC and VIR_ALLOC_N
> 
>  src/access/viraccessmanager.c       |   3 +-
>  src/admin/admin_server_dispatch.c   |   6 +-
>  src/hypervisor/domain_driver.c      |   3 +-
>  src/hypervisor/virclosecallbacks.c  |   7 +-
>  src/libvirt_private.syms            |   2 -
>  src/libxl/xen_xm.c                  |   9 +-
>  src/remote/remote_daemon_config.c   |   3 +-
>  src/remote/remote_daemon_dispatch.c | 279 +++++++++-------------------
>  src/remote/remote_daemon_stream.c   |   6 +-
>  src/remote/remote_driver.c          | 102 +++-------
>  src/test/test_driver.c              |  66 +++----
>  src/util/viralloc.c                 |  39 ----
>  src/util/viralloc.h                 |  33 ----
>  tests/viralloctest.c                |  89 +--------
>  14 files changed, 157 insertions(+), 490 deletions(-)

With the build error fixed

Reviewed-by: Pavel Hrdina <phrdina@redhat.com>