From 64f4c341bd660273ed5896235a9d7b73e44bd68c Mon Sep 17 00:00:00 2001
From: Joe Thornber <ejt@redhat.com>
Date: Mon, 10 Oct 2011 16:14:36 +0100
Subject: [PATCH] output superblock in xml format

---
 metadata_dump.cc |  4 ++++
 xml_format.cc    | 28 ++++++++++++++++++----------
 2 files changed, 22 insertions(+), 10 deletions(-)

diff --git a/metadata_dump.cc b/metadata_dump.cc
index 373585d..d6aee78 100644
--- a/metadata_dump.cc
+++ b/metadata_dump.cc
@@ -123,6 +123,8 @@ namespace {
 void
 metadata::dump(emitter::ptr e)
 {
+	e->begin_superblock("", sb_.time_, sb_.trans_id_, sb_.data_block_size_);
+
 	details_extractor::ptr de(new details_extractor);
 
 	details_.visit(de);
@@ -144,6 +146,8 @@ metadata::dump(emitter::ptr e)
 
 		e->end_device();
 	}
+
+	e->end_superblock();
 }
 
 //----------------------------------------------------------------
diff --git a/xml_format.cc b/xml_format.cc
index 92d712e..2a3655a 100644
--- a/xml_format.cc
+++ b/xml_format.cc
@@ -22,15 +22,19 @@ namespace {
 				      uint64_t time,
 				      uint64_t trans_id,
 				      uint32_t data_block_size) {
-			out_ << "begin superblock: " << uuid
-			     << ", " << time
-			     << ", " << trans_id
-			     << ", " << data_block_size
+			indent();
+			out_ << "<superblock uuid=\"" << uuid << "\""
+			     << " time=\"" << time << "\""
+			     << " transaction=\"" << trans_id << "\""
+			     << " data_block_size=\"" << data_block_size << "\">"
 			     << endl;
+			inc();
 		}
 
 		void end_superblock() {
-			out_ << "end superblock" << endl;
+			dec();
+			indent();
+			out_ << "</superblock>" << endl;
 		}
 
 		void begin_device(uint32_t dev_id,
@@ -49,21 +53,25 @@ namespace {
 
 		void end_device() {
 			dec();
+			indent();
 			out_ << "</device>" << endl;
 		}
 
 		void begin_named_mapping(string const &name) {
-			out_ << "begin named mapping"
-			     << endl;
+			indent();
+			out_ << "<named_mapping>" << endl;
+			inc();
 		}
 
 		void end_named_mapping() {
-			out_ << "end named mapping"
-			     << endl;
+			dec();
+			indent();
+			out_ << "</named_mapping>" << endl;
 		}
 
 		void identifier(string const &name) {
-			out_ << "identifier: " << name << endl;
+			indent();
+			out_ << "<identifier name=\"" << name << "\"/>" << endl;
 		}
 
 		void range_map(uint64_t origin_begin, uint64_t data_begin, uint64_t len) {