On 7/25/22 14:45, Peter Krempa wrote:
I've noticed few things that could be improved during review of
Jirka's
post copy migration changes but didn't want to interfer with the series
at that point.
Peter Krempa (5):
qemuMigrationDstFinishFresh: Avoid multi-line ternary operator in
function call
qemuMigrationDstPersist: Avoid multi-line ternary operator in function
call
qemu: migration: Overwrite 'dname' only when NULL
qemuMigrationSrcIOFunc: Avoid unnecessary string construction
qemu: monitor: Split up enum strings definitions
src/qemu/qemu_migration.c | 37 +++++++++++++++++++++++--------------
src/qemu/qemu_monitor.c | 35 ++++++++++++++++++++++++++---------
2 files changed, 49 insertions(+), 23 deletions(-)
Reviewed-by: Michal Privoznik <mprivozn(a)redhat.com>
But see my comment to 1/5 before pushing. There's one typo that needs
fixing.
Michal