On Wed, Oct 16, 2019 at 05:46:24PM +0200, Jiri Denemark wrote:
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@xxxxxxxxxx> >Reviewed-by: Ján Tomko <jtomko@xxxxxxxxxx> >--- > >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@xxxxxxxxxx> @@ -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 :-)
Yes, it's exactly what it looks like! :) Jano
Attachment:
signature.asc
Description: PGP signature
-- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list
