diff options
author | Noel Grandin <noel@peralex.com> | 2015-08-12 15:11:19 +0200 |
---|---|---|
committer | Noel Grandin <noel@peralex.com> | 2015-08-14 11:22:22 +0200 |
commit | fcd4c6dabecb348096ed8eb7ffe16c7ec059df31 (patch) | |
tree | add6f8305193f3d357c54ac127340a3265d2ed6e /sc/source | |
parent | 4a2d17c3a15117e91b8d158aa24d7f084596554b (diff) |
loplugin: defaultparams
Change-Id: If342e813063eb0a17f7a8f8e872290f056964e05
Diffstat (limited to 'sc/source')
26 files changed, 85 insertions, 87 deletions
diff --git a/sc/source/ui/vba/vbaformat.cxx b/sc/source/ui/vba/vbaformat.cxx index 0278b002595e..33a0584a10b9 100644 --- a/sc/source/ui/vba/vbaformat.cxx +++ b/sc/source/ui/vba/vbaformat.cxx @@ -554,7 +554,7 @@ ScVbaFormat<Ifc1>::getLocked( ) throw (script::BasicErrorException, uno::Runtim SfxItemSet* pDataSet = getCurrentDataSet(); if ( pDataSet ) { - const ScProtectionAttr& rProtAttr = static_cast<const ScProtectionAttr &>( pDataSet->Get(ATTR_PROTECTION, true) ); + const ScProtectionAttr& rProtAttr = static_cast<const ScProtectionAttr &>( pDataSet->Get(ATTR_PROTECTION) ); SfxItemState eState = pDataSet->GetItemState(ATTR_PROTECTION, true, NULL); if(eState != SfxItemState::DONTCARE) aCellProtection = uno::makeAny(rProtAttr.GetProtection()); @@ -587,7 +587,7 @@ ScVbaFormat<Ifc1>::getFormulaHidden( ) throw (script::BasicErrorException, uno: SfxItemSet* pDataSet = getCurrentDataSet(); if ( pDataSet ) { - const ScProtectionAttr& rProtAttr = static_cast<const ScProtectionAttr &>( pDataSet->Get(ATTR_PROTECTION, true) ); + const ScProtectionAttr& rProtAttr = static_cast<const ScProtectionAttr &>( pDataSet->Get(ATTR_PROTECTION) ); SfxItemState eState = pDataSet->GetItemState(ATTR_PROTECTION, true, NULL); if(eState != SfxItemState::DONTCARE) aBoolRet = uno::makeAny(rProtAttr.GetHideFormula()); diff --git a/sc/source/ui/vba/vbaname.cxx b/sc/source/ui/vba/vbaname.cxx index 6230344de292..4b0531a2e70a 100644 --- a/sc/source/ui/vba/vbaname.cxx +++ b/sc/source/ui/vba/vbaname.cxx @@ -139,7 +139,7 @@ void ScVbaName::setContent( const OUString& rContent, const formula::FormulaGra OUString ScVbaName::getValue() throw (css::uno::RuntimeException, std::exception) { - rtl::OUString sResult = getContent( formula::FormulaGrammar::GRAM_NATIVE_XL_A1, true ); + rtl::OUString sResult = getContent( formula::FormulaGrammar::GRAM_NATIVE_XL_A1 ); return sResult; } @@ -147,7 +147,7 @@ ScVbaName::getValue() throw (css::uno::RuntimeException, std::exception) void ScVbaName::setValue( const OUString & rValue ) throw (css::uno::RuntimeException, std::exception) { - setContent( rValue, formula::FormulaGrammar::GRAM_NATIVE_XL_A1, true ); + setContent( rValue, formula::FormulaGrammar::GRAM_NATIVE_XL_A1 ); } OUString @@ -177,14 +177,14 @@ ScVbaName::setRefersToLocal( const OUString & rRefersTo ) throw (css::uno::Runti OUString ScVbaName::getRefersToR1C1() throw (css::uno::RuntimeException, std::exception) { - rtl::OUString sResult = getContent( formula::FormulaGrammar::GRAM_NATIVE_XL_R1C1, true ); + rtl::OUString sResult = getContent( formula::FormulaGrammar::GRAM_NATIVE_XL_R1C1 ); return sResult; } void ScVbaName::setRefersToR1C1( const OUString & rRefersTo ) throw (css::uno::RuntimeException, std::exception) { - setContent( rRefersTo, formula::FormulaGrammar::GRAM_NATIVE_XL_R1C1, true ); + setContent( rRefersTo, formula::FormulaGrammar::GRAM_NATIVE_XL_R1C1 ); } OUString diff --git a/sc/source/ui/vba/vbarange.cxx b/sc/source/ui/vba/vbarange.cxx index a65dbc8d9648..e4eaf2f38bc3 100644 --- a/sc/source/ui/vba/vbarange.cxx +++ b/sc/source/ui/vba/vbarange.cxx @@ -2281,7 +2281,7 @@ void setCursor( const SCCOL& nCol, const SCROW& nRow, const uno::Reference< fra if ( bInSel ) pShell->SetCursor( nCol, nRow ); else - pShell->MoveCursorAbs( nCol, nRow, SC_FOLLOW_NONE, false, false, true, false ); + pShell->MoveCursorAbs( nCol, nRow, SC_FOLLOW_NONE, false, false, true ); } } diff --git a/sc/source/ui/view/cellsh1.cxx b/sc/source/ui/view/cellsh1.cxx index 54b28e1d48ff..64aa830826e3 100644 --- a/sc/source/ui/view/cellsh1.cxx +++ b/sc/source/ui/view/cellsh1.cxx @@ -720,7 +720,7 @@ void ScCellShell::ExecuteEdit( SfxRequest& rReq ) if ( nStartCol != nEndCol && nStartRow != nEndRow ) { - pDlg->SetEdStartValEnabled(false); + pDlg->SetEdStartValEnabled(); } if ( pDlg->Execute() == RET_OK ) @@ -1434,13 +1434,13 @@ void ScCellShell::ExecuteEdit( SfxRequest& rReq ) //check if horizontal shift will fit if ( !pData->GetDocument()->IsBlockEmpty( nStartTab, MAXCOL - nRangeSizeX, nStartY, - MAXCOL, nStartY + nRangeSizeY, false ) ) + MAXCOL, nStartY + nRangeSizeY ) ) nDisableShiftX = SC_CELL_SHIFT_DISABLE_RIGHT; //check if vertical shift will fit if ( !pData->GetDocument()->IsBlockEmpty( nStartTab, nStartX, MAXROW - nRangeSizeY, - nStartX + nRangeSizeX, MAXROW, false ) ) + nStartX + nRangeSizeX, MAXROW ) ) nDisableShiftY = SC_CELL_SHIFT_DISABLE_DOWN; if ( nDisableShiftX || nDisableShiftY ) diff --git a/sc/source/ui/view/cellsh3.cxx b/sc/source/ui/view/cellsh3.cxx index 0f961210759a..0df3ad943b1f 100644 --- a/sc/source/ui/view/cellsh3.cxx +++ b/sc/source/ui/view/cellsh3.cxx @@ -785,7 +785,7 @@ void ScCellShell::Execute( SfxRequest& rReq ) const ScMarkData& rMark = GetViewData()->GetMarkData(); if ( !rMark.IsMarked() && !rMark.IsMultiMarked() ) - pTabViewShell->MarkDataArea( true ); + pTabViewShell->MarkDataArea(); GetViewData()->GetSimpleArea( nStartCol,nStartRow,nStartTab, nEndCol,nEndRow,nEndTab ); diff --git a/sc/source/ui/view/drawview.cxx b/sc/source/ui/view/drawview.cxx index 8a622c2074f4..bc575d47a67a 100644 --- a/sc/source/ui/view/drawview.cxx +++ b/sc/source/ui/view/drawview.cxx @@ -66,7 +66,7 @@ void ScDrawView::Construct() EnableExtendedMouseEventDispatcher(false); EnableExtendedCommandEventDispatcher(false); - SetFrameDragSingles(true); + SetFrameDragSingles(); SetMinMoveDistancePixel( 2 ); SetHitTolerancePixel( 2 ); @@ -87,7 +87,7 @@ void ScDrawView::Construct() SetLayerLocked( pLayer->GetName(), bProt || !bEx ); pLayer = rAdmin.GetLayerPerID(SC_LAYER_INTERN); if (pLayer) - SetLayerLocked( pLayer->GetName(), true ); + SetLayerLocked( pLayer->GetName() ); pLayer = rAdmin.GetLayerPerID(SC_LAYER_FRONT); if (pLayer) { @@ -104,7 +104,7 @@ void ScDrawView::Construct() SetLayerVisible( pLayer->GetName(), false); } - SetSwapAsynchron(true); + SetSwapAsynchron(); } else { diff --git a/sc/source/ui/view/formatsh.cxx b/sc/source/ui/view/formatsh.cxx index a4f6a0a0c943..315afb3c7637 100644 --- a/sc/source/ui/view/formatsh.cxx +++ b/sc/source/ui/view/formatsh.cxx @@ -371,7 +371,7 @@ void ScFormatShell::ExecuteStyle( SfxRequest& rReq ) OUString aStyleName; sal_uInt16 nRetMask = 0xffff; - pStylePool->SetSearchMask( eFamily, SFXSTYLEBIT_ALL ); + pStylePool->SetSearchMask( eFamily ); switch ( nSlotId ) { @@ -1174,7 +1174,7 @@ void ScFormatShell::ExecuteNumFormat( SfxRequest& rReq ) SfxItemPool* pDocPool = GetViewData()->GetDocument()->GetPool(); SfxItemSet aNewSet( *pDocPool, ATTR_PATTERN_START, ATTR_PATTERN_END ); aNewSet.Put( *pItem ); - pTabViewShell->ApplyAttributes( &aNewSet, &rOldSet, true ); + pTabViewShell->ApplyAttributes( &aNewSet, &rOldSet ); } } break; diff --git a/sc/source/ui/view/gridwin.cxx b/sc/source/ui/view/gridwin.cxx index 1a3afa71ebd4..00ad26ed51f0 100644 --- a/sc/source/ui/view/gridwin.cxx +++ b/sc/source/ui/view/gridwin.cxx @@ -4324,7 +4324,7 @@ sal_Int8 ScGridWindow::DropTransferObj( ScTransferObj* pTransObj, SCCOL nDestPos if ( bDone ) { - pView->MarkRange( aDest, false, false ); + pView->MarkRange( aDest, false ); SCCOL nDCol = pViewData->GetCurX() - aSource.aStart.Col() + nCorrectCursorPosCol; SCROW nDRow = pViewData->GetCurY() - aSource.aStart.Row() + nCorrectCursorPosRow; @@ -4432,7 +4432,7 @@ sal_Int8 ScGridWindow::DropTransferObj( ScTransferObj* pTransObj, SCCOL nDestPos pView->EnterMatrix( aFormula.makeStringAndClear(), ::formula::FormulaGrammar::GRAM_NATIVE ); - pView->MarkRange( aDest, false, false ); + pView->MarkRange( aDest, false ); pView->SetCursor( aDest.aStart.Col(), aDest.aStart.Row() ); } @@ -4466,7 +4466,7 @@ sal_Int8 ScGridWindow::DropTransferObj( ScTransferObj* pTransObj, SCCOL nDestPos bDone = pView->PasteFromClip( IDF_ALL, pTransObj->GetDocument() ); // clip-doc if ( bDone ) { - pView->MarkRange( aDest, false, false ); + pView->MarkRange( aDest, false ); pView->SetCursor( aDest.aStart.Col(), aDest.aStart.Row() ); } } diff --git a/sc/source/ui/view/gridwin4.cxx b/sc/source/ui/view/gridwin4.cxx index 4839bb92d554..3cb269d1d4a8 100644 --- a/sc/source/ui/view/gridwin4.cxx +++ b/sc/source/ui/view/gridwin4.cxx @@ -745,7 +745,7 @@ void ScGridWindow::DrawContent(OutputDevice &rDevice, const ScTableInfo& rTableI aOutputData.DrawNoteMarks(*pContentDev); if ( !bLogicText ) - aOutputData.DrawStrings(false); // in pixel MapMode + aOutputData.DrawStrings(); // in pixel MapMode // edit cells and printer-metrics text must be before the buttons // (DataPilot buttons contain labels in UI font) @@ -1450,7 +1450,7 @@ void ScGridWindow::DrawButtons(SCCOL nX1, SCCOL nX2, const ScTableInfo& rTabInfo aComboButton.SetPosPixel( aRect.TopLeft() ); aComboButton.SetSizePixel( aRect.GetSize() ); pContentDev->SetClipRegion(vcl::Region(aRect)); - aComboButton.Draw( false, false ); + aComboButton.Draw( false ); pContentDev->SetClipRegion(); // always called from Draw() without clip region aComboButton.SetPosPixel( aOldPos ); // restore old state aComboButton.SetSizePixel( aOldSize ); // for MouseUp/Down (AutoFilter) diff --git a/sc/source/ui/view/gridwin_dbgutil.cxx b/sc/source/ui/view/gridwin_dbgutil.cxx index 494edfcedbe2..a6d884d2aeb3 100644 --- a/sc/source/ui/view/gridwin_dbgutil.cxx +++ b/sc/source/ui/view/gridwin_dbgutil.cxx @@ -41,7 +41,7 @@ void ScGridWindow::dumpColumnInformationPixel() SCTAB nTab = pViewData->GetTabNo(); for (SCCOL nCol = 0; nCol <= 20; ++nCol) { - sal_uInt16 nWidth = pDoc->GetColWidth(nCol, nTab, true); + sal_uInt16 nWidth = pDoc->GetColWidth(nCol, nTab); long nPixel = LogicToPixel(Point(nWidth, 0), MapMode(MAP_TWIP)).getX(); std::cout << "Column: " << nCol << ", Width: " << nPixel << "px" << std::endl; } @@ -53,7 +53,7 @@ void ScGridWindow::dumpColumnInformationHmm() SCTAB nTab = pViewData->GetTabNo(); for (SCCOL nCol = 0; nCol <= 20; ++nCol) { - sal_uInt16 nWidth = pDoc->GetColWidth(nCol, nTab, true); + sal_uInt16 nWidth = pDoc->GetColWidth(nCol, nTab); long nPixel = LogicToLogic(Point(nWidth, 0), MAP_TWIP, MAP_100TH_MM).getX(); std::cout << "Column: " << nCol << ", Width: " << nPixel << "hmm" << std::endl; } diff --git a/sc/source/ui/view/output.cxx b/sc/source/ui/view/output.cxx index a84aa3b5ec57..f4378baa6898 100644 --- a/sc/source/ui/view/output.cxx +++ b/sc/source/ui/view/output.cxx @@ -1985,7 +1985,7 @@ void ScOutputData::FindChanged() ScFormulaCell* pFCell = rCell.mpFormula; if ( !bProgress && pFCell->GetDirty() ) { - ScProgress::CreateInterpretProgress(mpDoc, true); + ScProgress::CreateInterpretProgress(mpDoc); bProgress = true; } if (pFCell->IsRunning()) diff --git a/sc/source/ui/view/output2.cxx b/sc/source/ui/view/output2.cxx index cd63e9dc3a85..97fd6da122f7 100644 --- a/sc/source/ui/view/output2.cxx +++ b/sc/source/ui/view/output2.cxx @@ -1048,7 +1048,7 @@ static inline void lcl_CreateInterpretProgress( bool& bProgress, ScDocument* pDo { if ( !bProgress && pFCell->GetDirty() ) { - ScProgress::CreateInterpretProgress( pDoc, true ); + ScProgress::CreateInterpretProgress( pDoc ); bProgress = true; } } @@ -3138,7 +3138,7 @@ void ScOutputData::DrawEditStandard(DrawEditParam& rParam) } else { - rParam.mpEngine->Draw(mpDev, aLogicStart, 0); + rParam.mpEngine->Draw(mpDev, aLogicStart); } if (bClip) @@ -4110,7 +4110,7 @@ void ScOutputData::DrawEditStacked(DrawEditParam& rParam) } else { - rParam.mpEngine->Draw( mpDev, aLogicStart, 0 ); + rParam.mpEngine->Draw( mpDev, aLogicStart ); } if (bClip) @@ -4435,7 +4435,7 @@ void ScOutputData::DrawEditAsianVertical(DrawEditParam& rParam) // the whole output area, not the text itself aLogicStart.X() -= rParam.mpEngine->GetPaperSize().Width(); - rParam.mpEngine->Draw(mpDev, aLogicStart, 0); + rParam.mpEngine->Draw(mpDev, aLogicStart); if (bClip) { diff --git a/sc/source/ui/view/preview.cxx b/sc/source/ui/view/preview.cxx index 43360991e46a..470634229483 100644 --- a/sc/source/ui/view/preview.cxx +++ b/sc/source/ui/view/preview.cxx @@ -175,8 +175,8 @@ void ScPreview::UpdateDrawView() // nTab must be right // The DrawView takes over the Design-Mode from the Model // (Settings "In opening Draftmode"), therefore to restore here - pDrawView->SetDesignMode( true ); - pDrawView->SetPrintPreview( true ); + pDrawView->SetDesignMode(); + pDrawView->SetPrintPreview(); pDrawView->ShowSdrPage(pPage); } } @@ -513,7 +513,7 @@ void ScPreview::DoPrint( ScPreviewLocationData* pFillLocation ) pEditEng->SetRefMapMode(aMMMode); SfxItemSet* pEditDefaults = new SfxItemSet( pEditEng->GetEmptyItemSet() ); rDefPattern.FillEditItemSet(pEditDefaults); - pEditEng->SetDefaults(pEditDefaults, true); + pEditEng->SetDefaults(pEditDefaults); Color aTextColor(COL_LIGHTGRAY); pEditDefaults->Put(SvxColorItem(aTextColor, EE_CHAR_COLOR)); @@ -1105,13 +1105,13 @@ void ScPreview::MouseButtonUp( const MouseEvent& rMEvt ) { aLRItem.SetLeft( (long)( aButtonUpPt.X() / HMM_PER_TWIPS + aOffset.X() / HMM_PER_TWIPS )); rStyleSet.Put( aLRItem ); - pDocShell->SetModified(true); + pDocShell->SetModified(); } else if( bRightRulerChange && bRightRulerMove ) { aLRItem.SetRight( (long)( nWidth - aButtonUpPt.X() / HMM_PER_TWIPS - aOffset.X() / HMM_PER_TWIPS )); rStyleSet.Put( aLRItem ); - pDocShell->SetModified(true); + pDocShell->SetModified(); } ScStyleSaveData aNewData; @@ -1177,13 +1177,13 @@ void ScPreview::MouseButtonUp( const MouseEvent& rMEvt ) { aULItem.SetUpperValue( (sal_uInt16)( aButtonUpPt.Y() / HMM_PER_TWIPS + aOffset.Y() / HMM_PER_TWIPS ) ); rStyleSet.Put( aULItem ); - pDocShell->SetModified(true); + pDocShell->SetModified(); } else if( bBottomRulerMove && bBottomRulerChange ) { aULItem.SetLowerValue( (sal_uInt16)( nHeight - aButtonUpPt.Y() / HMM_PER_TWIPS - aOffset.Y() / HMM_PER_TWIPS ) ); rStyleSet.Put( aULItem ); - pDocShell->SetModified(true); + pDocShell->SetModified(); } else if( bHeaderRulerMove && bHeaderRulerChange ) { @@ -1197,7 +1197,7 @@ void ScPreview::MouseButtonUp( const MouseEvent& rMEvt ) SvxSetItem aNewHeader( static_cast<const SvxSetItem&>(rStyleSet.Get(ATTR_PAGE_HEADERSET)) ); aNewHeader.GetItemSet().Put( SvxSizeItem( ATTR_PAGE_SIZE, aHeaderSize ) ); rStyleSet.Put( aNewHeader ); - pDocShell->SetModified(true); + pDocShell->SetModified(); } } else if( bFooterRulerMove && bFooterRulerChange ) @@ -1212,7 +1212,7 @@ void ScPreview::MouseButtonUp( const MouseEvent& rMEvt ) SvxSetItem aNewFooter( static_cast<const SvxSetItem&>(rStyleSet.Get(ATTR_PAGE_FOOTERSET)) ); aNewFooter.GetItemSet().Put( SvxSizeItem( ATTR_PAGE_SIZE, aFooterSize ) ); rStyleSet.Put( aNewFooter ); - pDocShell->SetModified(true); + pDocShell->SetModified(); } } @@ -1281,7 +1281,7 @@ void ScPreview::MouseButtonUp( const MouseEvent& rMEvt ) { pDocShell->GetDocFunc().SetWidthOrHeight( true, aCols, nTab, SC_SIZE_DIRECT, (sal_uInt16)nNewColWidth, true, true); - pDocShell->SetModified(true); + pDocShell->SetModified(); } if ( ValidTab( nTab ) ) { diff --git a/sc/source/ui/view/printfun.cxx b/sc/source/ui/view/printfun.cxx index aa13c16739e1..cff117ff905f 100644 --- a/sc/source/ui/view/printfun.cxx +++ b/sc/source/ui/view/printfun.cxx @@ -523,7 +523,7 @@ void ScPrintFunc::DrawToDev( ScDocument* pDoc, OutputDevice* pDev, double /* nPr { pDrawView.reset(new FmFormView( pModel, pDev )); pDrawView->ShowSdrPage(pDrawView->GetModel()->GetPage(nTab)); - pDrawView->SetPrintPreview( true ); + pDrawView->SetPrintPreview(); aOutputData.SetDrawView( pDrawView.get() ); } @@ -693,8 +693,7 @@ bool ScPrintFunc::AdjustPrintArea( bool bNew ) bChangeRow = true; } - pDoc->ExtendMerge( nStartCol,nStartRow, nEndCol,nEndRow, nPrintTab, - false ); // no Refresh, incl. Attrs + pDoc->ExtendMerge( nStartCol,nStartRow, nEndCol,nEndRow, nPrintTab ); // no Refresh, incl. Attrs if ( bChangeCol ) { @@ -1798,7 +1797,7 @@ void ScPrintFunc::PrintHF( long nPageNo, bool bHeader, long nStartY, long nDif = aPaperSize.Height() - (long) pEditEngine->GetTextHeight(); if (nDif > 0) aDraw.Y() += nDif / 2; - pEditEngine->Draw( pDev, aDraw, 0 ); + pEditEngine->Draw( pDev, aDraw ); } // center @@ -1812,7 +1811,7 @@ void ScPrintFunc::PrintHF( long nPageNo, bool bHeader, long nStartY, long nDif = aPaperSize.Height() - (long) pEditEngine->GetTextHeight(); if (nDif > 0) aDraw.Y() += nDif / 2; - pEditEngine->Draw( pDev, aDraw, 0 ); + pEditEngine->Draw( pDev, aDraw ); } // right @@ -1826,7 +1825,7 @@ void ScPrintFunc::PrintHF( long nPageNo, bool bHeader, long nStartY, long nDif = aPaperSize.Height() - (long) pEditEngine->GetTextHeight(); if (nDif > 0) aDraw.Y() += nDif / 2; - pEditEngine->Draw( pDev, aDraw, 0 ); + pEditEngine->Draw( pDev, aDraw ); } pDev->SetClipRegion(); @@ -1883,14 +1882,14 @@ long ScPrintFunc::DoNotes( long nNoteStart, bool bDoPrint, ScPreviewLocationData { if (bDoPrint) { - pEditEngine->Draw( pDev, Point( nPosX, nPosY ), 0 ); + pEditEngine->Draw( pDev, Point( nPosX, nPosY ) ); OUString aMarkStr(rPos.Format(SCA_VALID, pDoc, pDoc->GetAddressConvention())); aMarkStr += ":"; // cell position also via EditEngine, for correct positioning pEditEngine->SetText(aMarkStr); - pEditEngine->Draw( pDev, Point( aPageRect.Left(), nPosY ), 0 ); + pEditEngine->Draw( pDev, Point( aPageRect.Left(), nPosY ) ); } if ( pLocationData ) diff --git a/sc/source/ui/view/select.cxx b/sc/source/ui/view/select.cxx index 09169f11c695..fe1ffff7063b 100644 --- a/sc/source/ui/view/select.cxx +++ b/sc/source/ui/view/select.cxx @@ -238,7 +238,7 @@ void ScViewFunctionSet::SetAnchor( SCCOL nPosX, SCROW nPosY ) if (bRefMode) { - pView->DoneRefMode( false ); + pView->DoneRefMode(); aAnchorPos.Set( nPosX, nPosY, nTab ); pView->InitRefMode( aAnchorPos.Col(), aAnchorPos.Row(), aAnchorPos.Tab(), SC_REFTYPE_REF ); @@ -721,11 +721,11 @@ void ScViewFunctionSet::DeselectAll() bool bRefMode = SC_MOD()->IsFormulaMode(); if (bRefMode) { - pViewData->GetView()->DoneRefMode( false ); + pViewData->GetView()->DoneRefMode(); } else { - pViewData->GetView()->DoneBlockMode( false ); + pViewData->GetView()->DoneBlockMode(); pViewData->GetViewShell()->UpdateInputHandler(); } @@ -912,7 +912,7 @@ void ScHeaderFunctionSet::DeselectAtPoint( const Point& /* rPointPixel */ ) void ScHeaderFunctionSet::DeselectAll() { - pViewData->GetView()->DoneBlockMode( false ); + pViewData->GetView()->DoneBlockMode(); bAnchor = false; } diff --git a/sc/source/ui/view/tabview.cxx b/sc/source/ui/view/tabview.cxx index 9e4fccabe12b..a273d9c2851d 100644 --- a/sc/source/ui/view/tabview.cxx +++ b/sc/source/ui/view/tabview.cxx @@ -1389,7 +1389,7 @@ void ScTabView::UpdateHeaderWidth( const ScVSplitPos* pWhich, const SCROW* pPosY nEndPos = *pPosY; else nEndPos = aViewData.GetPosY( SC_SPLIT_BOTTOM ); - nEndPos += aViewData.CellsAtY( nEndPos, 1, SC_SPLIT_BOTTOM, SC_SIZE_NONE ); // VisibleCellsY + nEndPos += aViewData.CellsAtY( nEndPos, 1, SC_SPLIT_BOTTOM ); // VisibleCellsY if (nEndPos > MAXROW) nEndPos = lcl_LastVisible( aViewData ); @@ -1400,7 +1400,7 @@ void ScTabView::UpdateHeaderWidth( const ScVSplitPos* pWhich, const SCROW* pPosY nTopEnd = *pPosY; else nTopEnd = aViewData.GetPosY( SC_SPLIT_TOP ); - nTopEnd += aViewData.CellsAtY( nTopEnd, 1, SC_SPLIT_TOP, SC_SIZE_NONE );// VisibleCellsY + nTopEnd += aViewData.CellsAtY( nTopEnd, 1, SC_SPLIT_TOP );// VisibleCellsY if (nTopEnd > MAXROW) nTopEnd = lcl_LastVisible( aViewData ); @@ -2226,7 +2226,7 @@ void ScTabView::EnableRefInput(bool bFlag) // ab hier dynamisch angelegte - if(pTabControl!=nullptr) pTabControl->EnableInput(bFlag,true); + if(pTabControl!=nullptr) pTabControl->EnableInput(bFlag); if(pGridWin[SC_SPLIT_BOTTOMLEFT]!=nullptr) pGridWin[SC_SPLIT_BOTTOMLEFT]->EnableInput(bFlag,false); diff --git a/sc/source/ui/view/tabview2.cxx b/sc/source/ui/view/tabview2.cxx index 6d355f70af20..1df73e426bc8 100644 --- a/sc/source/ui/view/tabview2.cxx +++ b/sc/source/ui/view/tabview2.cxx @@ -483,7 +483,7 @@ void ScTabView::MarkCursor( SCCOL nCurX, SCROW nCurY, SCTAB nCurZ, bool bOldShift = bMoveIsShift; bMoveIsShift = false; // wirklich umsetzen - DoneBlockMode(false); //! direkt Variablen setzen? (-> kein Geflacker) + DoneBlockMode(); //! direkt Variablen setzen? (-> kein Geflacker) bMoveIsShift = bOldShift; InitBlockMode( aMarkRange.aStart.Col(), aMarkRange.aStart.Row(), diff --git a/sc/source/ui/view/tabview3.cxx b/sc/source/ui/view/tabview3.cxx index 80f7858f6363..b434c4ab2da8 100644 --- a/sc/source/ui/view/tabview3.cxx +++ b/sc/source/ui/view/tabview3.cxx @@ -105,7 +105,7 @@ void ScTabView::ClickCursor( SCCOL nPosX, SCROW nPosY, bool bControl ) if ( bRefMode ) { - DoneRefMode( false ); + DoneRefMode(); if (bControl) SC_MOD()->AddRefEntry(); @@ -1535,7 +1535,7 @@ void ScTabView::SetTabNo( SCTAB nTab, bool bNew, bool bExtendSelection, bool bSa FmFormShell* pFormSh = aViewData.GetViewShell()->GetFormShell(); if (pFormSh) { - bool bAllowed = pFormSh->PrepareClose(true); + bool bAllowed = pFormSh->PrepareClose(); if (!bAllowed) { //! Fehlermeldung? oder macht das die FormShell selber? @@ -2381,7 +2381,7 @@ void ScTabView::ActivateView( bool bActivate, bool bFirst ) aViewData.Activate(bActivate); - PaintBlock(false); // Repaint, Markierung je nach Active-Status + PaintBlock(); // Repaint, Markierung je nach Active-Status if (!bActivate) HideAllCursors(); // Cursor diff --git a/sc/source/ui/view/tabview5.cxx b/sc/source/ui/view/tabview5.cxx index 971fe9cb0cc0..c3f551647305 100644 --- a/sc/source/ui/view/tabview5.cxx +++ b/sc/source/ui/view/tabview5.cxx @@ -335,7 +335,7 @@ void ScTabView::UpdateLayerLocks() pDrawView->SetLayerLocked( pLayer->GetName(), bProt || !bEx || bShared ); pLayer = rAdmin.GetLayerPerID(SC_LAYER_INTERN); if (pLayer) - pDrawView->SetLayerLocked( pLayer->GetName(), true ); + pDrawView->SetLayerLocked( pLayer->GetName() ); pLayer = rAdmin.GetLayerPerID(SC_LAYER_FRONT); if (pLayer) pDrawView->SetLayerLocked( pLayer->GetName(), bProt || bShared ); @@ -478,7 +478,7 @@ void ScTabView::DrawEnableAnim(bool bSet) { if ( !pDrawView->IsAnimationEnabled() ) { - pDrawView->SetAnimationEnabled(true); + pDrawView->SetAnimationEnabled(); // Animierte GIFs muessen wieder gestartet werden: ScDocument* pDoc = aViewData.GetDocument(); diff --git a/sc/source/ui/view/tabvwsh3.cxx b/sc/source/ui/view/tabvwsh3.cxx index f0fc87f34a63..0e265f90630c 100644 --- a/sc/source/ui/view/tabvwsh3.cxx +++ b/sc/source/ui/view/tabvwsh3.cxx @@ -269,8 +269,7 @@ void ScTabViewShell::Execute( SfxRequest& rReq ) { // URL has to be decoded for escaped characters (%20) aAddress = INetURLObject::decode( aAddress, - INetURLObject::DECODE_WITH_CHARSET, - RTL_TEXTENCODING_UTF8 ); + INetURLObject::DECODE_WITH_CHARSET ); } bool bFound = false; diff --git a/sc/source/ui/view/viewdata.cxx b/sc/source/ui/view/viewdata.cxx index 18857ae06101..d6400dbe05a7 100644 --- a/sc/source/ui/view/viewdata.cxx +++ b/sc/source/ui/view/viewdata.cxx @@ -1696,22 +1696,22 @@ SCROW ScViewData::CellsAtY( SCsROW nPosY, SCsROW nDir, ScVSplitPos eWhichY, sal_ SCCOL ScViewData::VisibleCellsX( ScHSplitPos eWhichX ) const { - return CellsAtX( GetPosX( eWhichX ), 1, eWhichX, SC_SIZE_NONE ); + return CellsAtX( GetPosX( eWhichX ), 1, eWhichX ); } SCROW ScViewData::VisibleCellsY( ScVSplitPos eWhichY ) const { - return CellsAtY( GetPosY( eWhichY ), 1, eWhichY, SC_SIZE_NONE ); + return CellsAtY( GetPosY( eWhichY ), 1, eWhichY ); } SCCOL ScViewData::PrevCellsX( ScHSplitPos eWhichX ) const { - return CellsAtX( GetPosX( eWhichX ), -1, eWhichX, SC_SIZE_NONE ); + return CellsAtX( GetPosX( eWhichX ), -1, eWhichX ); } SCROW ScViewData::PrevCellsY( ScVSplitPos eWhichY ) const { - return CellsAtY( GetPosY( eWhichY ), -1, eWhichY, SC_SIZE_NONE ); + return CellsAtY( GetPosY( eWhichY ), -1, eWhichY ); } bool ScViewData::GetMergeSizePixel( SCCOL nX, SCROW nY, long& rSizeXPix, long& rSizeYPix ) const diff --git a/sc/source/ui/view/viewfun2.cxx b/sc/source/ui/view/viewfun2.cxx index 8ee3c3fb045e..c085289ea2b1 100644 --- a/sc/source/ui/view/viewfun2.cxx +++ b/sc/source/ui/view/viewfun2.cxx @@ -1739,7 +1739,7 @@ bool ScViewFunc::SearchAndReplace( const SvxSearchItem* pSearchItem, SfxViewFrame* pViewFrm = SfxViewFrame::Current(); if (pViewFrm) { - pViewFrm->ShowChildWindow(sc::SearchResultsDlgWrapper::GetChildWindowId(), true); + pViewFrm->ShowChildWindow(sc::SearchResultsDlgWrapper::GetChildWindowId()); SfxChildWindow* pWnd = pViewFrm->GetChildWindow(sc::SearchResultsDlgWrapper::GetChildWindowId()); if (pWnd) { diff --git a/sc/source/ui/view/viewfun3.cxx b/sc/source/ui/view/viewfun3.cxx index 62626012f0d1..d994e2b2a6fb 100644 --- a/sc/source/ui/view/viewfun3.cxx +++ b/sc/source/ui/view/viewfun3.cxx @@ -683,7 +683,7 @@ void ScViewFunc::PasteFromTransferable( const uno::Reference<datatransfer::XTran PasteDataFormat( nFormatId, aDataHelper.GetTransferable(), GetViewData().GetCurX(), GetViewData().GetCurY(), - NULL, false, false ); + NULL, false ); } } } @@ -1145,7 +1145,7 @@ bool ScViewFunc::PasteFromClip( InsertDeleteFlags nFlags, ScDocument* pClipDoc, // content outside the clip area. for (SCTAB nClipTab=0; nClipTab < pClipDoc->GetTableCount(); nClipTab++) if ( pClipDoc->HasTable(nClipTab) ) - if ( pClipDoc->ExtendMerge( nClipStartX,nClipStartY, nUndoEndCol,nUndoEndRow, nClipTab, false ) ) + if ( pClipDoc->ExtendMerge( nClipStartX,nClipStartY, nUndoEndCol,nUndoEndRow, nClipTab ) ) bClipOver = true; nUndoEndCol -= nClipStartX + nClipSizeX; nUndoEndRow -= nClipStartY + nClipSizeY; // now contains only the difference added by ExtendMerge @@ -1158,7 +1158,7 @@ bool ScViewFunc::PasteFromClip( InsertDeleteFlags nFlags, ScDocument* pClipDoc, return false; } - pDoc->ExtendMergeSel( nStartCol,nStartRow, nUndoEndCol,nUndoEndRow, aFilteredMark, false ); + pDoc->ExtendMergeSel( nStartCol,nStartRow, nUndoEndCol,nUndoEndRow, aFilteredMark ); // check cell-protection @@ -1228,7 +1228,7 @@ bool ScViewFunc::PasteFromClip( InsertDeleteFlags nFlags, ScDocument* pClipDoc, if ( bCutMode ) { pRefUndoDoc = new ScDocument( SCDOCMODE_UNDO ); - pRefUndoDoc->InitUndo( pDoc, 0, nTabCount-1, false, false ); + pRefUndoDoc->InitUndo( pDoc, 0, nTabCount-1, false ); pUndoData = new ScRefUndoData( pDoc ); } @@ -1515,8 +1515,8 @@ bool ScViewFunc::PasteMultiRangesFromClip( if (pDoc->IsUndoEnabled()) { pUndoDoc.reset(new ScDocument(SCDOCMODE_UNDO)); - pUndoDoc->InitUndoSelected(pDoc, aMark, false, false); - pDoc->CopyToDocument(aMarkedRange, nUndoFlags, false, pUndoDoc.get(), &aMark, true); + pUndoDoc->InitUndoSelected(pDoc, aMark, false); + pDoc->CopyToDocument(aMarkedRange, nUndoFlags, false, pUndoDoc.get(), &aMark); } ::std::unique_ptr<ScDocument> pMixDoc; @@ -1525,8 +1525,8 @@ bool ScViewFunc::PasteMultiRangesFromClip( if ( nFlags & IDF_CONTENTS ) { pMixDoc.reset(new ScDocument(SCDOCMODE_UNDO)); - pMixDoc->InitUndoSelected(pDoc, aMark, false, false); - pDoc->CopyToDocument(aMarkedRange, IDF_CONTENTS, false, pMixDoc.get(), &aMark, true); + pMixDoc->InitUndoSelected(pDoc, aMark, false); + pDoc->CopyToDocument(aMarkedRange, IDF_CONTENTS, false, pMixDoc.get(), &aMark); } } @@ -1580,7 +1580,7 @@ bool ScViewFunc::PasteMultiRangesFromClip( if (bInsertCells) pUndoMgr->AddUndoAction(new ScUndoWrapper(pUndo), true); else - pUndoMgr->AddUndoAction(pUndo, false); + pUndoMgr->AddUndoAction(pUndo); pUndoMgr->LeaveListAction(); } @@ -1661,11 +1661,11 @@ bool ScViewFunc::PasteFromClipToMultiRanges( if (pDoc->IsUndoEnabled()) { pUndoDoc.reset(new ScDocument(SCDOCMODE_UNDO)); - pUndoDoc->InitUndoSelected(pDoc, aMark, false, false); + pUndoDoc->InitUndoSelected(pDoc, aMark, false); for (size_t i = 0, n = aRanges.size(); i < n; ++i) { pDoc->CopyToDocument( - *aRanges[i], nUndoFlags, false, pUndoDoc.get(), &aMark, true); + *aRanges[i], nUndoFlags, false, pUndoDoc.get(), &aMark); } } @@ -1675,11 +1675,11 @@ bool ScViewFunc::PasteFromClipToMultiRanges( if (nFlags & IDF_CONTENTS) { pMixDoc.reset(new ScDocument(SCDOCMODE_UNDO)); - pMixDoc->InitUndoSelected(pDoc, aMark, false, false); + pMixDoc->InitUndoSelected(pDoc, aMark, false); for (size_t i = 0, n = aRanges.size(); i < n; ++i) { pDoc->CopyToDocument( - *aRanges[i], IDF_CONTENTS, false, pMixDoc.get(), &aMark, true); + *aRanges[i], IDF_CONTENTS, false, pMixDoc.get(), &aMark); } } } @@ -1737,7 +1737,7 @@ bool ScViewFunc::PasteFromClipToMultiRanges( ScUndoPaste* pUndo = new ScUndoPaste( pDocSh, aRanges, aMark, pUndoDoc.release(), NULL, nFlags|nUndoFlags, NULL, false, &aOptions); - pUndoMgr->AddUndoAction(pUndo, false); + pUndoMgr->AddUndoAction(pUndo); pUndoMgr->LeaveListAction(); } diff --git a/sc/source/ui/view/viewfun6.cxx b/sc/source/ui/view/viewfun6.cxx index 20db7b000d52..23652346988c 100644 --- a/sc/source/ui/view/viewfun6.cxx +++ b/sc/source/ui/view/viewfun6.cxx @@ -199,7 +199,7 @@ void ScViewFunc::DetectiveMarkPred() else { ScRange aRange; - ScRefTokenHelper::getRangeFromToken(aRange, p, aCurPos, false); + ScRefTokenHelper::getRangeFromToken(aRange, p, aCurPos); if (aRange.aStart.Tab() != aCurPos.Tab()) { // The first precedent range is on a different sheet. Jump to it @@ -325,7 +325,7 @@ void ScViewFunc::InsertCurrentTime(short nReqFmt, const OUString& rUndoStr) EditView* pTableView = pInputHdl->GetTableView(); if (pTableView) pTableView->InsertText( aString); - pInputHdl->DataChanged( false, true); + pInputHdl->DataChanged( false ); } else { diff --git a/sc/source/ui/view/viewfun7.cxx b/sc/source/ui/view/viewfun7.cxx index 578e62ed0d73..311924e60d89 100644 --- a/sc/source/ui/view/viewfun7.cxx +++ b/sc/source/ui/view/viewfun7.cxx @@ -133,7 +133,7 @@ void ScViewFunc::PasteDraw( const Point& rLogicPos, SdrModel* pModel, pDragEditView->GetSdrPageView()->GetPage() ) { if ( nDiffX != 0 || nDiffY != 0 ) - pDragEditView->MoveAllMarked(Size(nDiffX,nDiffY), false); + pDragEditView->MoveAllMarked(Size(nDiffX,nDiffY)); } else { diff --git a/sc/source/ui/view/viewfunc.cxx b/sc/source/ui/view/viewfunc.cxx index ce4826165f00..7375f184e4a3 100644 --- a/sc/source/ui/view/viewfunc.cxx +++ b/sc/source/ui/view/viewfunc.cxx @@ -1661,7 +1661,7 @@ void ScViewFunc::DeleteMulti( bool bRows, bool bRecord ) // all Formulas because of references SCTAB nTabCount = rDoc.GetTableCount(); - pUndoDoc->AddUndoTab( 0, nTabCount-1, false, false ); + pUndoDoc->AddUndoTab( 0, nTabCount-1, false ); rDoc.CopyToDocument( 0,0,0, MAXCOL,MAXROW,MAXTAB, IDF_FORMULA,false,pUndoDoc ); pUndoData = new ScRefUndoData( &rDoc ); @@ -1892,9 +1892,9 @@ void ScViewFunc::SetWidthOrHeight( if (bWidth) { if ( *itr == nFirstTab ) - pUndoDoc->InitUndo( &rDoc, *itr, *itr, true, false ); + pUndoDoc->InitUndo( &rDoc, *itr, *itr, true ); else - pUndoDoc->AddUndoTab( *itr, *itr, true, false ); + pUndoDoc->AddUndoTab( *itr, *itr, true ); rDoc.CopyToDocument( static_cast<SCCOL>(nStart), 0, *itr, static_cast<SCCOL>(nEnd), MAXROW, *itr, IDF_NONE, false, pUndoDoc ); @@ -2121,7 +2121,7 @@ void ScViewFunc::SetMarkedWidthOrHeight( bool bWidth, ScSizeMode eMode, sal_uInt SCTAB nTab = GetViewData().GetTabNo(); DoneBlockMode(); InitOwnBlockMode(); - rMark.SetMultiMarkArea( ScRange( nCol,nRow,nTab ), true ); + rMark.SetMultiMarkArea( ScRange( nCol,nRow,nTab ) ); MarkDataChanged(); } |