filter/source/svg/svgfilter.cxx |   12 ++++--------
 1 file changed, 4 insertions(+), 8 deletions(-)

New commits:
commit ac4b6b0a925236426bc2a8240bbde36d047696d9
Author:     Mike Kaganski <[email protected]>
AuthorDate: Tue Dec 2 12:39:10 2025 +0500
Commit:     Mike Kaganski <[email protected]>
CommitDate: Tue Dec 2 13:04:40 2025 +0100

    Don't use intermediate XInterface: XDrawPage is returned and needed
    
    Change-Id: Ia32b8fcae296eb3fd12ec2f60517ce2415d82fd0
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/194904
    Reviewed-by: Mike Kaganski <[email protected]>
    Tested-by: Jenkins

diff --git a/filter/source/svg/svgfilter.cxx b/filter/source/svg/svgfilter.cxx
index 7e129af41c92..ca9b07a80b24 100644
--- a/filter/source/svg/svgfilter.cxx
+++ b/filter/source/svg/svgfilter.cxx
@@ -519,7 +519,7 @@ bool SVGFilter::filterImpressOrDraw( const Sequence< 
PropertyValue >& rDescripto
              *  We get all master page that are targeted by at least one draw 
page.
              *  The master page are put in an unordered set.
              */
-            ObjectSet aMasterPageTargetSet;
+            std::unordered_set<uno::Reference<drawing::XDrawPage>> 
aMasterPageTargetSet;
             for(const uno::Reference<drawing::XDrawPage> & mSelectedPage : 
mSelectedPages)
             {
                 uno::Reference< drawing::XMasterPageTarget > 
xMasterPageTarget( mSelectedPage, uno::UNO_QUERY );
@@ -528,13 +528,9 @@ bool SVGFilter::filterImpressOrDraw( const Sequence< 
PropertyValue >& rDescripto
                     aMasterPageTargetSet.insert( 
xMasterPageTarget->getMasterPage() );
                 }
             }
-            // Later we move them to an uno::Sequence so we can get them by 
index
-            mMasterPageTargets.resize( aMasterPageTargetSet.size() );
-            sal_Int32 i = 0;
-            for (auto const& masterPageTarget : aMasterPageTargetSet)
-            {
-                mMasterPageTargets[i++].set(masterPageTarget,  uno::UNO_QUERY);
-            }
+            // Later we move them to a vector so we can get them by index
+            mMasterPageTargets.insert(mMasterPageTargets.end(), 
aMasterPageTargetSet.begin(),
+                                      aMasterPageTargetSet.end());
 
             bRet = implExport( rDescriptor );
         }

Reply via email to