[Libreoffice-commits] core.git: sfx2/qa sfx2/source

Noel Grandin noel.grandin at collabora.co.uk
Mon Jun 26 07:18:10 UTC 2017


 sfx2/qa/cppunit/test_metadatable.cxx       |    6 -
 sfx2/source/appl/appdispatchprovider.cxx   |    5 
 sfx2/source/appl/appserv.cxx               |  149 ++++++++++++++---------------
 sfx2/source/appl/shutdownicon.cxx          |    8 -
 sfx2/source/control/dispatch.cxx           |    3 
 sfx2/source/control/emojiviewitem.cxx      |    3 
 sfx2/source/control/recentdocsviewitem.cxx |    3 
 sfx2/source/control/templatelocalview.cxx  |    3 
 sfx2/source/control/templateviewitem.cxx   |    3 
 sfx2/source/dialog/dinfdlg.cxx             |    3 
 sfx2/source/dialog/filedlghelper.cxx       |    3 
 sfx2/source/dialog/printopt.cxx            |    3 
 sfx2/source/dialog/templdlg.cxx            |    8 -
 sfx2/source/doc/docfilt.cxx                |    3 
 sfx2/source/doc/doctempl.cxx               |    6 -
 sfx2/source/doc/doctemplates.cxx           |   15 --
 sfx2/source/doc/guisaveas.cxx              |    4 
 sfx2/source/doc/objserv.cxx                |   22 ----
 sfx2/source/doc/objstor.cxx                |    7 -
 sfx2/source/doc/sfxbasemodel.cxx           |   17 ---
 sfx2/source/doc/templatedlg.cxx            |    9 -
 sfx2/source/inet/inettbc.cxx               |    4 
 sfx2/source/sidebar/Theme.cxx              |    3 
 sfx2/source/view/frame2.cxx                |    3 
 sfx2/source/view/sfxbasecontroller.cxx     |    5 
 sfx2/source/view/viewfrm2.cxx              |    3 
 sfx2/source/view/viewprn.cxx               |    3 
 sfx2/source/view/viewsh.cxx                |   12 --
 28 files changed, 119 insertions(+), 197 deletions(-)

New commits:
commit 2dbe6a2d57956be392966c363d03f54cb68dd4fa
Author: Noel Grandin <noel.grandin at collabora.co.uk>
Date:   Sun Jun 25 19:11:12 2017 +0200

    loplugin:oncevar in sfx2
    
    Change-Id: I9aadcb9a12ef9b0c3f93ea255f2542e51d31fbf7
    Reviewed-on: https://gerrit.libreoffice.org/39242
    Reviewed-by: Noel Grandin <noel.grandin at collabora.co.uk>
    Tested-by: Noel Grandin <noel.grandin at collabora.co.uk>

diff --git a/sfx2/qa/cppunit/test_metadatable.cxx b/sfx2/qa/cppunit/test_metadatable.cxx
index bb883b6a9cc2..d469859c4cf0 100644
--- a/sfx2/qa/cppunit/test_metadatable.cxx
+++ b/sfx2/qa/cppunit/test_metadatable.cxx
@@ -88,9 +88,9 @@ void MetadatableTest::test()
     MockMetadatable m5(*pReg);
     OUString empty;
     OUString content( "content.xml" );
-    OUString styles( "styles.xml" );
-    OUString sid1( "id1" );
-    OUString sid2( "id2" );
+    OUString const styles( "styles.xml" );
+    OUString const sid1( "id1" );
+    OUString const sid2( "id2" );
     OUString sid3( "id3" );
     OUString sid4( "id4" );
     beans::StringPair id1(content, sid1);
diff --git a/sfx2/source/appl/appdispatchprovider.cxx b/sfx2/source/appl/appdispatchprovider.cxx
index e840361d2b3f..e7e70a878cf2 100644
--- a/sfx2/source/appl/appdispatchprovider.cxx
+++ b/sfx2/source/appl/appdispatchprovider.cxx
@@ -205,7 +205,6 @@ Sequence< frame::DispatchInformation > SAL_CALL SfxAppDispatchProvider::getConfi
     SfxSlotPool& rAppSlotPool = SfxGetpApp()->GetAppSlotPool_Impl();
 
     const SfxSlotMode nMode( SfxSlotMode::TOOLBOXCONFIG|SfxSlotMode::ACCELCONFIG|SfxSlotMode::MENUCONFIG );
-    OUString aCmdPrefix( ".uno:" );
 
     // Select group ( group 0 is internal )
     for (sal_uInt16 i=0; i< rAppSlotPool.GetGroupCount(); ++i)
@@ -222,9 +221,7 @@ Sequence< frame::DispatchInformation > SAL_CALL SfxAppDispatchProvider::getConfi
                     if ( pSfxSlot->GetMode() & nMode )
                     {
                         frame::DispatchInformation aCmdInfo;
-                        OUStringBuffer aBuf( aCmdPrefix );
-                        aBuf.appendAscii( pSfxSlot->GetUnoName() );
-                        aCmdInfo.Command = aBuf.makeStringAndClear();
+                        aCmdInfo.Command = ".uno:" + OUString::createFromAscii(pSfxSlot->GetUnoName());
                         aCmdInfo.GroupId = nCommandGroup;
                         aCmdList.push_back( aCmdInfo );
                     }
diff --git a/sfx2/source/appl/appserv.cxx b/sfx2/source/appl/appserv.cxx
index 4ee7dc17a2fe..efdd469b424e 100644
--- a/sfx2/source/appl/appserv.cxx
+++ b/sfx2/source/appl/appserv.cxx
@@ -791,7 +791,6 @@ void SfxApplication::MiscExec_Impl( SfxRequest& rReq )
                     css::uno::Sequence<OUString> aUserToolbars;
                     std::vector<OUString> aBackupList;
                     OUString aSidebarMode;
-                    bool bCorrectMode = true;
 
                     OUStringBuffer aPath = OUStringBuffer( "org.openoffice.Office.UI.ToolbarMode/Applications/" );
                     aPath.append( lcl_getAppName( eApp ) );
@@ -828,103 +827,100 @@ void SfxApplication::MiscExec_Impl( SfxRequest& rReq )
                         }
                     }
 
-                    if ( bCorrectMode )
+                    // Backup visible toolbar list and hide all toolbars
+                    Sequence<Reference<XUIElement>> aUIElements = xLayoutManager->getElements();
+                    for ( sal_Int32 i = 0; i < aUIElements.getLength(); i++ )
                     {
-                        // Backup visible toolbar list and hide all toolbars
-                        Sequence<Reference<XUIElement>> aUIElements = xLayoutManager->getElements();
-                        for ( sal_Int32 i = 0; i < aUIElements.getLength(); i++ )
+                        Reference< XUIElement > xUIElement( aUIElements[i] );
+                        Reference< XPropertySet > xPropertySet( aUIElements[i], UNO_QUERY );
+                        if ( xPropertySet.is() && xUIElement.is() )
                         {
-                            Reference< XUIElement > xUIElement( aUIElements[i] );
-                            Reference< XPropertySet > xPropertySet( aUIElements[i], UNO_QUERY );
-                            if ( xPropertySet.is() && xUIElement.is() )
+                            try
                             {
-                                try
-                                {
-                                    OUString aResName;
-                                    sal_Int16 nType( -1 );
-                                    xPropertySet->getPropertyValue( "Type" ) >>= nType;
-                                    xPropertySet->getPropertyValue( "ResourceURL" ) >>= aResName;
+                                OUString aResName;
+                                sal_Int16 nType( -1 );
+                                xPropertySet->getPropertyValue( "Type" ) >>= nType;
+                                xPropertySet->getPropertyValue( "ResourceURL" ) >>= aResName;
 
-                                    if (( nType == css::ui::UIElementType::TOOLBAR ) &&
-                                        !aResName.isEmpty() )
+                                if (( nType == css::ui::UIElementType::TOOLBAR ) &&
+                                    !aResName.isEmpty() )
+                                {
+                                    if ( xLayoutManager->isElementVisible( aResName ) )
                                     {
-                                        if ( xLayoutManager->isElementVisible( aResName ) )
-                                        {
-                                            aBackupList.push_back( aResName );
-                                        }
-                                        xLayoutManager->hideElement( aResName );
+                                        aBackupList.push_back( aResName );
                                     }
-                                }
-                                catch ( const Exception& )
-                                {
+                                    xLayoutManager->hideElement( aResName );
                                 }
                             }
+                            catch ( const Exception& )
+                            {
+                            }
                         }
+                    }
+
+                    // Show toolbars
+                    for ( OUString& rName : aMandatoryToolbars )
+                    {
+                        xLayoutManager->createElement( rName );
+                        xLayoutManager->showElement( rName );
+                    }
+
+                    for ( OUString& rName : aUserToolbars )
+                    {
+                        xLayoutManager->createElement( rName );
+                        xLayoutManager->showElement( rName );
+                    }
+
+                    // Sidebar
+                    pViewFrame->ShowChildWindow( SID_SIDEBAR );
 
-                        // Show toolbars
-                        for ( OUString& rName : aMandatoryToolbars )
+                    sfx2::sidebar::SidebarController* pSidebar =
+                            sfx2::sidebar::SidebarController::GetSidebarControllerForFrame( xFrame );
+                    if ( pSidebar )
+                    {
+                        if ( aSidebarMode.compareTo( "Arrow" ) == 0 )
                         {
-                            xLayoutManager->createElement( rName );
-                            xLayoutManager->showElement( rName );
+                            pSidebar->FadeOut();
                         }
-
-                        for ( OUString& rName : aUserToolbars )
+                        else if ( aSidebarMode.compareTo( "Tabs" ) == 0 )
                         {
-                            xLayoutManager->createElement( rName );
-                            xLayoutManager->showElement( rName );
+                            pSidebar->FadeIn();
+                            pSidebar->RequestOpenDeck();
+                            pSidebar->RequestCloseDeck();
                         }
-
-                        // Sidebar
-                        pViewFrame->ShowChildWindow( SID_SIDEBAR );
-
-                        sfx2::sidebar::SidebarController* pSidebar =
-                                sfx2::sidebar::SidebarController::GetSidebarControllerForFrame( xFrame );
-                        if ( pSidebar )
+                        else if ( aSidebarMode.compareTo( "Opened" ) == 0 )
                         {
-                            if ( aSidebarMode.compareTo( "Arrow" ) == 0 )
-                            {
-                                pSidebar->FadeOut();
-                            }
-                            else if ( aSidebarMode.compareTo( "Tabs" ) == 0 )
-                            {
-                                pSidebar->FadeIn();
-                                pSidebar->RequestOpenDeck();
-                                pSidebar->RequestCloseDeck();
-                            }
-                            else if ( aSidebarMode.compareTo( "Opened" ) == 0 )
-                            {
-                                pSidebar->FadeIn();
-                                pSidebar->RequestOpenDeck();
-                            }
+                            pSidebar->FadeIn();
+                            pSidebar->RequestOpenDeck();
                         }
+                    }
 
-                        // Show/Hide the Notebookbar
-                        const SfxPoolItem* pItem;
-                        pViewFrame->GetDispatcher()->QueryState( SID_NOTEBOOKBAR, pItem );
+                    // Show/Hide the Notebookbar
+                    const SfxPoolItem* pItem;
+                    pViewFrame->GetDispatcher()->QueryState( SID_NOTEBOOKBAR, pItem );
 
-                        // Save settings
-                        if ( pViewFrame == SfxViewFrame::Current() )
-                        {
-                            css::uno::Sequence<OUString> aBackup( aBackupList.size() );
-                            for ( size_t i = 0; i < aBackupList.size(); ++i )
-                                aBackup[i] = aBackupList[i];
+                    // Save settings
+                    if ( pViewFrame == SfxViewFrame::Current() )
+                    {
+                        css::uno::Sequence<OUString> aBackup( aBackupList.size() );
+                        for ( size_t i = 0; i < aBackupList.size(); ++i )
+                            aBackup[i] = aBackupList[i];
 
-                            for ( sal_Int32 nReadIndex = 0; nReadIndex < nCount; ++nReadIndex )
-                            {
-                                const utl::OConfigurationNode aModeNode( aModesNode.openNode( aModeNodeNames[nReadIndex] ) );
-                                if ( !aModeNode.isValid() )
-                                    continue;
+                        for ( sal_Int32 nReadIndex = 0; nReadIndex < nCount; ++nReadIndex )
+                        {
+                            const utl::OConfigurationNode aModeNode( aModesNode.openNode( aModeNodeNames[nReadIndex] ) );
+                            if ( !aModeNode.isValid() )
+                                continue;
 
-                                OUString aCommandArg = comphelper::getString( aModeNode.getNodeValue( "CommandArg" ) );
+                            OUString aCommandArg = comphelper::getString( aModeNode.getNodeValue( "CommandArg" ) );
 
-                                if ( aCommandArg.compareTo( aCurrentMode ) == 0 )
-                                {
-                                    aModeNode.setNodeValue( "UserToolbars", makeAny( aBackup ) );
-                                    break;
-                                }
+                            if ( aCommandArg.compareTo( aCurrentMode ) == 0 )
+                            {
+                                aModeNode.setNodeValue( "UserToolbars", makeAny( aBackup ) );
+                                break;
                             }
-                            aModesNode.commit();
                         }
+                        aModesNode.commit();
                     }
                 }
 
@@ -963,8 +959,7 @@ void SfxApplication::MiscExec_Impl( SfxRequest& rReq )
 
                 if ( xLayoutManager.is() )
                 {
-                    OUString aToolbarResName( "private:resource/toolbar/" );
-                    OUStringBuffer aBuf( aToolbarResName );
+                    OUStringBuffer aBuf( "private:resource/toolbar/" );
                     aBuf.append( pToolbarName->GetValue() );
 
                     // Evaluate Parameter
diff --git a/sfx2/source/appl/shutdownicon.cxx b/sfx2/source/appl/shutdownicon.cxx
index 7496e6aa153f..031c395cf3c0 100644
--- a/sfx2/source/appl/shutdownicon.cxx
+++ b/sfx2/source/appl/shutdownicon.cxx
@@ -377,13 +377,11 @@ IMPL_LINK( ShutdownIcon, DialogClosedHdl_Impl, FileDialogHelper*, /*unused*/, vo
                 aArgs[0].Name = "InteractionHandler";
                 aArgs[0].Value <<= xInteraction;
 
-                sal_Int16 nMacroExecMode = css::document::MacroExecMode::USE_CONFIG;
                 aArgs[1].Name = "MacroExecutionMode";
-                aArgs[1].Value <<= nMacroExecMode;
+                aArgs[1].Value <<= sal_Int16(css::document::MacroExecMode::USE_CONFIG);
 
-                sal_Int16 nUpdateDoc = css::document::UpdateDocMode::ACCORDING_TO_CONFIG;
                 aArgs[2].Name = "UpdateDocMode";
-                aArgs[2].Value <<= nUpdateDoc;
+                aArgs[2].Value <<= sal_Int16(css::document::UpdateDocMode::ACCORDING_TO_CONFIG);
 
                 // use the filedlghelper to get the current filter name,
                 // because it removes the extensions before you get the filter name.
@@ -717,6 +715,7 @@ OUString ShutdownIcon::getShortcutName()
     return OUString();
 #else
 
+#ifdef _WIN32
     OUString aShortcutName( "StarOffice 6.0"  );
     ResMgr* pMgr = SfxResMgr::GetResMgr();
     if( pMgr )
@@ -724,7 +723,6 @@ OUString ShutdownIcon::getShortcutName()
         ::SolarMutexGuard aGuard;
         aShortcutName = SfxResId(STR_QUICKSTART_LNKNAME);
     }
-#ifdef _WIN32
     aShortcutName += ".lnk";
 
     OUString aShortcut(GetAutostartFolderNameW32());
diff --git a/sfx2/source/control/dispatch.cxx b/sfx2/source/control/dispatch.cxx
index d3ec2a340bfd..f00259b1356f 100644
--- a/sfx2/source/control/dispatch.cxx
+++ b/sfx2/source/control/dispatch.cxx
@@ -931,8 +931,7 @@ const SfxSlot* SfxDispatcher::GetSlot( const OUString& rCommand )
         }
     }
 
-    sal_uInt16 nFirstShell = 0;
-    for ( sal_uInt16 i = nFirstShell; i < nTotCount; ++i )
+    for ( sal_uInt16 i = 0; i < nTotCount; ++i )
     {
         SfxShell *pObjShell = GetShell(i);
         SfxInterface *pIFace = pObjShell->GetInterface();
diff --git a/sfx2/source/control/emojiviewitem.cxx b/sfx2/source/control/emojiviewitem.cxx
index ca3be4cc56e6..9b4c8c7888d3 100644
--- a/sfx2/source/control/emojiviewitem.cxx
+++ b/sfx2/source/control/emojiviewitem.cxx
@@ -64,8 +64,7 @@ void EmojiViewItem::Paint(drawinglayer::processor2d::BaseProcessor2D *pProcessor
 {
     BColor aFillColor = pAttrs->aFillColor;
 
-    int nCount = 2;
-    drawinglayer::primitive2d::Primitive2DContainer aSeq(nCount);
+    drawinglayer::primitive2d::Primitive2DContainer aSeq(2);
     double fTransparence = 0.0;
 
     // Draw background
diff --git a/sfx2/source/control/recentdocsviewitem.cxx b/sfx2/source/control/recentdocsviewitem.cxx
index 289edbb7ba19..c1dc04d49008 100644
--- a/sfx2/source/control/recentdocsviewitem.cxx
+++ b/sfx2/source/control/recentdocsviewitem.cxx
@@ -196,8 +196,7 @@ void RecentDocsViewItem::OpenDocument()
     Reference<util::XURLTransformer> xTrans(util::URLTransformer::create(::comphelper::getProcessComponentContext()));
     xTrans->parseStrict(aTargetURL);
 
-    sal_Int32 nSize = 2;
-    aArgsList.realloc(nSize);
+    aArgsList.realloc(2);
     aArgsList[0].Name = "Referer";
     aArgsList[0].Value <<= OUString("private:user");
 
diff --git a/sfx2/source/control/templatelocalview.cxx b/sfx2/source/control/templatelocalview.cxx
index 57acf992c8cb..83f25c641165 100644
--- a/sfx2/source/control/templatelocalview.cxx
+++ b/sfx2/source/control/templatelocalview.cxx
@@ -396,13 +396,12 @@ sal_uInt16 TemplateLocalView::createRegion(const OUString &rName)
 
 bool TemplateLocalView::renameRegion(const OUString &rTitle, const OUString &rNewTitle)
 {
-    sal_uInt16 nDocId = USHRT_MAX;
     TemplateContainerItem *pRegion = getRegion(rTitle);
 
     if(pRegion)
     {
         sal_uInt16 nRegionId = pRegion->mnRegionId;
-        return mpDocTemplates->SetName( rNewTitle, nRegionId, nDocId );
+        return mpDocTemplates->SetName( rNewTitle, nRegionId, USHRT_MAX/*nDocId*/ );
     }
     return false;
 }
diff --git a/sfx2/source/control/templateviewitem.cxx b/sfx2/source/control/templateviewitem.cxx
index ffa069410ab0..36c4242e5991 100644
--- a/sfx2/source/control/templateviewitem.cxx
+++ b/sfx2/source/control/templateviewitem.cxx
@@ -59,8 +59,7 @@ void TemplateViewItem::Paint(drawinglayer::processor2d::BaseProcessor2D *pProces
 {
     BColor aFillColor = pAttrs->aFillColor;
 
-    int nCount = 5;
-    drawinglayer::primitive2d::Primitive2DContainer aSeq(nCount);
+    drawinglayer::primitive2d::Primitive2DContainer aSeq(5);
     double fTransparence = 0.0;
 
     // Draw background
diff --git a/sfx2/source/dialog/dinfdlg.cxx b/sfx2/source/dialog/dinfdlg.cxx
index 4524caefe1af..c749ca3b2137 100644
--- a/sfx2/source/dialog/dinfdlg.cxx
+++ b/sfx2/source/dialog/dinfdlg.cxx
@@ -897,11 +897,10 @@ void SfxDocumentPage::ImplUpdateSignatures()
                 s = m_aMultiSignedStr;
             else if ( aInfos.getLength() == 1 )
             {
-                OUString aCN_Id("CN");
                 const security::DocumentSignatureInformation& rInfo = aInfos[ 0 ];
                 s = GetDateTimeString( rInfo.SignatureDate, rInfo.SignatureTime );
                 s += ", ";
-                s += GetContentPart( rInfo.Signer->getSubjectName(), aCN_Id );
+                s += GetContentPart( rInfo.Signer->getSubjectName(), "CN" );
             }
             m_pSignedValFt->SetText( s );
         }
diff --git a/sfx2/source/dialog/filedlghelper.cxx b/sfx2/source/dialog/filedlghelper.cxx
index acb10f0d2c56..14140b6b8b51 100644
--- a/sfx2/source/dialog/filedlghelper.cxx
+++ b/sfx2/source/dialog/filedlghelper.cxx
@@ -453,14 +453,13 @@ void FileDialogHelper_Impl::updateExportButton()
     uno::Reference < XFilePickerControlAccess > xCtrlAccess( mxFileDlg, UNO_QUERY );
     if ( xCtrlAccess.is() )
     {
-        OUString sEllipses( "..."  );
         OUString sOldLabel( xCtrlAccess->getLabel( CommonFilePickerElementIds::PUSHBUTTON_OK ) );
 
         // initialize button label; we need the label with the mnemonic char
         if ( maButtonLabel.isEmpty() || maButtonLabel.indexOf( MNEMONIC_CHAR ) == -1 )
         {
             // cut the ellipses, if necessary
-            sal_Int32 nIndex = sOldLabel.indexOf( sEllipses );
+            sal_Int32 nIndex = sOldLabel.indexOf( "..." );
             if ( -1 == nIndex )
                 nIndex = sOldLabel.getLength();
             maButtonLabel = sOldLabel.copy( 0, nIndex );
diff --git a/sfx2/source/dialog/printopt.cxx b/sfx2/source/dialog/printopt.cxx
index 6a57f2622aaf..a9fa42dad284 100644
--- a/sfx2/source/dialog/printopt.cxx
+++ b/sfx2/source/dialog/printopt.cxx
@@ -145,7 +145,6 @@ bool SfxCommonPrintOptionsTabPage::FillItemSet( SfxItemSet* /*rSet*/ )
     SvtPrintWarningOptions  aWarnOptions;
     SvtPrinterOptions       aPrinterOptions;
     SvtPrintFileOptions     aPrintFileOptions;
-    bool                    bModified = false;
 
 
     if( m_pPaperSizeCB->IsValueChangedFromSaved())
@@ -161,7 +160,7 @@ bool SfxCommonPrintOptionsTabPage::FillItemSet( SfxItemSet* /*rSet*/ )
     aPrinterOptions.SetPrinterOptions( maPrinterOptions );
     aPrintFileOptions.SetPrinterOptions( maPrintFileOptions );
 
-    return bModified;
+    return false;
 }
 
 void SfxCommonPrintOptionsTabPage::Reset( const SfxItemSet* /*rSet*/ )
diff --git a/sfx2/source/dialog/templdlg.cxx b/sfx2/source/dialog/templdlg.cxx
index 48a874f34996..4013c0283e39 100644
--- a/sfx2/source/dialog/templdlg.cxx
+++ b/sfx2/source/dialog/templdlg.cxx
@@ -787,10 +787,7 @@ void SfxCommonTemplateDialog_Impl::ReadResource()
 
     LoadedFamilies();
 
-    sal_uInt16 nStart = SID_STYLE_FAMILY1;
-    sal_uInt16 nEnd = SID_STYLE_FAMILY4;
-
-    for ( i = nStart; i <= nEnd; i++ )
+    for ( i = SID_STYLE_FAMILY1; i <= SID_STYLE_FAMILY4; i++ )
         pBindings->Update(i);
 }
 
@@ -1763,8 +1760,7 @@ sal_Int32 SfxCommonTemplateDialog_Impl::LoadFactoryStyleFilter( SfxObjectShell*
 
     ::comphelper::SequenceAsHashMap aFactoryProps(
         xModuleManager->getByName( getModuleIdentifier( xModuleManager, i_pObjSh ) ) );
-    sal_Int32 nDefault = -1;
-    sal_Int32 nFilter = aFactoryProps.getUnpackedValueOrDefault( "ooSetupFactoryStyleFilter", nDefault );
+    sal_Int32 nFilter = aFactoryProps.getUnpackedValueOrDefault( "ooSetupFactoryStyleFilter", sal_Int32(-1) );
 
     m_bWantHierarchical =
         (nFilter & SFXSTYLEBIT_HIERARCHY) != 0;
diff --git a/sfx2/source/doc/docfilt.cxx b/sfx2/source/doc/docfilt.cxx
index 99406c4ef772..ed5a13c1be80 100644
--- a/sfx2/source/doc/docfilt.cxx
+++ b/sfx2/source/doc/docfilt.cxx
@@ -68,14 +68,13 @@ SfxFilter::SfxFilter( const OUString &rName,
     OUString aExts = GetWildcard().getGlob();
     OUString aShort, aLong;
     OUString aRet;
-    sal_uInt16 nMaxLength = USHRT_MAX;
     OUString aTest;
     sal_uInt16 nPos = 0;
     while (!(aRet = aExts.getToken(nPos++, ';')).isEmpty() )
     {
         aTest = aRet;
         aTest = aTest.replaceFirst( "*." , "" );
-        if( aTest.getLength() <= nMaxLength )
+        if( aTest.getLength() <= USHRT_MAX )
         {
             if (!aShort.isEmpty())
                 aShort += ";";
diff --git a/sfx2/source/doc/doctempl.cxx b/sfx2/source/doc/doctempl.cxx
index cca1bee7cabe..5a844d5591f4 100644
--- a/sfx2/source/doc/doctempl.cxx
+++ b/sfx2/source/doc/doctempl.cxx
@@ -1516,11 +1516,10 @@ void SfxDocTemplate_Impl::AddRegion( const OUString& rTitle,
 
     try
     {
-        ResultSetInclude eInclude = INCLUDE_DOCUMENTS_ONLY;
         Sequence< NumberedSortingInfo >     aSortingInfo(1);
         aSortingInfo.getArray()->ColumnIndex = 1;
         aSortingInfo.getArray()->Ascending = true;
-        xResultSet = rContent.createSortedCursor( aProps, aSortingInfo, m_rCompareFactory, eInclude );
+        xResultSet = rContent.createSortedCursor( aProps, aSortingInfo, m_rCompareFactory, INCLUDE_DOCUMENTS_ONLY );
     }
     catch ( Exception& ) {}
 
@@ -1547,11 +1546,10 @@ void SfxDocTemplate_Impl::CreateFromHierarchy( Content &rTemplRoot )
 
     try
     {
-        ResultSetInclude eInclude = INCLUDE_FOLDERS_ONLY;
         Sequence< NumberedSortingInfo >     aSortingInfo(1);
         aSortingInfo.getArray()->ColumnIndex = 1;
         aSortingInfo.getArray()->Ascending = true;
-        xResultSet = rTemplRoot.createSortedCursor( aProps, aSortingInfo, m_rCompareFactory, eInclude );
+        xResultSet = rTemplRoot.createSortedCursor( aProps, aSortingInfo, m_rCompareFactory, INCLUDE_FOLDERS_ONLY );
     }
     catch ( Exception& ) {}
 
diff --git a/sfx2/source/doc/doctemplates.cxx b/sfx2/source/doc/doctemplates.cxx
index 0ea4a1f093c2..c811fde66138 100644
--- a/sfx2/source/doc/doctemplates.cxx
+++ b/sfx2/source/doc/doctemplates.cxx
@@ -1522,8 +1522,7 @@ bool SfxDocTplService_Impl::removeGroup( const OUString& rGroupName )
 
         try
         {
-            ResultSetInclude eInclude = INCLUDE_DOCUMENTS_ONLY;
-            xResultSet = aGroup.createCursor( aProps, eInclude );
+            xResultSet = aGroup.createCursor( aProps, INCLUDE_DOCUMENTS_ONLY );
 
             if ( xResultSet.is() )
             {
@@ -1633,8 +1632,7 @@ bool SfxDocTplService_Impl::renameGroup( const OUString& rOldName,
     {
         uno::Reference< XResultSet > xResultSet;
         Sequence< OUString > aProps { TARGET_URL };
-        ResultSetInclude eInclude = INCLUDE_DOCUMENTS_ONLY;
-        xResultSet = aGroup.createCursor( aProps, eInclude );
+        xResultSet = aGroup.createCursor( aProps, INCLUDE_DOCUMENTS_ONLY );
 
         if ( xResultSet.is() )
         {
@@ -2328,8 +2326,7 @@ void SfxDocTplService_Impl::addHierGroup( GroupList_Impl& rList,
     try
     {
         aContent = Content(rOwnURL, maCmdEnv, comphelper::getProcessComponentContext());
-        ResultSetInclude eInclude = INCLUDE_DOCUMENTS_ONLY;
-        xResultSet = aContent.createCursor( aProps, eInclude );
+        xResultSet = aContent.createCursor( aProps, INCLUDE_DOCUMENTS_ONLY );
     }
     catch (ContentCreationException&)
     {
@@ -2439,8 +2436,7 @@ void SfxDocTplService_Impl::addFsysGroup( GroupList_Impl& rList,
         // that should happen quietly
         uno::Reference< XCommandEnvironment > aQuietEnv;
         aContent = Content( rOwnURL, aQuietEnv, comphelper::getProcessComponentContext() );
-        ResultSetInclude eInclude = INCLUDE_DOCUMENTS_ONLY;
-        xResultSet = aContent.createCursor( aProps, eInclude );
+        xResultSet = aContent.createCursor( aProps, INCLUDE_DOCUMENTS_ONLY );
     }
     catch ( Exception& ) {}
 
@@ -2500,8 +2496,7 @@ void SfxDocTplService_Impl::createFromContent( GroupList_Impl& rList,
 
     try
     {
-        ResultSetInclude eInclude = INCLUDE_FOLDERS_ONLY;
-        xResultSet = rContent.createCursor( aProps, eInclude );
+        xResultSet = rContent.createCursor( aProps, INCLUDE_FOLDERS_ONLY );
     }
     catch ( Exception& ) {}
 
diff --git a/sfx2/source/doc/guisaveas.cxx b/sfx2/source/doc/guisaveas.cxx
index 7adac1e93a7d..bd3c541f8d81 100644
--- a/sfx2/source/doc/guisaveas.cxx
+++ b/sfx2/source/doc/guisaveas.cxx
@@ -232,12 +232,10 @@ public:
     {
         if ( m_bRestoreSettings )
         {
-            OUString aLoadReadonlyString( "LoadReadonly" );
-
             try
             {
                 if ( m_bReadOnlySupported )
-                    m_xDocumentSettings->setPropertyValue( aLoadReadonlyString, uno::makeAny( m_bPreserveReadOnly ) );
+                    m_xDocumentSettings->setPropertyValue( "LoadReadonly", uno::makeAny( m_bPreserveReadOnly ) );
             }
             catch( const uno::Exception& )
             {
diff --git a/sfx2/source/doc/objserv.cxx b/sfx2/source/doc/objserv.cxx
index dc2cfa229ed1..e890a04ae430 100644
--- a/sfx2/source/doc/objserv.cxx
+++ b/sfx2/source/doc/objserv.cxx
@@ -729,28 +729,6 @@ void SfxObjectShell::ExecFile_Impl(SfxRequest &rReq)
 
         case SID_CLOSEDOC:
         {
-            SfxViewFrame *pFrame = GetFrame();
-
-            bool bInFrameSet = false;
-            sal_uInt16 nFrames=0;
-            pFrame = SfxViewFrame::GetFirst( this );
-            while ( pFrame )
-            {
-                nFrames++;
-                pFrame = SfxViewFrame::GetNext( *pFrame, this );
-            }
-
-            if ( bInFrameSet )
-            {
-                // Close all views that are not in a FrameSet.
-                pFrame = SfxViewFrame::GetFirst( this );
-                while ( pFrame )
-                {
-                    pFrame->GetFrame().DoClose();
-                    pFrame = SfxViewFrame::GetNext( *pFrame, this );
-                }
-            }
-
             // Evaluate Parameter
             const SfxBoolItem* pSaveItem = rReq.GetArg<SfxBoolItem>(SID_CLOSEDOC_SAVE);
             const SfxStringItem* pNameItem = rReq.GetArg<SfxStringItem>(SID_CLOSEDOC_FILENAME);
diff --git a/sfx2/source/doc/objstor.cxx b/sfx2/source/doc/objstor.cxx
index 3ae613ff60c8..6714c7a0ef94 100644
--- a/sfx2/source/doc/objstor.cxx
+++ b/sfx2/source/doc/objstor.cxx
@@ -3042,14 +3042,13 @@ uno::Reference< embed::XStorage > SfxObjectShell::GetStorage()
 
 bool SfxObjectShell::SaveChildren( bool bObjectsOnly )
 {
-    bool bResult = true;
     if ( pImpl->mpObjectContainer )
     {
         bool bOasis = ( SotStorage::GetVersion( GetStorage() ) > SOFFICE_FILEFORMAT_60 );
         GetEmbeddedObjectContainer().StoreChildren(bOasis,bObjectsOnly);
     }
 
-    return bResult;
+    return true;
 }
 
 bool SfxObjectShell::SaveAsChildren( SfxMedium& rMedium )
@@ -3118,12 +3117,10 @@ bool SfxObjectShell::SwitchChildrenPersistance( const uno::Reference< embed::XSt
         return false;
     }
 
-    bool bResult = true;
-
     if ( pImpl->mpObjectContainer )
         pImpl->mpObjectContainer->SetPersistentEntries(xStorage,bForceNonModified);
 
-    return bResult;
+    return true;
 }
 
 // Never call this method directly, always use the DoSaveCompleted call
diff --git a/sfx2/source/doc/sfxbasemodel.cxx b/sfx2/source/doc/sfxbasemodel.cxx
index e2ebee05f9bb..2c90f8280363 100644
--- a/sfx2/source/doc/sfxbasemodel.cxx
+++ b/sfx2/source/doc/sfxbasemodel.cxx
@@ -3430,14 +3430,12 @@ Reference< ui::XUIConfigurationManager2 > SfxBaseModel::getUIConfigurationManage
         if ( xConfigStorage.is() )
         {
             OUString aMediaTypeProp( "MediaType" );
-            OUString aUIConfigMediaType(
-                    "application/vnd.sun.xml.ui.configuration"  );
             OUString aMediaType;
             Reference< beans::XPropertySet > xPropSet( xConfigStorage, UNO_QUERY );
             Any a = xPropSet->getPropertyValue( aMediaTypeProp );
             if ( !( a >>= aMediaType ) ||  aMediaType.isEmpty())
             {
-                xPropSet->setPropertyValue( aMediaTypeProp, Any(aUIConfigMediaType) );
+                xPropSet->setPropertyValue( aMediaTypeProp, Any(OUString("application/vnd.sun.xml.ui.configuration")) );
             }
         }
         else
@@ -3452,10 +3450,9 @@ Reference< ui::XUIConfigurationManager2 > SfxBaseModel::getUIConfigurationManage
         {
             // Import old UI configuration from OOo 1.x
             Reference< embed::XStorage > xOOo1ConfigStorage;
-            OUString         aOOo1UIConfigFolderName( "Configurations" );
 
             // Try to open with READ
-            xOOo1ConfigStorage = getDocumentSubStorage( aOOo1UIConfigFolderName, embed::ElementModes::READ );
+            xOOo1ConfigStorage = getDocumentSubStorage( "Configurations", embed::ElementModes::READ );
             if ( xOOo1ConfigStorage.is() )
             {
                 Reference< XComponentContext > xContext( ::comphelper::getProcessComponentContext() );
@@ -3467,12 +3464,10 @@ Reference< ui::XUIConfigurationManager2 > SfxBaseModel::getUIConfigurationManage
                 {
                     SfxObjectShell* pObjShell = SfxBaseModel::GetObjectShell();
 
-                    OUString aNum( "private:resource/toolbar/custom_OOo1x_" );
-                    OUString aTitle( "Toolbar " );
                     for ( size_t i = 0; i < rToolbars.size(); i++ )
                     {
-                        OUString aCustomTbxName = aNum + OUString::number( i + 1 );
-                        OUString aCustomTbxTitle = aTitle + OUString::number( i + 1 );
+                        OUString aCustomTbxName = "private:resource/toolbar/custom_OOo1x_" + OUString::number( i + 1 );
+                        OUString aCustomTbxTitle = "Toolbar " + OUString::number( i + 1 );
 
                         Reference< container::XIndexContainer > xToolbar = rToolbars[i];
                         ConvertSlotsToCommands( pObjShell, xToolbar );
@@ -3484,9 +3479,7 @@ Reference< ui::XUIConfigurationManager2 > SfxBaseModel::getUIConfigurationManage
                             {
                                 try
                                 {
-                                    OUString aPropName( "UIName" );
-                                    Any           aAny( aCustomTbxTitle );
-                                    xPropSet->setPropertyValue( aPropName, aAny );
+                                    xPropSet->setPropertyValue( "UIName", Any( aCustomTbxTitle ) );
                                 }
                                 catch ( beans::UnknownPropertyException& )
                                 {
diff --git a/sfx2/source/doc/templatedlg.cxx b/sfx2/source/doc/templatedlg.cxx
index f4be8a17ae76..a202a820d797 100644
--- a/sfx2/source/doc/templatedlg.cxx
+++ b/sfx2/source/doc/templatedlg.cxx
@@ -885,10 +885,8 @@ void SfxTemplateManagerDlg::OnTemplateState (const ThumbnailViewItem *pItem)
 
 void SfxTemplateManagerDlg::OnTemplateImportCategory(const OUString& sCategory)
 {
-    sal_Int16 nDialogType =
-        css::ui::dialogs::TemplateDescription::FILEOPEN_SIMPLE;
-
-    sfx2::FileDialogHelper aFileDlg(nDialogType, FileDialogFlags::MultiSelection);
+    sfx2::FileDialogHelper aFileDlg(css::ui::dialogs::TemplateDescription::FILEOPEN_SIMPLE,
+                                    FileDialogFlags::MultiSelection);
 
     // add "All" filter
     aFileDlg.AddFilter( SfxResId(STR_SFX_FILTERNAME_ALL),
@@ -1069,7 +1067,6 @@ void SfxTemplateManagerDlg::OnTemplateExport()
 
 void SfxTemplateManagerDlg::OnTemplateLink ()
 {
-    OUString sNode("TemplateRepositoryURL");
     OUString sNodePath("/org.openoffice.Office.Common/Help/StartCenter");
     try
     {
@@ -1086,7 +1083,7 @@ void SfxTemplateManagerDlg::OnTemplateLink ()
         {
             OUString sURL;
             //throws css::container::NoSuchElementException, css::lang::WrappedTargetException
-            Any value( xNameAccess->getByName(sNode) );
+            Any value( xNameAccess->getByName("TemplateRepositoryURL") );
             sURL = value.get<OUString> ();
             localizeWebserviceURI(sURL);
 
diff --git a/sfx2/source/inet/inettbc.cxx b/sfx2/source/inet/inettbc.cxx
index c44509fe207d..978209c6e941 100644
--- a/sfx2/source/inet/inettbc.cxx
+++ b/sfx2/source/inet/inettbc.cxx
@@ -96,12 +96,10 @@ void SfxURLToolBoxControl_Impl::OpenURL( const OUString& rName ) const
     if ( xDispatchProvider.is() )
     {
         URL             aTargetURL;
-        OUString aTarget( "_default" );
-
         aTargetURL.Complete = aName;
 
         getURLTransformer()->parseStrict( aTargetURL );
-        Reference< XDispatch > xDispatch = xDispatchProvider->queryDispatch( aTargetURL, aTarget, 0 );
+        Reference< XDispatch > xDispatch = xDispatchProvider->queryDispatch( aTargetURL, "_default", 0 );
         if ( xDispatch.is() )
         {
             Sequence< PropertyValue > aArgs( 2 );
diff --git a/sfx2/source/sidebar/Theme.cxx b/sfx2/source/sidebar/Theme.cxx
index 2b29083f5bd9..6b130d984545 100644
--- a/sfx2/source/sidebar/Theme.cxx
+++ b/sfx2/source/sidebar/Theme.cxx
@@ -551,7 +551,8 @@ css::uno::Sequence<css::beans::Property> SAL_CALL Theme::getProperties()
 {
     ::std::vector<beans::Property> aProperties;
 
-    for (sal_Int32 nItem(Begin_),nEnd(End_); nItem!=nEnd; ++nItem)
+    sal_Int32 const nEnd(End_);
+    for (sal_Int32 nItem(Begin_); nItem!=nEnd; ++nItem)
     {
         const ThemeItem eItem (static_cast<ThemeItem>(nItem));
         const PropertyType eType (GetPropertyType(eItem));
diff --git a/sfx2/source/view/frame2.cxx b/sfx2/source/view/frame2.cxx
index fbb44fc5b6e7..e89086bc02f6 100644
--- a/sfx2/source/view/frame2.cxx
+++ b/sfx2/source/view/frame2.cxx
@@ -247,10 +247,9 @@ SfxFrame* SfxFrame::Create( SfxObjectShell& rDoc, vcl::Window& rWindow, SfxInter
         aLoadArgs = aArgs.getPropertyValues();
 
         // load the doc into that frame
-        OUString sLoaderURL( "private:object" );
         Reference< XComponentLoader > xLoader( xFrame, UNO_QUERY_THROW );
         xLoader->loadComponentFromURL(
-            sLoaderURL,
+            "private:object",
             "_self",
             0,
             aLoadArgs
diff --git a/sfx2/source/view/sfxbasecontroller.cxx b/sfx2/source/view/sfxbasecontroller.cxx
index 448adcfb4812..bede7424acaf 100644
--- a/sfx2/source/view/sfxbasecontroller.cxx
+++ b/sfx2/source/view/sfxbasecontroller.cxx
@@ -1144,7 +1144,6 @@ uno::Sequence< frame::DispatchInformation > SAL_CALL SfxBaseController::getConfi
 
         SfxViewFrame* pViewFrame( m_pData->m_pViewShell->GetFrame() );
         SfxSlotPool*  pPool( &SfxSlotPool::GetSlotPool( pViewFrame ));
-        OUString aCmdPrefix( ".uno:" );
 
         SfxSlotPool* pSlotPool = pPool ? pPool : &SFX_SLOTPOOL();
         for ( sal_uInt16 i=0; i<pSlotPool->GetGroupCount(); i++ )
@@ -1161,9 +1160,7 @@ uno::Sequence< frame::DispatchInformation > SAL_CALL SfxBaseController::getConfi
                         if ( pSfxSlot->GetMode() & nMode )
                         {
                             frame::DispatchInformation aCmdInfo;
-                            OUStringBuffer aBuf( aCmdPrefix );
-                            aBuf.appendAscii( pSfxSlot->GetUnoName() );
-                            aCmdInfo.Command = aBuf.makeStringAndClear();
+                            aCmdInfo.Command = ".uno:" + OUString::createFromAscii( pSfxSlot->GetUnoName() );
                             aCmdInfo.GroupId = nCommandGroup;
                             aCmdList.push_back( aCmdInfo );
                         }
diff --git a/sfx2/source/view/viewfrm2.cxx b/sfx2/source/view/viewfrm2.cxx
index 116df435e91b..b6e5b96a1e4d 100644
--- a/sfx2/source/view/viewfrm2.cxx
+++ b/sfx2/source/view/viewfrm2.cxx
@@ -233,8 +233,7 @@ void SfxViewFrame::Exec_Impl(SfxRequest &rReq )
 
                 // Document only needs to be queried, if no other View present.
                 bool bClosed = false;
-                bool bUI = true;
-                if ( ( bOther || pDocSh->PrepareClose( bUI ) ) )
+                if ( ( bOther || pDocSh->PrepareClose( true/*bUI*/ ) ) )
                 {
                     if ( !bOther )
                         pDocSh->SetModified( false );
diff --git a/sfx2/source/view/viewprn.cxx b/sfx2/source/view/viewprn.cxx
index a616d2d1237f..2a4fca34715c 100644
--- a/sfx2/source/view/viewprn.cxx
+++ b/sfx2/source/view/viewprn.cxx
@@ -642,10 +642,9 @@ void SfxViewShell::ExecPrint_Impl( SfxRequest &rReq )
     // no help button in dialogs if called from the help window
     // (pressing help button would exchange the current page inside the help
     // document that is going to be printed!)
-    OUString aHelpFilterName( "writer_web_HTML_help" );
     SfxMedium* pMedium = GetViewFrame()->GetObjectShell()->GetMedium();
     std::shared_ptr<const SfxFilter> pFilter = pMedium ? pMedium->GetFilter() : nullptr;
-    bool bPrintOnHelp = ( pFilter && pFilter->GetFilterName() == aHelpFilterName );
+    bool bPrintOnHelp = ( pFilter && pFilter->GetFilterName() == "writer_web_HTML_help" );
 
     const sal_uInt16 nId = rReq.GetSlot();
     switch( nId )
diff --git a/sfx2/source/view/viewsh.cxx b/sfx2/source/view/viewsh.cxx
index dd81339f9783..2fc7afaf330a 100644
--- a/sfx2/source/view/viewsh.cxx
+++ b/sfx2/source/view/viewsh.cxx
@@ -418,17 +418,14 @@ void SfxViewShell::ExecMisc_Impl( SfxRequest &rReq )
                     aValue >>= xLayoutManager;
                     if ( xLayoutManager.is() )
                     {
-                        OUString aTextResString( "private:resource/toolbar/textobjectbar" );
-                        uno::Reference< ui::XUIElement > xElement = xLayoutManager->getElement( aTextResString );
+                        uno::Reference< ui::XUIElement > xElement = xLayoutManager->getElement( "private:resource/toolbar/textobjectbar" );
                         if(!xElement.is())
                         {
-                            OUString aFrameResString( "private:resource/toolbar/frameobjectbar" );
-                            xElement = xLayoutManager->getElement( aFrameResString );
+                            xElement = xLayoutManager->getElement( "private:resource/toolbar/frameobjectbar" );
                         }
                         if(!xElement.is())
                         {
-                            OUString aOleResString( "private:resource/toolbar/oleobjectbar" );
-                            xElement = xLayoutManager->getElement( aOleResString );
+                            xElement = xLayoutManager->getElement( "private:resource/toolbar/oleobjectbar" );
                         }
                         if(xElement.is())
                         {
@@ -585,7 +582,6 @@ void SfxViewShell::ExecMisc_Impl( SfxRequest &rReq )
                 OUString aFilterName;
                 OUString aTypeName( "generic_HTML" );
                 OUString aFileName;
-                OUString aExtension( "htm" );
 
                 OUString aLocation = xStorable->getLocation();
                 INetURLObject aFileObj( aLocation );
@@ -637,7 +633,7 @@ void SfxViewShell::ExecMisc_Impl( SfxRequest &rReq )
 
                 INetURLObject aFilePathObj( aTempDir.GetURL() );
                 aFilePathObj.insertName( aFileName );
-                aFilePathObj.setExtension( aExtension );
+                aFilePathObj.setExtension( "htm" );
 
                 OUString aFileURL = aFilePathObj.GetMainURL( INetURLObject::DecodeMechanism::NONE );
 


More information about the Libreoffice-commits mailing list