virsh: Allow more use of VIR_AUTO* in virsh

[libvirt] [PATCH 0/5] virsh: Allow more use of VIR_AUTO* in virsh

Peter Krempa posted 5 patches 4 years, 7 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.1568637841.git.pkrempa@redhat.com

tools/virsh-domain.c | 51 ++++++++++++++------------------------------
tools/virsh-util.h   |  3 +++
2 files changed, 19 insertions(+), 35 deletions(-)

Expand all Fold all

[libvirt] [PATCH 0/5] virsh: Allow more use of VIR_AUTO* in virsh

Posted by Peter Krempa 4 years, 7 months ago

See patch 1 for impl and description and patch 2 for example.

Peter Krempa (5):
  virsh: Allow using VIR_AUTOPTR for releasing virDomainPtr in virsh
  virsh: demonstrate use of VIR_AUTOPTR(virshDomain) on
    'send-process-signal'
  virsh: Use virshDomain type in 'inject-nmi'
  virsh: Use VIR_AUTO machinery in cmdQemuMonitorCommand
  virsh: Don't open-code virJSONStringReformat in cmdQemuMonitorCommand

 tools/virsh-domain.c | 51 ++++++++++++++------------------------------
 tools/virsh-util.h   |  3 +++
 2 files changed, 19 insertions(+), 35 deletions(-)

-- 
2.21.0

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

Re: [libvirt] [PATCH 0/5] virsh: Allow more use of VIR_AUTO* in virsh

Posted by Ján Tomko 4 years, 7 months ago

On Mon, Sep 16, 2019 at 02:45:31PM +0200, Peter Krempa wrote:
>See patch 1 for impl and description and patch 2 for example.
>
>Peter Krempa (5):
>  virsh: Allow using VIR_AUTOPTR for releasing virDomainPtr in virsh
>  virsh: demonstrate use of VIR_AUTOPTR(virshDomain) on
>    'send-process-signal'
>  virsh: Use virshDomain type in 'inject-nmi'
>  virsh: Use VIR_AUTO machinery in cmdQemuMonitorCommand
>  virsh: Don't open-code virJSONStringReformat in cmdQemuMonitorCommand
>
> tools/virsh-domain.c | 51 ++++++++++++++------------------------------
> tools/virsh-util.h   |  3 +++
> 2 files changed, 19 insertions(+), 35 deletions(-)
>

Reviewed-by: Ján Tomko <jtomko@redhat.com>

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