fix the JSONFormatter flush & close_section

ceph · Aug 1, 2015 · 90dde3b · 90dde3b
1 parent 5c6075f
commit 90dde3b
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/src/common/Formatter.cc b/src/common/Formatter.cc
@@ -126,8 +126,6 @@ void JSONFormatter::flush(std::ostream& os)
 {
   finish_pending_string();
   os << m_ss.str();
-  if (m_pretty)
-    os << "\n";
   m_ss.clear();
   m_ss.str("");
 }
@@ -238,6 +236,8 @@ void JSONFormatter::close_section()
   }
   m_ss << (entry.is_array ? ']' : '}');
   m_stack.pop_back();
+  if (m_pretty && (m_stack.size() == 0))
+    m_ss << "\n";
 }
 
 void JSONFormatter::finish_pending_string()

diff --git a/src/test/crush/CrushWrapper.cc b/src/test/crush/CrushWrapper.cc
@@ -688,6 +688,7 @@ TEST(CrushWrapper, dump_rules) {
     Formatter *f = Formatter::create("json-pretty");
     c->dump_rules(f);
     stringstream ss;
+    f->close_section();
     f->flush(ss);
     delete f;
     EXPECT_EQ("\n", ss.str());