From f6b489885b75001b373c3aa2d3c7f1bd33520c06 Mon Sep 17 00:00:00 2001
From: Martin Smith <martin.smith@nokia.com>
Date: Wed, 6 Oct 2010 10:30:21 +0200
Subject: qdoc: Cleaned up a lot of redundant table stuff.

Now just uses <ul> for redundant lists.
---
 tools/qdoc3/ditaxmlgenerator.cpp | 96 +++-------------------------------------
 tools/qdoc3/ditaxmlgenerator.h   |  3 +-
 2 files changed, 8 insertions(+), 91 deletions(-)

diff --git a/tools/qdoc3/ditaxmlgenerator.cpp b/tools/qdoc3/ditaxmlgenerator.cpp
index 6ec8934..5458b7c 100644
--- a/tools/qdoc3/ditaxmlgenerator.cpp
+++ b/tools/qdoc3/ditaxmlgenerator.cpp
@@ -3238,90 +3238,19 @@ void DitaXmlGenerator::generateSection(const NodeList& nl,
                                        CodeMarker* marker,
                                        CodeMarker::SynopsisStyle style)
 {
-    bool name_alignment = true;
     if (!nl.isEmpty()) {
-        bool twoColumn = false;
-        if (style == CodeMarker::SeparateList) {
-            name_alignment = false;
-            twoColumn = (nl.count() >= 16);
-        }
-        else if (nl.first()->type() == Node::Property) {
-            twoColumn = (nl.count() >= 5);
-            name_alignment = false;
-        }
-        if (name_alignment) {
-            xmlWriter().writeStartElement("table");
-            xmlWriter().writeAttribute("outputclass","alignedsummary");
-            xmlWriter().writeStartElement("tgroup");
-            xmlWriter().writeAttribute("cols","2");
-            xmlWriter().writeStartElement("tbody");
-        }
-        else {
-            if (twoColumn) {
-                xmlWriter().writeStartElement("table");
-                xmlWriter().writeAttribute("outputclass","propsummary");
-                xmlWriter().writeStartElement("tgroup");
-                xmlWriter().writeAttribute("cols","1");
-                xmlWriter().writeStartElement("tbody");
-                xmlWriter().writeStartElement("row");
-                xmlWriter().writeStartElement("entry");
-                xmlWriter().writeAttribute("outputclass","topAlign");
-            }
-            xmlWriter().writeStartElement("ul");
-        }
-
-        int i = 0;
+        xmlWriter().writeStartElement("ul");
         NodeList::ConstIterator m = nl.begin();
         while (m != nl.end()) {
-            if ((*m)->access() == Node::Private) {
-                ++m;
-                continue;
-            }
-
-            if (name_alignment) {
-                xmlWriter().writeStartElement("row");
-                xmlWriter().writeStartElement("entry");
-                xmlWriter().writeAttribute("outputclass","memItemLeft rightAlign topAlign");
-            }
-            else {
-                if (twoColumn && i == (int) (nl.count() + 1) / 2) {
-                    xmlWriter().writeEndElement(); // </ul>
-                    xmlWriter().writeEndElement(); // </entry>
-                    xmlWriter().writeStartElement("entry");
-                    xmlWriter().writeAttribute("outputclass","topAlign");
-                    xmlWriter().writeStartElement("ul");
-                }
+            if ((*m)->access() != Node::Private) {
                 xmlWriter().writeStartElement("li");
-                xmlWriter().writeAttribute("outputclass","fn");
-            }
-
-            QString marked = getMarkedUpSynopsis(*m, relative, marker, style);
-            writeText(marked, marker, relative, name_alignment);
-            if (name_alignment) {
-                xmlWriter().writeEndElement(); // </entry>
-                xmlWriter().writeEndElement(); // </row>
-            }
-            else {
+                QString marked = getMarkedUpSynopsis(*m, relative, marker, style);
+                writeText(marked, marker, relative);
                 xmlWriter().writeEndElement(); // </li>
             }
-            i++;
             ++m;
         }
-        if (name_alignment) {
-            xmlWriter().writeEndElement(); // </tbody>
-            xmlWriter().writeEndElement(); // </tgroup>
-            xmlWriter().writeEndElement(); // </table>
-        }
-        else {
-            xmlWriter().writeEndElement(); // </ul>
-            if (twoColumn) {
-                xmlWriter().writeEndElement(); // </entry>
-                xmlWriter().writeEndElement(); // </row>
-                xmlWriter().writeEndElement(); // </tbody>
-                xmlWriter().writeEndElement(); // </tgroup>
-                xmlWriter().writeEndElement(); // </table>
-            }
-        }
+        xmlWriter().writeEndElement(); // </ul>
     }
 }
 
@@ -3416,8 +3345,7 @@ QString DitaXmlGenerator::getMarkedUpSynopsis(const Node* node,
  */
 void DitaXmlGenerator::writeText(const QString& markedCode,
                                  CodeMarker* marker,
-                                 const Node* relative,
-                                 bool nameAlignment)
+                                 const Node* relative)
 {
     QString src = markedCode;
     QString html;
@@ -3487,19 +3415,9 @@ void DitaXmlGenerator::writeText(const QString& markedCode,
         // 0       1         2           3       4        5
         "link", "type", "headerfile", "func", "param", "extra"
     };
-    bool done = false;
+
     for (int i = 0, n = src.size(); i < n;) {
         if (src.at(i) == charLangle && src.at(i + 1) == charAt) {
-            if (nameAlignment && !done) {
-                if (!html.isEmpty()) {
-                    xmlWriter().writeCharacters(html);
-                    html.clear();
-                }
-                xmlWriter().writeEndElement(); // </<entry>
-                xmlWriter().writeStartElement("entry");
-                xmlWriter().writeAttribute("outputclass","memItemRight bottomAlign");
-                done = true;
-            }
             i += 2;
             bool handled = false;
             for (int k = 0; k != 6; ++k) {
diff --git a/tools/qdoc3/ditaxmlgenerator.h b/tools/qdoc3/ditaxmlgenerator.h
index cf26274..8bd9406 100644
--- a/tools/qdoc3/ditaxmlgenerator.h
+++ b/tools/qdoc3/ditaxmlgenerator.h
@@ -208,8 +208,7 @@ class DitaXmlGenerator : public PageGenerator
                                       CodeMarker* marker);
     void writeText(const QString& markedCode, 
                    CodeMarker* marker, 
-                   const Node* relative,
-                   bool nameAlignment = false);
+                   const Node* relative);
 
     void generateFullName(const Node* apparentNode, 
                           const Node* relative, 
-- 
cgit v0.12