BUG: don't create empty <itemizedlist>s (#7289), dblatex didn't like that

Alex

1 files changed, 13 insertions, 10 deletions

diff --git a/Source/cmDocumentationFormatterDocbook.cxx b/Source/cmDocumentationFormatterDocbook.cxx
index a34689b..90c968a 100644
--- a/Source/cmDocumentationFormatterDocbook.cxx
+++ b/Source/cmDocumentationFormatterDocbook.cxx
@@ -134,20 +134,23 @@ void cmDocumentationFormatterDocbook
   const std::vector<cmDocumentationEntry> &entries = 
     section.GetEntries();
 
-  os << "<itemizedlist>\n";
-  for(std::vector<cmDocumentationEntry>::const_iterator op 
-        = entries.begin(); op != entries.end(); ++ op )
+  if (!entries.empty())
     {
-    if(op->Name.size())
+    os << "<itemizedlist>\n";
+    for(std::vector<cmDocumentationEntry>::const_iterator op 
+          = entries.begin(); op != entries.end(); ++ op )
       {
-      os << "    <listitem><link linkend=\"" << prefix << "_";
-      cmDocumentationPrintDocbookEscapes(os, op->Name.c_str());
-      os << "\"><emphasis><literal>";
-      cmDocumentationPrintDocbookEscapes(os, op->Name.c_str());
-      os << "</literal></emphasis></link></listitem>\n";
+      if(op->Name.size())
+        {
+        os << "    <listitem><link linkend=\"" << prefix << "_";
+        cmDocumentationPrintDocbookEscapes(os, op->Name.c_str());
+        os << "\"><emphasis><literal>";
+        cmDocumentationPrintDocbookEscapes(os, op->Name.c_str());
+        os << "</literal></emphasis></link></listitem>\n";
+        }
       }
+    os << "</itemizedlist>\n" ;
     }
-  os << "</itemizedlist>\n" ;
 
   for(std::vector<cmDocumentationEntry>::const_iterator op = entries.begin(); 
       op != entries.end();)