On Fri, Jul 05, 2019 at 23:37:32 -0500, Eric Blake wrote:
The code to check whether a redefined snapshot/checkpoint XML is
attempting to create a cycle in the list of moments is lengthy, and
common between the two types of list. Therefore, it belongs in the
shared base file.
Signed-off-by: Eric Blake <eblake(a)redhat.com>
---
src/conf/virdomainmomentobjlist.h | 3 +++
src/conf/virdomainsnapshotobjlist.h | 3 +++
src/conf/snapshot_conf.c | 41 ++++-------------------------
src/conf/virdomainmomentobjlist.c | 40 ++++++++++++++++++++++++++++
src/conf/virdomainsnapshotobjlist.c | 9 +++++++
tests/virsh-snapshot | 2 +-
6 files changed, 61 insertions(+), 37 deletions(-)
ACK, although it would be great if the new function would also document
what it's doing.