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@redhat.com> Reviewed-by: Ján Tomko <jtomko@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@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 :-)