From 8a0e9e106b5b2db642a265a5747cd16892b7d7cf Mon Sep 17 00:00:00 2001 From: Peter Krempa Date: Fri, 4 Mar 2022 14:37:13 +0100 Subject: [PATCH] qemuSnapshotCreateAlignDisks: Rewrite logic for selecting default memory snapshot mode MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit Use an if/else branch rather than a expression with a ternary operator. Signed-off-by: Peter Krempa Reviewed-by: Ján Tomko --- src/qemu/qemu_snapshot.c | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/src/qemu/qemu_snapshot.c b/src/qemu/qemu_snapshot.c index f92b571b06..355a2fd782 100644 --- a/src/qemu/qemu_snapshot.c +++ b/src/qemu/qemu_snapshot.c @@ -1672,9 +1672,10 @@ qemuSnapshotCreateAlignDisks(virDomainObj *vm, return -1; } - def->memory = (def->state == VIR_DOMAIN_SNAPSHOT_SHUTOFF ? - VIR_DOMAIN_SNAPSHOT_LOCATION_NONE : - VIR_DOMAIN_SNAPSHOT_LOCATION_INTERNAL); + if (def->state == VIR_DOMAIN_SNAPSHOT_SHUTOFF) + def->memory = VIR_DOMAIN_SNAPSHOT_LOCATION_NONE; + else + def->memory = VIR_DOMAIN_SNAPSHOT_LOCATION_INTERNAL; } if (virDomainSnapshotAlignDisks(def, NULL, align_location, true) < 0) return -1; -- 2.39.5