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

[Stephan Bergmann (via logerrit)]

 filter/source/svg/svgexport.cxx |   15 +++++----------
 1 file changed, 5 insertions(+), 10 deletions(-)

New commits:
commit f9f88e2c4fbd33bbe60a8df34bfd1cffaf5ee6df
Author:     Stephan Bergmann <sbergman at redhat.com>
AuthorDate: Tue Aug 4 14:28:50 2020 +0200
Commit:     Stephan Bergmann <sbergman at redhat.com>
CommitDate: Tue Aug 4 17:29:10 2020 +0200

    loplugin:referencecasting (clang-cl)
    
    ("the source reference is already a subtype of the destination reference")
    
    Change-Id: I1183eaa2015bd9d6567001bb14f7809dffcae7ba
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/100088
    Tested-by: Jenkins
    Reviewed-by: Stephan Bergmann <sbergman at redhat.com>

diff --git a/filter/source/svg/svgexport.cxx b/filter/source/svg/svgexport.cxx
index 76ad8c3b2ca2..f28d9811cf64 100644
--- a/filter/source/svg/svgexport.cxx
+++ b/filter/source/svg/svgexport.cxx
@@ -1593,16 +1593,11 @@ bool SVGFilter::implExportMasterPages( const std::vector< Reference< css::drawin
     {
         if( rxPages[i].is() )
         {
-            Reference< css::drawing::XShapes > xShapes( rxPages[i], UNO_QUERY );
+            // add id attribute
+            const OUString & sPageId = implGetValidIDFromInterface( rxPages[i] );
+            mpSVGExport->AddAttribute( XML_NAMESPACE_NONE, "id", sPageId );
 
-            if( xShapes.is() )
-            {
-                // add id attribute
-                const OUString & sPageId = implGetValidIDFromInterface( rxPages[i] );
-                mpSVGExport->AddAttribute( XML_NAMESPACE_NONE, "id", sPageId );
-
-                bRet = implExportPage( sPageId, rxPages[i], xShapes, true /* is a master page */ ) || bRet;
-            }
+            bRet = implExportPage( sPageId, rxPages[i], rxPages[i], true /* is a master page */ ) || bRet;
         }
     }
     return bRet;
@@ -1654,7 +1649,7 @@ void SVGFilter::implExportDrawPages( const std::vector< Reference< css::drawing:
             }
             else
             {
-                xShapes.set( rxPages[i], UNO_QUERY );
+                xShapes = rxPages[i];
             }
 
             if( xShapes.is() )