sw/source/core/docnode/node.cxx |   10 +---------
 1 file changed, 1 insertion(+), 9 deletions(-)

New commits:
commit b39b15c639960a37ed53d82cef44e64ef58c141e
Author:     Miklos Vajna <vmik...@collabora.com>
AuthorDate: Fri Jun 14 16:43:18 2024 +0200
Commit:     Miklos Vajna <vmik...@collabora.com>
CommitDate: Fri Jun 14 20:00:27 2024 +0200

    sw doc model xml dump: SwStartNode::dumpAsXml() is never called for tables
    
    SwTableNode is a subclass of SwStartNode, and SwTableNode provides its
    own SwTableNode::dumpAsXml(), so this is dead code.
    
    Change-Id: I051398c0e004839b750ea642dc636eeab8d73fa4
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/168891
    Reviewed-by: Miklos Vajna <vmik...@collabora.com>
    Tested-by: Jenkins

diff --git a/sw/source/core/docnode/node.cxx b/sw/source/core/docnode/node.cxx
index ffa6679a5e05..a7edb3a7e6ac 100644
--- a/sw/source/core/docnode/node.cxx
+++ b/sw/source/core/docnode/node.cxx
@@ -1002,14 +1002,8 @@ void SwStartNode::CheckSectionCondColl() const
 void SwStartNode::dumpAsXml(xmlTextWriterPtr pWriter) const
 {
     const char* pName = "???";
-    switch (GetNodeType())
+    switch(GetStartNodeType())
     {
-    case SwNodeType::Table:
-        pName = "table";
-        break;
-    default:
-        switch(GetStartNodeType())
-        {
         case SwNormalStartNode:
             pName = "start";
             break;
@@ -1028,8 +1022,6 @@ void SwStartNode::dumpAsXml(xmlTextWriterPtr pWriter) 
const
         case SwFooterStartNode:
             pName = "footer";
             break;
-        }
-        break;
     }
 
     (void)xmlTextWriterStartElement(pWriter, BAD_CAST(pName));

Reply via email to