writerfilter/source/dmapper/NumberingManager.cxx |   10 ++++------
 1 file changed, 4 insertions(+), 6 deletions(-)

New commits:
commit 2c364d7564ee24d119c21d1738d78982303c4cdc
Author:     Mike Kaganski <mike.kagan...@collabora.com>
AuthorDate: Sat Apr 8 10:01:53 2023 +0200
Commit:     Mike Kaganski <mike.kagan...@collabora.com>
CommitDate: Sat Apr 8 13:55:10 2023 +0200

    Simplify a bit
    
    Avoids creation of immediately destructed object in case of outline
    
    Change-Id: I72bfb4c5b4020619b61d432c3eee59304fce9795
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/150144
    Tested-by: Jenkins
    Reviewed-by: Mike Kaganski <mike.kagan...@collabora.com>

diff --git a/writerfilter/source/dmapper/NumberingManager.cxx 
b/writerfilter/source/dmapper/NumberingManager.cxx
index ea4fb36ec4a2..717eb5ea9788 100644
--- a/writerfilter/source/dmapper/NumberingManager.cxx
+++ b/writerfilter/source/dmapper/NumberingManager.cxx
@@ -531,17 +531,15 @@ void ListDef::CreateNumberingRules( DomainMapper& 
rDMapper,
     try
     {
         // Create the numbering style
-        uno::Reference< beans::XPropertySet > xStyle (
-            xFactory->createInstance("com.sun.star.style.NumberingStyle"),
-            uno::UNO_QUERY_THROW );
-
         if (GetId() == nOutline)
             m_StyleName = "Outline"; //SwNumRule.GetOutlineRuleName()
         else
-            xStyles->insertByName(GetStyleName(GetId(), xStyles), 
css::uno::Any(xStyle));
+            xStyles->insertByName(
+                GetStyleName(GetId(), xStyles),
+                
css::uno::Any(xFactory->createInstance("com.sun.star.style.NumberingStyle")));
 
         uno::Any oStyle = xStyles->getByName(GetStyleName());
-        xStyle.set( oStyle, uno::UNO_QUERY_THROW );
+        uno::Reference< beans::XPropertySet > xStyle( oStyle, 
uno::UNO_QUERY_THROW );
 
         // Get the default OOo Numbering style rules
         uno::Any aRules = xStyle->getPropertyValue( getPropertyName( 
PROP_NUMBERING_RULES ) );

Reply via email to