diff options
author | Noel Grandin <noel@peralex.com> | 2015-08-20 12:58:15 +0200 |
---|---|---|
committer | Noel Grandin <noel@peralex.com> | 2015-08-20 13:52:17 +0200 |
commit | c53b880f0058889f9f4850bb474323213cd49c81 (patch) | |
tree | e284030d45f41ed264abd7460c5bca59c08d784c /sd/source/ui | |
parent | b0f205a33da99da3c8b4a376eecf395bbbc2df3b (diff) |
loplugin:defaultparams
Change-Id: Ib7a4be246e5d82f5a704a188f8b950f187d04138
Diffstat (limited to 'sd/source/ui')
24 files changed, 44 insertions, 44 deletions
diff --git a/sd/source/ui/dlg/LayerTabBar.cxx b/sd/source/ui/dlg/LayerTabBar.cxx index 500d78cbcfd0..4e47bbc81121 100644 --- a/sd/source/ui/dlg/LayerTabBar.cxx +++ b/sd/source/ui/dlg/LayerTabBar.cxx @@ -272,7 +272,7 @@ void LayerTabBar::EndRenaming() // the View then already has to know the Layer pView->SetActiveLayer(aNewName); pLayer->SetName(aNewName); - rDoc.SetChanged(true); + rDoc.SetChanged(); } } } diff --git a/sd/source/ui/dlg/PhotoAlbumDialog.cxx b/sd/source/ui/dlg/PhotoAlbumDialog.cxx index 2740dbc80ab1..fe664700d343 100644 --- a/sd/source/ui/dlg/PhotoAlbumDialog.cxx +++ b/sd/source/ui/dlg/PhotoAlbumDialog.cxx @@ -504,8 +504,8 @@ IMPL_LINK_NOARG(SdPhotoAlbumDialog, FileHdl) { // Store full path, show filename only. Use INetURLObject to display spaces in filename correctly INetURLObject aUrl = INetURLObject(aFilesArr[i]); - sal_Int16 nPos = pImagesLst->InsertEntry( aUrl.GetLastName(INetURLObject::DECODE_WITH_CHARSET, RTL_TEXTENCODING_UTF8) ); - pImagesLst->SetEntryData(nPos, new OUString(aUrl.GetMainURL(INetURLObject::DECODE_WITH_CHARSET, RTL_TEXTENCODING_UTF8))); + sal_Int16 nPos = pImagesLst->InsertEntry( aUrl.GetLastName(INetURLObject::DECODE_WITH_CHARSET) ); + pImagesLst->SetEntryData(nPos, new OUString(aUrl.GetMainURL(INetURLObject::DECODE_WITH_CHARSET))); } } } diff --git a/sd/source/ui/dlg/dlgolbul.cxx b/sd/source/ui/dlg/dlgolbul.cxx index 2582ca25c534..292029707244 100644 --- a/sd/source/ui/dlg/dlgolbul.cxx +++ b/sd/source/ui/dlg/dlgolbul.cxx @@ -116,7 +116,7 @@ OutlineBulletDlg::OutlineBulletDlg( if(pRule) { SvxNumRule aNewRule( *pRule ); - aNewRule.SetFeatureFlag( SvxNumRuleFlags::NO_NUMBERS, true ); + aNewRule.SetFeatureFlag( SvxNumRuleFlags::NO_NUMBERS ); SvxNumBulletItem aNewItem( aNewRule, EE_PARA_NUMBULLET ); aInputSet.Put(aNewItem); diff --git a/sd/source/ui/dlg/ins_paste.cxx b/sd/source/ui/dlg/ins_paste.cxx index ad9e9821e4e0..2b4a592ef63f 100644 --- a/sd/source/ui/dlg/ins_paste.cxx +++ b/sd/source/ui/dlg/ins_paste.cxx @@ -25,7 +25,7 @@ SdInsertPasteDlg::SdInsertPasteDlg(vcl::Window* pWindow) { get(m_pRbBefore, "before"); get(m_pRbAfter, "after"); - m_pRbAfter->Check( true ); + m_pRbAfter->Check(); } SdInsertPasteDlg::~SdInsertPasteDlg() diff --git a/sd/source/ui/dlg/inspagob.cxx b/sd/source/ui/dlg/inspagob.cxx index c77ba67a4c40..ed7d6720923e 100644 --- a/sd/source/ui/dlg/inspagob.cxx +++ b/sd/source/ui/dlg/inspagob.cxx @@ -87,7 +87,7 @@ void SdInsertPagesObjsDlg::Reset() m_pLbTree->InsertEntry( rName, aImgText, aImgText ); } - m_pCbxMasters->Check( true ); + m_pCbxMasters->Check(); } std::vector<OUString> SdInsertPagesObjsDlg::GetList( const sal_uInt16 nType ) diff --git a/sd/source/ui/dlg/navigatr.cxx b/sd/source/ui/dlg/navigatr.cxx index 343be422b595..e0ae7d049e15 100644 --- a/sd/source/ui/dlg/navigatr.cxx +++ b/sd/source/ui/dlg/navigatr.cxx @@ -715,7 +715,7 @@ void SdNavigatorWin::RefreshDocumentLB( const OUString* pDocName ) // is shown in url notation! aStr = pDocShell->GetName(); - maLbDocs->InsertEntry( aStr, LISTBOX_APPEND ); + maLbDocs->InsertEntry( aStr ); if( pDocShell == pCurrentDocShell ) aInfo.SetActive(); diff --git a/sd/source/ui/dlg/present.cxx b/sd/source/ui/dlg/present.cxx index bd67c3766e34..8809295caed3 100644 --- a/sd/source/ui/dlg/present.cxx +++ b/sd/source/ui/dlg/present.cxx @@ -136,11 +136,11 @@ SdStartPresentationDlg::SdStartPresentationDlg( vcl::Window* pWindow, aCbxAutoLogo->Check( static_cast<const SfxBoolItem&>( rOutAttrs.Get( ATTR_PRESENT_SHOW_PAUSELOGO ) ).GetValue() ); if( bWindow ) - aRbtWindow->Check( true ); + aRbtWindow->Check(); else if( bEndless ) - aRbtAuto->Check( true ); + aRbtAuto->Check(); else - aRbtStandard->Check( true ); + aRbtStandard->Check(); InitMonitorSettings(); diff --git a/sd/source/ui/dlg/sdtreelb.cxx b/sd/source/ui/dlg/sdtreelb.cxx index 9282a4788b34..bac890e5890c 100644 --- a/sd/source/ui/dlg/sdtreelb.cxx +++ b/sd/source/ui/dlg/sdtreelb.cxx @@ -1146,9 +1146,9 @@ void SdPageObjsTLB::StartDrag( sal_Int8 nAction, const Point& rPosPixel) // Select all entries and disable them as drop targets. SetSelectionMode(MULTIPLE_SELECTION); - SetCursor(NULL, false); + SetCursor(static_cast<SvTreeListEntry*>(NULL)); SelectAll(true, false); - EnableSelectionAsDropTarget(false, true); + EnableSelectionAsDropTarget(false); // Enable only the entries as drop targets that are children of the // page under the mouse. @@ -1156,16 +1156,16 @@ void SdPageObjsTLB::StartDrag( sal_Int8 nAction, const Point& rPosPixel) if (pParent != NULL) { SelectAll(false, false); - Select(pParent, true); + Select(pParent); // for (SvTreeListEntry*pChild=FirstChild(pParent); pChild!=NULL; pChild=NextSibling(pChild)) // Select(pChild, sal_True); - EnableSelectionAsDropTarget(true, true);//sal_False); + EnableSelectionAsDropTarget(true);//sal_False); } // Set selection back to the entry under the mouse. SelectAll(false, false); SetSelectionMode(SINGLE_SELECTION); - Select(pEntry, true); + Select(pEntry); // We can delete the Navigator from ExecuteDrag (when switching to // another document type), but that would kill the StarView MouseMove diff --git a/sd/source/ui/docshell/docshel3.cxx b/sd/source/ui/docshell/docshel3.cxx index 81d7118716bc..567bcdb520a6 100644 --- a/sd/source/ui/docshell/docshel3.cxx +++ b/sd/source/ui/docshell/docshel3.cxx @@ -297,7 +297,7 @@ void DrawDocShell::Execute( SfxRequest& rReq ) const OUString aStrNone("LANGUAGE_NONE"); const OUString aStrResetLangs("RESET_LANGUAGES"); sal_Int32 nPos = -1; - if (-1 != (nPos = aNewLangTxt.indexOf( aDocumentLangPrefix , 0 ))) + if (-1 != (nPos = aNewLangTxt.indexOf( aDocumentLangPrefix ))) { aNewLangTxt = aNewLangTxt.replaceAt( nPos, aDocumentLangPrefix.getLength(), "" ); } diff --git a/sd/source/ui/docshell/docshel4.cxx b/sd/source/ui/docshell/docshel4.cxx index 3b568abd2f70..93de709e9b24 100644 --- a/sd/source/ui/docshell/docshel4.cxx +++ b/sd/source/ui/docshell/docshel4.cxx @@ -390,7 +390,7 @@ bool DrawDocShell::ImportFrom(SfxMedium &rMedium, nControlWord &=~ EEControlBits::ULSPACEFIRSTPARA; ((EditEngine&)rOutl.GetEditEngine()).SetControlWord( nControlWord ); - mpDoc->SetSummationOfParagraphs( true ); + mpDoc->SetSummationOfParagraphs(); } const bool bRet = SfxObjectShell::ImportFrom(rMedium, xInsertPosition); @@ -657,7 +657,7 @@ bool DrawDocShell::SaveCompleted( const ::com::sun::star::uno::Reference< ::com: SfxViewFrame::Current(); if( pFrame ) - pFrame->GetBindings().Invalidate( SID_NAVIGATOR_STATE, true, false ); + pFrame->GetBindings().Invalidate( SID_NAVIGATOR_STATE, true ); } return bRet; } @@ -825,7 +825,7 @@ bool DrawDocShell::GotoBookmark(const OUString& rBookmark) ? pDrawViewShell->GetViewFrame() : SfxViewFrame::Current() )->GetBindings(); - rBindings.Invalidate(SID_NAVIGATOR_STATE, true, false); + rBindings.Invalidate(SID_NAVIGATOR_STATE, true); rBindings.Invalidate(SID_NAVIGATOR_PAGENAME); } @@ -1010,7 +1010,7 @@ bool DrawDocShell::GotoTreeBookmark(const OUString& rBookmark) SfxBindings& rBindings = ( ( mpViewShell && mpViewShell->GetViewFrame() ) ? mpViewShell->GetViewFrame() : SfxViewFrame::Current() )->GetBindings(); - rBindings.Invalidate(SID_NAVIGATOR_STATE, true, false); + rBindings.Invalidate(SID_NAVIGATOR_STATE, true); rBindings.Invalidate(SID_NAVIGATOR_PAGENAME); } diff --git a/sd/source/ui/framework/configuration/ResourceId.cxx b/sd/source/ui/framework/configuration/ResourceId.cxx index 3f343c989069..e3a780a60daa 100644 --- a/sd/source/ui/framework/configuration/ResourceId.cxx +++ b/sd/source/ui/framework/configuration/ResourceId.cxx @@ -187,7 +187,7 @@ OUString SAL_CALL // Get the prefix that ends with the second "/". const OUString& rsResourceURL (maResourceURLs[0]); - sal_Int32 nPrefixEnd (rsResourceURL.indexOf('/', 0)); + sal_Int32 nPrefixEnd (rsResourceURL.indexOf('/')); if (nPrefixEnd >= 0) nPrefixEnd = rsResourceURL.indexOf('/', nPrefixEnd+1) + 1; else diff --git a/sd/source/ui/framework/factories/ChildWindowPane.cxx b/sd/source/ui/framework/factories/ChildWindowPane.cxx index c9746f4e7bd9..240c1686323a 100644 --- a/sd/source/ui/framework/factories/ChildWindowPane.cxx +++ b/sd/source/ui/framework/factories/ChildWindowPane.cxx @@ -156,7 +156,7 @@ vcl::Window* ChildWindowPane::GetWindow() // The child window is not yet visible. Ask the view frame // to show it and try again to get access to the child // window. - pViewFrame->ShowChildWindow(mnChildWindowId, true); + pViewFrame->ShowChildWindow(mnChildWindowId); pChildWindow = pViewFrame->GetChildWindow(mnChildWindowId); } diff --git a/sd/source/ui/func/fucushow.cxx b/sd/source/ui/func/fucushow.cxx index e8eaec958789..3c898a24e2cb 100644 --- a/sd/source/ui/func/fucushow.cxx +++ b/sd/source/ui/func/fucushow.cxx @@ -63,7 +63,7 @@ void FuCustomShowDlg::DoExecute( SfxRequest& ) sal_uInt16 nRet = pDlg->Execute(); if( pDlg->IsModified() ) { - mpDoc->SetChanged( true ); + mpDoc->SetChanged(); sd::PresentationSettings& rSettings = mpDoc->getPresentationSettings(); rSettings.mbCustomShow = pDlg->IsCustomShow(); } diff --git a/sd/source/ui/func/fudraw.cxx b/sd/source/ui/func/fudraw.cxx index 9e4e134954ed..2d34ed61be5b 100644 --- a/sd/source/ui/func/fudraw.cxx +++ b/sd/source/ui/func/fudraw.cxx @@ -429,7 +429,7 @@ bool FuDraw::KeyInput(const KeyEvent& rKEvt) { // mark last object mpView->UnmarkAllObj(); - mpView->MarkNextObj(false); + mpView->MarkNextObj(); if(mpView->AreObjectsMarked()) mpView->MakeVisible(mpView->GetAllMarkedRect(), *mpWindow); diff --git a/sd/source/ui/func/fuformatpaintbrush.cxx b/sd/source/ui/func/fuformatpaintbrush.cxx index 380bcef2bd1f..8c6ff03dbadb 100644 --- a/sd/source/ui/func/fuformatpaintbrush.cxx +++ b/sd/source/ui/func/fuformatpaintbrush.cxx @@ -140,7 +140,7 @@ bool FuFormatPaintBrush::MouseButtonDown(const MouseEvent& rMEvt) { sal_uInt16 nHitLog = sal_uInt16 ( mpWindow->PixelToLogic(Size(HITPIX,0)).Width() ); bool bToggle = false; - mpView->MarkObj(mpWindow->PixelToLogic( rMEvt.GetPosPixel() ), nHitLog, bToggle, false); + mpView->MarkObj(mpWindow->PixelToLogic( rMEvt.GetPosPixel() ), nHitLog, bToggle); return true; } } diff --git a/sd/source/ui/func/fuinsert.cxx b/sd/source/ui/func/fuinsert.cxx index 7691f2dc0113..c4689548566b 100644 --- a/sd/source/ui/func/fuinsert.cxx +++ b/sd/source/ui/func/fuinsert.cxx @@ -373,7 +373,7 @@ void FuInsertOLE::DoExecute( SfxRequest& rReq ) bool bRet = true; if( pPickObj ) - mpView->ReplaceObjectAtView(pPickObj, *pPV, pOleObj, true ); + mpView->ReplaceObjectAtView(pPickObj, *pPV, pOleObj ); else bRet = mpView->InsertObjectAtView(pOleObj, *pPV, SdrInsertFlags::SETDEFLAYER); diff --git a/sd/source/ui/func/fuinsfil.cxx b/sd/source/ui/func/fuinsfil.cxx index 8260105e62c6..9359bf493ed8 100644 --- a/sd/source/ui/func/fuinsfil.cxx +++ b/sd/source/ui/func/fuinsfil.cxx @@ -495,7 +495,7 @@ void FuInsertFile::InsTextOrRTFinDrMode(SfxMedium* pMedium) while ( pOutliner->GetParagraphCount() > 1 ) { Paragraph* pPara = pOutliner->GetParagraph( 0 ); - sal_uLong nLen = pOutliner->GetText( pPara, 1 ).getLength(); + sal_uLong nLen = pOutliner->GetText( pPara ).getLength(); pOutliner->QuickDelete( ESelection( 0, nLen, 1, 0 ) ); pOutliner->QuickInsertLineBreak( ESelection( 0, nLen, 0, nLen ) ); } diff --git a/sd/source/ui/func/fuolbull.cxx b/sd/source/ui/func/fuolbull.cxx index c4befa02eef1..e6538561d9b2 100644 --- a/sd/source/ui/func/fuolbull.cxx +++ b/sd/source/ui/func/fuolbull.cxx @@ -354,7 +354,7 @@ const SfxPoolItem* FuOutlineBullet::GetNumBulletItem(SfxItemSet& aNewAttr, sal_u if(pLclRule) { SvxNumRule aNewRule( *pLclRule ); - aNewRule.SetFeatureFlag( SvxNumRuleFlags::NO_NUMBERS, true ); + aNewRule.SetFeatureFlag( SvxNumRuleFlags::NO_NUMBERS ); SvxNumBulletItem aNewItem( aNewRule, EE_PARA_NUMBULLET ); aNewAttr.Put(aNewItem); diff --git a/sd/source/ui/func/fupage.cxx b/sd/source/ui/func/fupage.cxx index 75a58f2ac47e..518f35683542 100644 --- a/sd/source/ui/func/fupage.cxx +++ b/sd/source/ui/func/fupage.cxx @@ -457,7 +457,7 @@ const SfxItemSet* FuPage::ExecuteDialog( vcl::Window* pParent ) mpDoc->SetDefaultWritingMode( nVal == FRMDIR_HORI_RIGHT_TOP ? ::com::sun::star::text::WritingMode_RL_TB : ::com::sun::star::text::WritingMode_LR_TB ); } - mpDoc->SetChanged(true); + mpDoc->SetChanged(); // BackgroundFill of Masterpage: no hard attributes allowed SdrPage& rUsedMasterPage = mpPage->IsMasterPage() ? *mpPage : mpPage->TRG_GetMasterPage(); diff --git a/sd/source/ui/func/fupoor.cxx b/sd/source/ui/func/fupoor.cxx index e0ced626668a..b2be7eb2884e 100644 --- a/sd/source/ui/func/fupoor.cxx +++ b/sd/source/ui/func/fupoor.cxx @@ -754,7 +754,7 @@ bool FuPoor::KeyInput(const KeyEvent& rKEvt) // switch snapping off if(!bWasNoSnap) - ((SdrDragStat&)rDragStat).SetNoSnap(true); + ((SdrDragStat&)rDragStat).SetNoSnap(); if(bWasSnapEnabled) mpView->SetSnapEnabled(false); diff --git a/sd/source/ui/func/fusel.cxx b/sd/source/ui/func/fusel.cxx index dd9520af513b..63a78c3568ae 100644 --- a/sd/source/ui/func/fusel.cxx +++ b/sd/source/ui/func/fusel.cxx @@ -391,7 +391,7 @@ bool FuSelection::MouseButtonDown(const MouseEvent& rMEvt) bToggle = true; } - bMarked = mpView->MarkObj(aMDPos, nHitLog, bToggle, false); + bMarked = mpView->MarkObj(aMDPos, nHitLog, bToggle); } } @@ -537,7 +537,7 @@ bool FuSelection::MouseButtonDown(const MouseEvent& rMEvt) } else { - bMarked = mpView->MarkObj(aMDPos, nHitLog, rMEvt.IsShift(), false); + bMarked = mpView->MarkObj(aMDPos, nHitLog, rMEvt.IsShift()); } } @@ -687,7 +687,7 @@ bool FuSelection::MouseButtonUp(const MouseEvent& rMEvt) if (pPV->IsObjSelectable(pObj)) { mpView->UnmarkAllObj(); - mpView->MarkObj(pObj,pPV,false,false); + mpView->MarkObj(pObj,pPV,false); return true; } } @@ -1362,7 +1362,7 @@ bool FuSelection::AnimateObj(SdrObject* pObj, const Point& rPos) { // Assign verb mpView->UnmarkAll(); - mpView->MarkObj(pObj, mpView->GetSdrPageView(), false, false); + mpView->MarkObj(pObj, mpView->GetSdrPageView(), false); pDrViewSh->DoVerb((sal_Int16)pInfo->mnVerb); bAnimated = true; } diff --git a/sd/source/ui/func/fusldlg.cxx b/sd/source/ui/func/fusldlg.cxx index 8309219a8d31..870e44567b6b 100644 --- a/sd/source/ui/func/fusldlg.cxx +++ b/sd/source/ui/func/fusldlg.cxx @@ -222,7 +222,7 @@ void FuSlideShowDlg::DoExecute( SfxRequest& ) // is something has changed, we set the modified flag if ( bValuesChanged ) - mpDoc->SetChanged( true ); + mpDoc->SetChanged(); } } diff --git a/sd/source/ui/func/futempl.cxx b/sd/source/ui/func/futempl.cxx index 8404daaf82e2..445feef4126e 100644 --- a/sd/source/ui/func/futempl.cxx +++ b/sd/source/ui/func/futempl.cxx @@ -163,7 +163,7 @@ void FuTemplate::DoExecute( SfxRequest& rReq ) { case SID_STYLE_NEW: { - SfxStyleSheetBase *p = pSSPool->Find(aStyleName, (SfxStyleFamily) nFamily, SFXSTYLEBIT_ALL ); + SfxStyleSheetBase *p = pSSPool->Find(aStyleName, (SfxStyleFamily) nFamily ); if(p) { pSSPool->Remove(p); @@ -186,7 +186,7 @@ void FuTemplate::DoExecute( SfxRequest& rReq ) case SID_STYLE_NEW_BY_EXAMPLE: { // at the moment, the dialog to enter the name of the template is still opened - SfxStyleSheetBase *p = pSSPool->Find(aStyleName, (SfxStyleFamily) nFamily, SFXSTYLEBIT_ALL ); + SfxStyleSheetBase *p = pSSPool->Find(aStyleName, (SfxStyleFamily) nFamily ); if(p) { pSSPool->Remove(p); @@ -207,7 +207,7 @@ void FuTemplate::DoExecute( SfxRequest& rReq ) { pSSPool->Remove( pStyleSheet ); nRetMask = sal_uInt16(true); - mpDoc->SetChanged(true); + mpDoc->SetChanged(); } else { @@ -245,7 +245,7 @@ void FuTemplate::DoExecute( SfxRequest& rReq ) (pOldStyleSheet->GetFamily() == SD_STYLE_FAMILY_MASTERPAGE && mpDoc->GetDocumentType() == DOCUMENT_TYPE_DRAW) ) { mpView->SetStyleSheet( static_cast<SfxStyleSheet*>(pStyleSheet)); - mpDoc->SetChanged(true); + mpDoc->SetChanged(); mpViewShell->GetViewFrame()->GetBindings().Invalidate( SID_STYLE_FAMILY2 ); } } @@ -552,7 +552,7 @@ void FuTemplate::DoExecute( SfxRequest& rReq ) } } - mpDoc->SetChanged(true); + mpDoc->SetChanged(); } break; @@ -598,7 +598,7 @@ void FuTemplate::DoExecute( SfxRequest& rReq ) } static_cast<SfxStyleSheet*>( pStyleSheet )->Broadcast( SfxSimpleHint( SFX_HINT_DATACHANGED ) ); - mpDoc->SetChanged(true); + mpDoc->SetChanged(); mpViewShell->GetViewFrame()->GetBindings().Invalidate( SID_STYLE_FAMILY2 ); } @@ -625,7 +625,7 @@ void FuTemplate::DoExecute( SfxRequest& rReq ) mpView->SetStyleSheet( static_cast<SfxStyleSheet*>(pStyleSheet)); static_cast<SfxStyleSheet*>( pStyleSheet )->Broadcast( SfxSimpleHint( SFX_HINT_DATACHANGED ) ); - mpDoc->SetChanged(true); + mpDoc->SetChanged(); mpViewShell->GetViewFrame()->GetBindings().Invalidate( SID_STYLE_FAMILY2 ); } } diff --git a/sd/source/ui/func/futext.cxx b/sd/source/ui/func/futext.cxx index c3b40902319a..a359f7ad7d80 100644 --- a/sd/source/ui/func/futext.cxx +++ b/sd/source/ui/func/futext.cxx @@ -156,7 +156,7 @@ void FuText::disposing() { if(mpView) { - if(mpView->SdrEndTextEdit(false) == SDRENDTEXTEDIT_DELETED) + if(mpView->SdrEndTextEdit() == SDRENDTEXTEDIT_DELETED) mxTextObj.reset( 0 ); // reset the RequestHandler of the used Outliner to the handler of the document @@ -660,7 +660,7 @@ bool FuText::MouseButtonUp(const MouseEvent& rMEvt) if (pPV->IsObjSelectable(pObj)) { mpView->UnmarkAllObj(); - mpView->MarkObj(pObj,pPV,false,false); + mpView->MarkObj(pObj,pPV,false); return bReturn; } } |