[v2] chardev handling cleanups (chardev refactors part 1)

[PATCH v2 0/3] chardev handling cleanups (chardev refactors part 1)

Peter Krempa posted 3 patches 2 years, 5 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/cover.1637326546.git.pkrempa@redhat.com

src/qemu/qemu_monitor_json.c | 192 +++++++++++++++++------------------
1 file changed, 96 insertions(+), 96 deletions(-)

Expand all Fold all

[PATCH v2 0/3] chardev handling cleanups (chardev refactors part 1)

Posted by Peter Krempa 2 years, 5 months ago

All patches except for 6/16 are now pushed.

Pathces 1 and 2 in this series are new, split of 6/16 as requested in
the review.

Patch 3 is the rebase of 6/16 on top of the split out stuff and fix to
the [B] point in the original review.

Peter Krempa (3):
  qemuMonitorJSONAttachCharDevGetProps: Rename 'backend_type' and 'data'
  qemuMonitorJSONBuildChrChardevReconnect: Unify with
    qemuBuildChrChardevReconnectStr
  qemuMonitorJSONAttachCharDevGetProps: Modernize construction of JSON
    objects

 src/qemu/qemu_monitor_json.c | 192 +++++++++++++++++------------------
 1 file changed, 96 insertions(+), 96 deletions(-)

-- 
2.31.1

Re: [PATCH v2 0/3] chardev handling cleanups (chardev refactors part 1)

Posted by Ján Tomko 2 years, 5 months ago

On a Friday in 2021, Peter Krempa wrote:
>All patches except for 6/16 are now pushed.
>
>Pathces 1 and 2 in this series are new, split of 6/16 as requested in
>the review.
>
>Patch 3 is the rebase of 6/16 on top of the split out stuff and fix to
>the [B] point in the original review.
>
>Peter Krempa (3):
>  qemuMonitorJSONAttachCharDevGetProps: Rename 'backend_type' and 'data'
>  qemuMonitorJSONBuildChrChardevReconnect: Unify with
>    qemuBuildChrChardevReconnectStr
>  qemuMonitorJSONAttachCharDevGetProps: Modernize construction of JSON
>    objects
>
> src/qemu/qemu_monitor_json.c | 192 +++++++++++++++++------------------
> 1 file changed, 96 insertions(+), 96 deletions(-)
>

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

Jano