On Wed, Oct 16, 2019 at 13:15:19 +0200, Ján Tomko wrote:
On Tue, Oct 15, 2019 at 05:34:47PM +0200, Jiri Denemark wrote:
>Let's store qemuMonitorCPUDefInfo directly in the array of CPUs in
>qemuMonitorCPUDefs rather then using an array of pointers.
>
>Signed-off-by: Jiri Denemark <jdenemar(a)redhat.com>
>Reviewed-by: Ján Tomko <jtomko(a)redhat.com>
>---
>
>Notes:
> Version 2:
> - trivial rebase
>
> src/qemu/qemu_capabilities.c | 14 +++++++-------
> src/qemu/qemu_monitor.c | 5 ++---
> src/qemu/qemu_monitor.h | 2 +-
> src/qemu/qemu_monitor_json.c | 7 +------
> tests/qemumonitorjsontest.c | 8 ++++----
> 5 files changed, 15 insertions(+), 21 deletions(-)
>
Reviewed-by: Ján Tomko <jtomko(a)redhat.com>
@@ -3573,7 +3572,7 @@ qemuMonitorCPUDefsNew(size_t count)
g_autoptr(qemuMonitorCPUDefs) defs = NULL;
defs = g_new0(qemuMonitorCPUDefs, 1);
- defs->cpus = g_new0(qemuMonitorCPUDefInfoPtr, count);
+ defs->cpus = g_new0(qemuMonitorCPUDefInfo, count);
defs->ncpus = count;
return g_steal_pointer(&defs);
}
Looks like a change in something that doesn't exist yet :-)