mirror of
https://gitlab.com/libvirt/libvirt.git
synced 2024-12-22 05:35:25 +00:00
vbox_snapshot_conf: Keep indent in snapshot XML
As mentioned in previous commit, VirtualBox has its own snapshot XML which we parse, change and then format back. During this, we ought to keep the indentation to produce better looking result (especially when we want to compare the output in tests later on, like we do in vboxsnapshotxmltest). Signed-off-by: Michal Privoznik <mprivozn@redhat.com> Reviewed-by: Ján Tomko <jtomko@redhat.com>
This commit is contained in:
parent
d8cb1cd50c
commit
d1f58b10f6
@ -25,6 +25,7 @@
|
|||||||
#include "virstring.h"
|
#include "virstring.h"
|
||||||
#include "virxml.h"
|
#include "virxml.h"
|
||||||
|
|
||||||
|
#include <libxml/xmlsave.h>
|
||||||
#include <libxml/xpathInternals.h>
|
#include <libxml/xpathInternals.h>
|
||||||
|
|
||||||
#define VIR_FROM_THIS VIR_FROM_VBOX
|
#define VIR_FROM_THIS VIR_FROM_VBOX
|
||||||
@ -364,12 +365,11 @@ virVBoxSnapshotConfSerializeSnapshot(xmlNodePtr node,
|
|||||||
xmlParserErrors parseError = XML_ERR_OK;
|
xmlParserErrors parseError = XML_ERR_OK;
|
||||||
char *uuid = NULL;
|
char *uuid = NULL;
|
||||||
char *timeStamp = NULL;
|
char *timeStamp = NULL;
|
||||||
|
|
||||||
g_auto(GStrv) firstRegex = NULL;
|
g_auto(GStrv) firstRegex = NULL;
|
||||||
int firstRegexResult = 0;
|
int firstRegexResult = 0;
|
||||||
g_auto(GStrv) secondRegex = NULL;
|
g_auto(GStrv) secondRegex = NULL;
|
||||||
int secondRegexResult = 0;
|
int secondRegexResult = 0;
|
||||||
const int parseFlags = XML_PARSE_NONET;
|
const int parseFlags = XML_PARSE_NONET | XML_PARSE_NOBLANKS;
|
||||||
|
|
||||||
uuid = g_strdup_printf("{%s}", snapshot->uuid);
|
uuid = g_strdup_printf("{%s}", snapshot->uuid);
|
||||||
|
|
||||||
@ -940,12 +940,14 @@ virVBoxSnapshotConfSaveVboxFile(virVBoxSnapshotConfMachine *machine,
|
|||||||
xmlParserErrors parseError = XML_ERR_OK;
|
xmlParserErrors parseError = XML_ERR_OK;
|
||||||
char *currentSnapshot = NULL;
|
char *currentSnapshot = NULL;
|
||||||
char *timeStamp = NULL;
|
char *timeStamp = NULL;
|
||||||
|
|
||||||
g_auto(GStrv) firstRegex = NULL;
|
g_auto(GStrv) firstRegex = NULL;
|
||||||
int firstRegexResult = 0;
|
int firstRegexResult = 0;
|
||||||
g_auto(GStrv) secondRegex = NULL;
|
g_auto(GStrv) secondRegex = NULL;
|
||||||
int secondRegexResult = 0;
|
int secondRegexResult = 0;
|
||||||
const int parseFlags = XML_PARSE_NONET;
|
const int parseFlags = XML_PARSE_NONET | XML_PARSE_NOBLANKS;
|
||||||
|
int oldIndentTreeOutput = xmlIndentTreeOutput;
|
||||||
|
|
||||||
|
xmlIndentTreeOutput = 1;
|
||||||
|
|
||||||
if (machine == NULL) {
|
if (machine == NULL) {
|
||||||
virReportError(VIR_ERR_INTERNAL_ERROR, "%s",
|
virReportError(VIR_ERR_INTERNAL_ERROR, "%s",
|
||||||
@ -1127,6 +1129,8 @@ virVBoxSnapshotConfSaveVboxFile(virVBoxSnapshotConfMachine *machine,
|
|||||||
ret = 0;
|
ret = 0;
|
||||||
|
|
||||||
cleanup:
|
cleanup:
|
||||||
|
xmlIndentTreeOutput = oldIndentTreeOutput;
|
||||||
|
|
||||||
VIR_FREE(currentSnapshot);
|
VIR_FREE(currentSnapshot);
|
||||||
VIR_FREE(timeStamp);
|
VIR_FREE(timeStamp);
|
||||||
|
|
||||||
|
Loading…
Reference in New Issue
Block a user