[Libreoffice-commits] core.git: filter/source

[Xisco Fauli]

 filter/source/svg/svgreader.cxx |   18 ++++++++----------
 1 file changed, 8 insertions(+), 10 deletions(-)

New commits:
commit f268e6f1fdd741ebdb5256ed9c72643846472332
Author: Xisco Fauli <anistenis at gmail.com>
Date:   Tue Dec 1 18:59:26 2015 +0100

    SVG: Revert previous commits for tdf#95332 and tdf#95733
    
    I was wrong for thinking that there was a problem when
    elements inside def elements were not written.
    Actually, Cristina's commit
    ( 5e27d23d7e665cc0aeac8fc9ea2236bf8bc088ff ) was correct
    as elements inside defs don't have to be written at least
    they're refered by <use> element, which isn't supported in Libo
    right now. Working on implementing support for use element
    
    Change-Id: Id0b8d8e9b27887f75cc961881daeb82089b2e62b
    Reviewed-on: https://gerrit.libreoffice.org/20333
    Tested-by: Jenkins <ci at libreoffice.org>
    Reviewed-by: Katarina Behrens <Katarina.Behrens at cib.de>

diff --git a/filter/source/svg/svgreader.cxx b/filter/source/svg/svgreader.cxx
index 730b42c..b9737f1 100644
--- a/filter/source/svg/svgreader.cxx
+++ b/filter/source/svg/svgreader.cxx
@@ -88,22 +88,20 @@ template<typename Func> void visitElements(Func& rFunc,
     // notify children processing
     rFunc.push();
 
+    if (eCaller == SHAPE_WRITER && rElem->getTagName() == "defs")
+        return;
+
     // recurse over children
     uno::Reference<xml::dom::XNodeList> xChildren( rElem->getChildNodes() );
     const sal_Int32 nNumNodes( xChildren->getLength() );
     for( sal_Int32 i=0; i<nNumNodes; ++i )
     {
         if( xChildren->item(i)->getNodeType() == xml::dom::NodeType_ELEMENT_NODE ){
-            //tdf#65864
-            //TODO: support clipPath
-            if( !(eCaller == SHAPE_WRITER &&
-                rElem->getTagName() == "clipPath")){
-                    visitElements( rFunc,
-                                   uno::Reference<xml::dom::XElement>(
-                                       xChildren->item(i),
-                                       uno::UNO_QUERY_THROW),
-                                   eCaller );
-            }
+            visitElements( rFunc,
+                uno::Reference<xml::dom::XElement>(
+                xChildren->item(i),
+                uno::UNO_QUERY_THROW),
+                eCaller );
         }
     }