From 0d9bd68f650a892010f4f587e80a0fa0b892a6d0 Mon Sep 17 00:00:00 2001
From: Sage Weil <sage@newdream.net>
Date: Sat, 10 Oct 2009 21:34:40 -0700
Subject: [PATCH] monmaptool: use generic MonMap::print methods

---
 src/mon/MonMap.cc | 18 ++++++++++++++++++
 src/mon/MonMap.h  |  7 +++++++
 src/monmaptool.cc | 11 +----------
 3 files changed, 26 insertions(+), 10 deletions(-)

diff --git a/src/mon/MonMap.cc b/src/mon/MonMap.cc
index bc7711e8886..551c3fda4d5 100644
--- a/src/mon/MonMap.cc
+++ b/src/mon/MonMap.cc
@@ -26,3 +26,21 @@ int MonMap::read(const char *fn)
   decode(bl);
   return 0;
 }
+
+void MonMap::print_summary(ostream& out)
+{
+  out << "e" << epoch << ": "
+      << mon_inst.size() << " mons at";
+  for (unsigned i = 0; i<mon_inst.size(); i++)
+    out << " " << mon_inst[i].addr;
+}
+ 
+void MonMap::print(ostream& out)
+{
+  out << "epoch " << epoch << "\n";
+  out << "fsid " << fsid << "\n";
+  out << "last_changed " << last_changed << "\n";
+  for (unsigned i=0; i<mon_inst.size(); i++) {
+    out << "\t" << mon_inst[i] << "\n";
+  }
+}
diff --git a/src/mon/MonMap.h b/src/mon/MonMap.h
index 24f4ef0f17b..8b6b43e040b 100644
--- a/src/mon/MonMap.h
+++ b/src/mon/MonMap.h
@@ -116,6 +116,8 @@ class MonMap {
   int write(const char *fn);
   int read(const char *fn);
 
+  void print(ostream& out);
+  void print_summary(ostream& out);
 };
 
 inline void encode(MonMap &m, bufferlist &bl) {
@@ -125,4 +127,9 @@ inline void decode(MonMap &m, bufferlist::iterator &p) {
   m.decode(p);
 }
 
+inline ostream& operator<<(ostream& out, MonMap& m) {
+  m.print_summary(out);
+  return out;
+}
+
 #endif
diff --git a/src/monmaptool.cc b/src/monmaptool.cc
index 50b86c75ec2..e42008a557b 100644
--- a/src/monmaptool.cc
+++ b/src/monmaptool.cc
@@ -32,15 +32,6 @@ void usage()
   exit(1);
 }
 
-void printmap(const char *me, MonMap *m)
-{
-  cout << me << ": monmap: epoch " << m->epoch << std::endl
-       << me << ": monmap: fsid " << m->fsid << std::endl;
-  for (unsigned i=0; i<m->mon_inst.size(); i++)
-    cout << me << ": monmap:  " //<< "mon" << i << " " 
-	 << m->mon_inst[i] << std::endl;
-}
-
 int main(int argc, const char **argv)
 {
   vector<const char*> args;
@@ -131,7 +122,7 @@ int main(int argc, const char **argv)
     monmap.epoch++;
 
   if (print) 
-    printmap(me, &monmap);
+    monmap.print(cout);
 
   if (modified) {
     // write it out