loplugin: defaultparams

Change-Id: Idd291659ca786b7bd2755c8faf98edc51314687b

19 files changed, 47 insertions, 48 deletions

diff --git a/sc/source/ui/navipi/content.cxx b/sc/source/ui/navipi/content.cxx
index 106cb58f1dcd..ab70ebcaf7ed 100644
--- a/sc/source/ui/navipi/content.cxx
+++ b/sc/source/ui/navipi/content.cxx
@@ -761,7 +761,7 @@ void ScContentTree::ObjectFresh( sal_uInt16 nType, SvTreeListEntry* pEntry )
             if( pOldEntry )
             {
                 Expand(pParent);
-                Select( pOldEntry, true);
+                Select( pOldEntry );
             }
         }
         }
diff --git a/sc/source/ui/navipi/navipi.cxx b/sc/source/ui/navipi/navipi.cxx
index 04ec62ba7521..e865dfcc6e8a 100644
--- a/sc/source/ui/navipi/navipi.cxx
+++ b/sc/source/ui/navipi/navipi.cxx
@@ -421,7 +421,7 @@ void CommandToolBox::UpdateButtons()
     }
     else
     {
-        EnableItem( IID_CHANGEROOT, true );
+        EnableItem( IID_CHANGEROOT );
         bool bRootSet = rDlg.aLbEntries->GetRootType() != SC_CONTENT_ROOT;
         CheckItem( IID_CHANGEROOT, bRootSet );
     }
diff --git a/sc/source/ui/navipi/scenwnd.cxx b/sc/source/ui/navipi/scenwnd.cxx
index eedca0337742..a8e53ab997a5 100644
--- a/sc/source/ui/navipi/scenwnd.cxx
+++ b/sc/source/ui/navipi/scenwnd.cxx
@@ -87,7 +87,7 @@ void ScScenarioListBox::UpdateEntries( const std::vector<OUString> &aNewEntryLis
                 aEntry.mbProtected = !(*iter).isEmpty() && (*iter)[0] != '0';
 
                 maEntries.push_back( aEntry );
-                InsertEntry( aEntry.maName, LISTBOX_APPEND );
+                InsertEntry( aEntry.maName );
             }
             SetUpdateMode( true );
             SetNoSelection();
diff --git a/sc/source/ui/optdlg/calcoptionsdlg.cxx b/sc/source/ui/optdlg/calcoptionsdlg.cxx
index dc41f5f96501..912a09897d53 100644
--- a/sc/source/ui/optdlg/calcoptionsdlg.cxx
+++ b/sc/source/ui/optdlg/calcoptionsdlg.cxx
@@ -91,7 +91,7 @@ ScCalcOptionsDialog::ScCalcOptionsDialog(vcl::Window* pParent, const ScCalcConfi
     get(mpFtMemory, "memory");
 
     get(mpConversion,"comboConversion");
-    mpConversion->SelectEntryPos(static_cast<sal_Int32>(rConfig.meStringConversion), true);
+    mpConversion->SelectEntryPos(static_cast<sal_Int32>(rConfig.meStringConversion));
     mpConversion->SetSelectHdl(LINK(this, ScCalcOptionsDialog, ConversionModifiedHdl));
 
     get(mpEmptyAsZero,"checkEmptyAsZero");
@@ -211,7 +211,7 @@ IMPL_LINK(ScCalcOptionsDialog, ConversionModifiedHdl, ListBox*, pConv )
          break;
          case ScCalcConfig::StringConversion::ZERO:
                     maConfig.mbEmptyStringAsZero = true;
-                    mpEmptyAsZero->Check(true);
+                    mpEmptyAsZero->Check();
                     mpEmptyAsZero->Enable(false);
          break;
          case ScCalcConfig::StringConversion::UNAMBIGUOUS:
diff --git a/sc/source/ui/optdlg/tpformula.cxx b/sc/source/ui/optdlg/tpformula.cxx
index 2e4643e5d055..255d71a31754 100644
--- a/sc/source/ui/optdlg/tpformula.cxx
+++ b/sc/source/ui/optdlg/tpformula.cxx
@@ -125,14 +125,14 @@ void ScTpFormulaOptions::UpdateCustomCalcRadioButtons(bool bDefault)
 {
     if (bDefault)
     {
-        mpBtnCustomCalcDefault->Check(true);
+        mpBtnCustomCalcDefault->Check();
         mpBtnCustomCalcCustom->Check(false);
         mpBtnCustomCalcDetails->Disable();
     }
     else
     {
         mpBtnCustomCalcDefault->Check(false);
-        mpBtnCustomCalcCustom->Check(true);
+        mpBtnCustomCalcCustom->Check();
         mpBtnCustomCalcDetails->Enable();
     }
 }
diff --git a/sc/source/ui/pagedlg/tphfedit.cxx b/sc/source/ui/pagedlg/tphfedit.cxx
index a44818341e0c..4d907e0c2230 100644
--- a/sc/source/ui/pagedlg/tphfedit.cxx
+++ b/sc/source/ui/pagedlg/tphfedit.cxx
@@ -234,7 +234,7 @@ void ScEditWindow::Paint( vcl::RenderContext& rRenderContext, const Rectangle& r
     pEdView->Paint(rRect);
 
     if( HasFocus() )
-        pEdView->ShowCursor(true,true);
+        pEdView->ShowCursor(true);
 }
 
 void ScEditWindow::MouseMove( const MouseEvent& rMEvt )
@@ -283,7 +283,7 @@ void ScEditWindow::Command( const CommandEvent& rCEvt )
 
 void ScEditWindow::GetFocus()
 {
-    pEdView->ShowCursor(true,true);
+    pEdView->ShowCursor(true);
     pActiveEdWnd = this;
 
     ::com::sun::star::uno::Reference< ::com::sun::star::accessibility::XAccessible > xTemp = xAcc;
diff --git a/sc/source/ui/sidebar/AlignmentPropertyPanel.cxx b/sc/source/ui/sidebar/AlignmentPropertyPanel.cxx
index 66c5fc9d0a04..270b68f7a634 100644
--- a/sc/source/ui/sidebar/AlignmentPropertyPanel.cxx
+++ b/sc/source/ui/sidebar/AlignmentPropertyPanel.cxx
@@ -222,7 +222,7 @@ IMPL_LINK_NOARG(AlignmentPropertyPanel, CBOXMergnCellClkHdl)
         GetBindings()->GetDispatcher()->Execute(FID_MERGE_ON, SfxCallMode::RECORD);
     else
         GetBindings()->GetDispatcher()->Execute(FID_MERGE_OFF, SfxCallMode::RECORD);
-    GetBindings()->Invalidate(FID_MERGE_TOGGLE,true,false);
+    GetBindings()->Invalidate(FID_MERGE_TOGGLE,true);
     //modified end
 
     return 0;
@@ -349,7 +349,7 @@ void AlignmentPropertyPanel::NotifyItemUpdate(
             }
             else if(eState == SfxItemState::DONTCARE)
             {
-                mpCBXWrapText->EnableTriState(true);
+                mpCBXWrapText->EnableTriState();
                 mpCBXWrapText->SetState(TRISTATE_INDET);
             }
         }
@@ -367,7 +367,7 @@ void AlignmentPropertyPanel::NotifyItemUpdate(
         {
             mbMultiDisable = true;
             mpTextOrientBox->Disable();
-            mpCBStacked->EnableTriState(true);
+            mpCBStacked->EnableTriState();
             mpCBStacked->SetState(TRISTATE_INDET);
         }
         break;
diff --git a/sc/source/ui/sidebar/CellBorderStyleControl.cxx b/sc/source/ui/sidebar/CellBorderStyleControl.cxx
index 548be75168b7..8a80d4086d3f 100644
--- a/sc/source/ui/sidebar/CellBorderStyleControl.cxx
+++ b/sc/source/ui/sidebar/CellBorderStyleControl.cxx
@@ -194,7 +194,7 @@ IMPL_LINK_TYPED(CellBorderStyleControl, TB1SelectHdl, ToolBox*, pToolBox, void)
     aBorderInner.SetValid( SvxBoxInfoItemValidFlags::RIGHT,     0 != (nValidFlags&FRM_VALID_RIGHT ));
     aBorderInner.SetValid( SvxBoxInfoItemValidFlags::HORI,      0 != (nValidFlags&FRM_VALID_HINNER ));
     aBorderInner.SetValid( SvxBoxInfoItemValidFlags::VERT,      0 != (nValidFlags&FRM_VALID_VINNER));
-    aBorderInner.SetValid( SvxBoxInfoItemValidFlags::DISTANCE, true );
+    aBorderInner.SetValid( SvxBoxInfoItemValidFlags::DISTANCE );
     aBorderInner.SetValid( SvxBoxInfoItemValidFlags::DISABLE, false );
 
     mrCellAppearancePropertyPanel.GetBindings()->GetDispatcher()->Execute(SID_ATTR_BORDER, SfxCallMode::RECORD, &aBorderOuter, &aBorderInner, 0L);
@@ -262,7 +262,7 @@ IMPL_LINK_TYPED(CellBorderStyleControl, TB2SelectHdl, ToolBox *, pToolBox, void)
         aBorderInner.SetValid( SvxBoxInfoItemValidFlags::RIGHT,     0 != (nValidFlags&FRM_VALID_RIGHT ));
         aBorderInner.SetValid( SvxBoxInfoItemValidFlags::HORI,      0 != (nValidFlags&FRM_VALID_HINNER ));
         aBorderInner.SetValid( SvxBoxInfoItemValidFlags::VERT,      0 != (nValidFlags&FRM_VALID_VINNER));
-        aBorderInner.SetValid( SvxBoxInfoItemValidFlags::DISTANCE, true );
+        aBorderInner.SetValid( SvxBoxInfoItemValidFlags::DISTANCE );
         aBorderInner.SetValid( SvxBoxInfoItemValidFlags::DISABLE, false );
 
         mrCellAppearancePropertyPanel.GetBindings()->GetDispatcher()->Execute(SID_ATTR_BORDER, SfxCallMode::RECORD, &aBorderOuter, &aBorderInner, 0L);
@@ -333,7 +333,7 @@ IMPL_LINK_TYPED(CellBorderStyleControl, TB3SelectHdl, ToolBox *, pToolBox, void)
     aBorderInner.SetValid( SvxBoxInfoItemValidFlags::RIGHT,     0 != (nValidFlags&FRM_VALID_RIGHT ));
     aBorderInner.SetValid( SvxBoxInfoItemValidFlags::HORI,      0 != (nValidFlags&FRM_VALID_HINNER ));
     aBorderInner.SetValid( SvxBoxInfoItemValidFlags::VERT,      0 != (nValidFlags&FRM_VALID_VINNER));
-    aBorderInner.SetValid( SvxBoxInfoItemValidFlags::DISTANCE, true );
+    aBorderInner.SetValid( SvxBoxInfoItemValidFlags::DISTANCE );
     aBorderInner.SetValid( SvxBoxInfoItemValidFlags::DISABLE, false );
 
     mrCellAppearancePropertyPanel.GetBindings()->GetDispatcher()->Execute(SID_ATTR_BORDER, SfxCallMode::RECORD, &aBorderOuter, &aBorderInner, 0L);
diff --git a/sc/source/ui/sidebar/CellLineStyleValueSet.cxx b/sc/source/ui/sidebar/CellLineStyleValueSet.cxx
index 7f3c6444d1a4..dadc65d63f40 100644
--- a/sc/source/ui/sidebar/CellLineStyleValueSet.cxx
+++ b/sc/source/ui/sidebar/CellLineStyleValueSet.cxx
@@ -28,7 +28,7 @@ CellLineStyleValueSet::CellLineStyleValueSet( vcl::Window* pParent, const ResId&
     pVDev(NULL),
     nSelItem(0)
 {
-    SetColCount( 1 );
+    SetColCount();
     SetLineCount( 9);
 }
 
diff --git a/sc/source/ui/undo/undobase.cxx b/sc/source/ui/undo/undobase.cxx
index a383eed17f2f..cecea867609f 100644
--- a/sc/source/ui/undo/undobase.cxx
+++ b/sc/source/ui/undo/undobase.cxx
@@ -408,7 +408,7 @@ void ScMultiBlockUndo::ShowBlock()
     SCTAB nTab = pViewShell->GetViewData().GetTabNo();
     aRange.aStart.SetTab(nTab);
     aRange.aEnd.SetTab(nTab);
-    pViewShell->MarkRange(aRange, false, false);
+    pViewShell->MarkRange(aRange, false);
 
     for (size_t i = 1, n = maBlockRanges.size(); i < n; ++i)
     {
diff --git a/sc/source/ui/undo/undoblk.cxx b/sc/source/ui/undo/undoblk.cxx
index 379d213e3213..1430a12df683 100644
--- a/sc/source/ui/undo/undoblk.cxx
+++ b/sc/source/ui/undo/undoblk.cxx
@@ -556,7 +556,7 @@ void ScUndoDeleteCells::Redo()
 void ScUndoDeleteCells::Repeat(SfxRepeatTarget& rTarget)
 {
     if (rTarget.ISA(ScTabViewTarget))
-        static_cast<ScTabViewTarget&>(rTarget).GetViewShell()->DeleteCells( eCmd, true );
+        static_cast<ScTabViewTarget&>(rTarget).GetViewShell()->DeleteCells( eCmd );
 }
 
 bool ScUndoDeleteCells::CanRepeat(SfxRepeatTarget& rTarget) const
@@ -735,7 +735,7 @@ void ScUndoDeleteMulti::Repeat(SfxRepeatTarget& rTarget)
 {
     // if single selection
     if (rTarget.ISA(ScTabViewTarget))
-        static_cast<ScTabViewTarget&>(rTarget).GetViewShell()->DeleteCells( DEL_DELROWS, true );
+        static_cast<ScTabViewTarget&>(rTarget).GetViewShell()->DeleteCells( DEL_DELROWS );
 }
 
 bool ScUndoDeleteMulti::CanRepeat(SfxRepeatTarget& rTarget) const
@@ -1766,7 +1766,7 @@ void ScUndoSelectionStyle::Repeat(SfxRepeatTarget& rTarget)
         }
 
         ScTabViewShell& rViewShell = *static_cast<ScTabViewTarget&>(rTarget).GetViewShell();
-        rViewShell.SetStyleSheetToMarked( pStyleSheet, true );
+        rViewShell.SetStyleSheetToMarked( pStyleSheet );
     }
 }
 
diff --git a/sc/source/ui/undo/undoblk3.cxx b/sc/source/ui/undo/undoblk3.cxx
index 764d7170f624..3d84110e6ed1 100644
--- a/sc/source/ui/undo/undoblk3.cxx
+++ b/sc/source/ui/undo/undoblk3.cxx
@@ -211,7 +211,7 @@ void ScUndoDeleteContents::Redo()
 void ScUndoDeleteContents::Repeat(SfxRepeatTarget& rTarget)
 {
     if (rTarget.ISA(ScTabViewTarget))
-        static_cast<ScTabViewTarget&>(rTarget).GetViewShell()->DeleteContents( nFlags, true );
+        static_cast<ScTabViewTarget&>(rTarget).GetViewShell()->DeleteContents( nFlags );
 }
 
 bool ScUndoDeleteContents::CanRepeat(SfxRepeatTarget& rTarget) const
@@ -481,7 +481,7 @@ void ScUndoSelectionAttr::Repeat(SfxRepeatTarget& rTarget)
     {
         ScTabViewShell& rViewShell = *static_cast<ScTabViewTarget&>(rTarget).GetViewShell();
         if (pLineOuter)
-            rViewShell.ApplyPatternLines( *pApplyPattern, pLineOuter, pLineInner, true );
+            rViewShell.ApplyPatternLines( *pApplyPattern, pLineOuter, pLineInner );
         else
             rViewShell.ApplySelectionPattern( *pApplyPattern, true );
     }
@@ -628,10 +628,10 @@ void ScUndoAutoFill::Repeat(SfxRepeatTarget& rTarget)
     {
         ScTabViewShell& rViewShell = *static_cast<ScTabViewTarget&>(rTarget).GetViewShell();
         if (eFillCmd==FILL_SIMPLE)
-            rViewShell.FillSimple( eFillDir, true );
+            rViewShell.FillSimple( eFillDir );
         else
             rViewShell.FillSeries( eFillDir, eFillCmd, eFillDateCmd,
-                                   fStartValue, fStepValue, fMaxValue, true );
+                                   fStartValue, fStepValue, fMaxValue );
     }
 }
 
@@ -906,7 +906,7 @@ void ScUndoAutoFormat::Redo()
 void ScUndoAutoFormat::Repeat(SfxRepeatTarget& rTarget)
 {
     if (rTarget.ISA(ScTabViewTarget))
-        static_cast<ScTabViewTarget&>(rTarget).GetViewShell()->AutoFormat( nFormatNo, true );
+        static_cast<ScTabViewTarget&>(rTarget).GetViewShell()->AutoFormat( nFormatNo );
 }
 
 bool ScUndoAutoFormat::CanRepeat(SfxRepeatTarget& rTarget) const
@@ -1267,7 +1267,7 @@ void ScUndoConversion::Redo()
 void ScUndoConversion::Repeat( SfxRepeatTarget& rTarget )
 {
     if( rTarget.ISA( ScTabViewTarget ) )
-        static_cast<ScTabViewTarget&>(rTarget).GetViewShell()->DoSheetConversion( maConvParam, true );
+        static_cast<ScTabViewTarget&>(rTarget).GetViewShell()->DoSheetConversion( maConvParam );
 }
 
 bool ScUndoConversion::CanRepeat(SfxRepeatTarget& rTarget) const
diff --git a/sc/source/ui/undo/undocell.cxx b/sc/source/ui/undo/undocell.cxx
index c239686cc030..c69c30816d0c 100644
--- a/sc/source/ui/undo/undocell.cxx
+++ b/sc/source/ui/undo/undocell.cxx
@@ -718,7 +718,7 @@ void ScUndoThesaurus::Redo()
 void ScUndoThesaurus::Repeat(SfxRepeatTarget& rTarget)
 {
     if (rTarget.ISA(ScTabViewTarget))
-        static_cast<ScTabViewTarget&>(rTarget).GetViewShell()->DoThesaurus( true );
+        static_cast<ScTabViewTarget&>(rTarget).GetViewShell()->DoThesaurus();
 }
 
 bool ScUndoThesaurus::CanRepeat(SfxRepeatTarget& rTarget) const
diff --git a/sc/source/ui/undo/undodat.cxx b/sc/source/ui/undo/undodat.cxx
index 9fd47025438c..f636c596f9ba 100644
--- a/sc/source/ui/undo/undodat.cxx
+++ b/sc/source/ui/undo/undodat.cxx
@@ -226,9 +226,9 @@ void ScUndoMakeOutline::Repeat(SfxRepeatTarget& rTarget)
         ScTabViewShell& rViewShell = *static_cast<ScTabViewTarget&>(rTarget).GetViewShell();
 
         if (bMake)
-            rViewShell.MakeOutline( bColumns, true );
+            rViewShell.MakeOutline( bColumns );
         else
-            rViewShell.RemoveOutline( bColumns, true );
+            rViewShell.RemoveOutline( bColumns );
     }
 }
 
@@ -415,9 +415,9 @@ void ScUndoOutlineBlock::Repeat(SfxRepeatTarget& rTarget)
         ScTabViewShell& rViewShell = *static_cast<ScTabViewTarget&>(rTarget).GetViewShell();
 
         if (bShow)
-            rViewShell.ShowMarkedOutlines( true );
+            rViewShell.ShowMarkedOutlines();
         else
-            rViewShell.HideMarkedOutlines( true );
+            rViewShell.HideMarkedOutlines();
     }
 }
 
@@ -503,7 +503,7 @@ void ScUndoRemoveAllOutlines::Redo()
 void ScUndoRemoveAllOutlines::Repeat(SfxRepeatTarget& rTarget)
 {
     if (rTarget.ISA(ScTabViewTarget))
-        static_cast<ScTabViewTarget&>(rTarget).GetViewShell()->RemoveAllOutlines( true );
+        static_cast<ScTabViewTarget&>(rTarget).GetViewShell()->RemoveAllOutlines();
 }
 
 bool ScUndoRemoveAllOutlines::CanRepeat(SfxRepeatTarget& rTarget) const
@@ -606,7 +606,7 @@ void ScUndoAutoOutline::Redo()
 void ScUndoAutoOutline::Repeat(SfxRepeatTarget& rTarget)
 {
     if (rTarget.ISA(ScTabViewTarget))
-        static_cast<ScTabViewTarget&>(rTarget).GetViewShell()->AutoOutline( true );
+        static_cast<ScTabViewTarget&>(rTarget).GetViewShell()->AutoOutline();
 }
 
 bool ScUndoAutoOutline::CanRepeat(SfxRepeatTarget& rTarget) const
@@ -1417,7 +1417,7 @@ void ScUndoRepeatDB::Redo()
 void ScUndoRepeatDB::Repeat(SfxRepeatTarget& rTarget)
 {
     if (rTarget.ISA(ScTabViewTarget))
-        static_cast<ScTabViewTarget&>(rTarget).GetViewShell()->RepeatDB( true );
+        static_cast<ScTabViewTarget&>(rTarget).GetViewShell()->RepeatDB();
 }
 
 bool ScUndoRepeatDB::CanRepeat(SfxRepeatTarget& rTarget) const
diff --git a/sc/source/ui/undo/undotab.cxx b/sc/source/ui/undo/undotab.cxx
index d7950c8de266..e6acdbd6f9d1 100644
--- a/sc/source/ui/undo/undotab.cxx
+++ b/sc/source/ui/undo/undotab.cxx
@@ -415,7 +415,7 @@ void ScUndoDeleteTab::Repeat(SfxRepeatTarget& rTarget)
     if (rTarget.ISA(ScTabViewTarget))
     {
         ScTabViewShell* pViewShell = static_cast<ScTabViewTarget&>(rTarget).GetViewShell();
-        pViewShell->DeleteTable( pViewShell->GetViewData().GetTabNo(), true );
+        pViewShell->DeleteTable( pViewShell->GetViewData().GetTabNo() );
     }
 }
 
diff --git a/sc/source/ui/unoobj/cellsuno.cxx b/sc/source/ui/unoobj/cellsuno.cxx
index f2f8193fd663..7895ef94179a 100644
--- a/sc/source/ui/unoobj/cellsuno.cxx
+++ b/sc/source/ui/unoobj/cellsuno.cxx
@@ -1517,7 +1517,7 @@ const ScPatternAttr* ScCellRangesBase::GetCurrentAttrsDeep()
     if ( !pCurrentDeep && pDocShell )
     {
         ScDocument& rDoc = pDocShell->GetDocument();
-        pCurrentDeep = rDoc.CreateSelectionPattern( *GetMarkData(), true );
+        pCurrentDeep = rDoc.CreateSelectionPattern( *GetMarkData() );
     }
     return pCurrentDeep;
 }
@@ -3605,7 +3605,7 @@ uno::Reference<sheet::XSheetCellRanges> SAL_CALL ScCellRangesBase::queryContentC
                 }
 
                 if (bAdd)
-                    aMarkData.SetMultiMarkArea(aIter.GetPos(), true);
+                    aMarkData.SetMultiMarkArea(aIter.GetPos());
             }
 
         }
@@ -3660,7 +3660,7 @@ uno::Reference<sheet::XSheetCellRanges> SAL_CALL ScCellRangesBase::queryFormulaC
                     }
 
                     if (bAdd)
-                        aMarkData.SetMultiMarkArea(aIter.GetPos(), true);
+                        aMarkData.SetMultiMarkArea(aIter.GetPos());
                 }
             }
         }
@@ -3832,7 +3832,7 @@ uno::Reference<sheet::XSheetCellRanges> SAL_CALL ScCellRangesBase::queryPreceden
                     {
                         if ( bRecursive && !bFound && !aMarkData.IsAllMarked( aRefRange ) )
                             bFound = true;
-                        aMarkData.SetMultiMarkArea(aRefRange, true);
+                        aMarkData.SetMultiMarkArea(aRefRange);
                     }
                 }
             }
@@ -3892,7 +3892,7 @@ uno::Reference<sheet::XSheetCellRanges> SAL_CALL ScCellRangesBase::queryDependen
                     ScRange aCellRange(aCellIter.GetPos());
                     if ( bRecursive && !bFound && !aMarkData.IsAllMarked( aCellRange ) )
                         bFound = true;
-                    aMarkData.SetMultiMarkArea(aCellRange, true);
+                    aMarkData.SetMultiMarkArea(aCellRange);
                 }
             }
 
diff --git a/sc/source/ui/unoobj/chart2uno.cxx b/sc/source/ui/unoobj/chart2uno.cxx
index bc67a5b5c977..e0793a22ed21 100644
--- a/sc/source/ui/unoobj/chart2uno.cxx
+++ b/sc/source/ui/unoobj/chart2uno.cxx
@@ -2269,7 +2269,6 @@ OUString SAL_CALL ScChart2DataProvider::convertRangeFromXML( const OUString& sXM
     throw ( uno::RuntimeException, lang::IllegalArgumentException, std::exception )
 {
     const sal_Unicode cSep = ' ';
-    const sal_Unicode cQuote = '\'';
 
     if (!m_pDocument)
     {
@@ -2281,14 +2280,14 @@ OUString SAL_CALL ScChart2DataProvider::convertRangeFromXML( const OUString& sXM
         while( nOffset >= 0 )
         {
             OUString sToken;
-            ScRangeStringConverter::GetTokenByOffset( sToken, sXMLRange, nOffset, cSep, cQuote );
+            ScRangeStringConverter::GetTokenByOffset( sToken, sXMLRange, nOffset, cSep );
             if( nOffset >= 0 )
             {
                 // convert one address (remove dots)
 
                 OUString aUIString(sToken);
 
-                sal_Int32 nIndex = ScRangeStringConverter::IndexOf( sToken, ':', 0, cQuote );
+                sal_Int32 nIndex = ScRangeStringConverter::IndexOf( sToken, ':', 0 );
                 if ( nIndex >= 0 && nIndex < aUIString.getLength() - 1 &&
                         aUIString[nIndex + 1] == '.' )
                     aUIString = aUIString.replaceAt( nIndex + 1, 1, "" );
diff --git a/sc/source/ui/unoobj/docuno.cxx b/sc/source/ui/unoobj/docuno.cxx
index 059626c79bd2..693301db2acd 100644
--- a/sc/source/ui/unoobj/docuno.cxx
+++ b/sc/source/ui/unoobj/docuno.cxx
@@ -551,8 +551,8 @@ Size ScModelObj::getDocumentSize()
         return aSize;
 
     // convert to twips
-    aSize.setWidth(rDoc.GetColWidth(0, nEndCol, nTab, true));
-    aSize.setHeight(rDoc.GetRowHeight(0, nEndRow, nTab, true));
+    aSize.setWidth(rDoc.GetColWidth(0, nEndCol, nTab));
+    aSize.setHeight(rDoc.GetRowHeight(0, nEndRow, nTab));
 
     return aSize;
 }
@@ -1544,7 +1544,7 @@ void SAL_CALL ScModelObj::render( sal_Int32 nSelRenderer, const uno::Any& aSelec
     {
         pDrawView = new FmFormView( pModel, pDev );
         pDrawView->ShowSdrPage(pDrawView->GetModel()->GetPage(nTab));
-        pDrawView->SetPrintPreview( true );
+        pDrawView->SetPrintPreview();
     }
 
     ScRange aRange;
@@ -1940,7 +1940,7 @@ void SAL_CALL ScModelObj::consolidate(
     if (pDocShell)
     {
         const ScConsolidateParam& rParam = xImpl->GetParam();
-        pDocShell->DoConsolidate( rParam, true );
+        pDocShell->DoConsolidate( rParam );
         pDocShell->GetDocument().SetConsolidateDlgData( &rParam );
     }
 }
diff --git a/sc/source/ui/unoobj/styleuno.cxx b/sc/source/ui/unoobj/styleuno.cxx
index 7bbce7a4b1cb..c0d33728dbe6 100644
--- a/sc/source/ui/unoobj/styleuno.cxx
+++ b/sc/source/ui/unoobj/styleuno.cxx
@@ -1483,7 +1483,7 @@ void SAL_CALL ScStyleObj::setAllPropertiesToDefault()
             SvxBoxInfoItem aBoxInfoItem( ATTR_BORDER_INNER );
             aBoxInfoItem.SetTable( false );
             aBoxInfoItem.SetDist( true );
-            aBoxInfoItem.SetValid( SvxBoxInfoItemValidFlags::DISTANCE, true );
+            aBoxInfoItem.SetValid( SvxBoxInfoItemValidFlags::DISTANCE );
             rSet.Put( aBoxInfoItem );
 
             pDocShell->PageStyleModified( aStyleName, true );