[v2] Remove macros "VIR_INSERT_ELEMENT_COPY" and "VIR_INSERT_ELEMENT_COPY_INPLACE"

[libvirt PATCH v2 0/2] Remove macros "VIR_INSERT_ELEMENT_COPY" and "VIR_INSERT_ELEMENT_COPY_INPLACE"

Tim Wiederhake posted 2 patches 2 years, 9 months ago

Diff against v1
Download series mbox

Test syntax-check failed

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

src/qemu/qemu_capabilities.c | 37 ++++++++++++++++++------------------
src/util/viralloc.h          | 22 ++++-----------------
2 files changed, 22 insertions(+), 37 deletions(-)

Expand all Fold all

[libvirt PATCH v2 0/2] Remove macros "VIR_INSERT_ELEMENT_COPY" and "VIR_INSERT_ELEMENT_COPY_INPLACE"

Posted by Tim Wiederhake 2 years, 9 months ago

See https://listman.redhat.com/archives/libvir-list/2021-July/msg00163.html

Changes since V1:
* Fixed a mistake in the copy-out of virQEMUCapsGetMachineTypesCaps, where
the array was allocated with the un-adjusted size.

Tim Wiederhake (2):
  virQEMUCapsGetMachineTypesCaps: Use GPtrArray
  viralloc: Delete VIR_INSERT_ELEMENT_COPY and
    VIR_INSERT_ELEMENT_COPY_INPLACE

 src/qemu/qemu_capabilities.c | 37 ++++++++++++++++++------------------
 src/util/viralloc.h          | 22 ++++-----------------
 2 files changed, 22 insertions(+), 37 deletions(-)

-- 
2.31.1

Re: [libvirt PATCH v2 0/2] Remove macros "VIR_INSERT_ELEMENT_COPY" and "VIR_INSERT_ELEMENT_COPY_INPLACE"

Posted by Michal Prívozník 2 years, 9 months ago

On 7/9/21 3:23 PM, Tim Wiederhake wrote:
> See https://listman.redhat.com/archives/libvir-list/2021-July/msg00163.html
> 
> Changes since V1:
> * Fixed a mistake in the copy-out of virQEMUCapsGetMachineTypesCaps, where
> the array was allocated with the un-adjusted size.
> 
> Tim Wiederhake (2):
>   virQEMUCapsGetMachineTypesCaps: Use GPtrArray
>   viralloc: Delete VIR_INSERT_ELEMENT_COPY and
>     VIR_INSERT_ELEMENT_COPY_INPLACE
> 
>  src/qemu/qemu_capabilities.c | 37 ++++++++++++++++++------------------
>  src/util/viralloc.h          | 22 ++++-----------------
>  2 files changed, 22 insertions(+), 37 deletions(-)
> 

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

Michal