diff options
author | Stephan Bergmann <sbergman@redhat.com> | 2015-10-19 17:49:28 +0200 |
---|---|---|
committer | Stephan Bergmann <sbergman@redhat.com> | 2015-10-19 17:49:28 +0200 |
commit | 4180210a27240e01c9a0db0688d51b1449e4c4b8 (patch) | |
tree | 0a3b7c356d3257296702847c7a0f63d6a948b03e /cui/source | |
parent | 25c047fec71050bcedc6c9d355f35486f615c3c4 (diff) |
loplugin:defaultparams
Change-Id: Ia4cd4faaffaf42555b16ada5e259e85ddb496236
Diffstat (limited to 'cui/source')
28 files changed, 82 insertions, 84 deletions
diff --git a/cui/source/customize/acccfg.cxx b/cui/source/customize/acccfg.cxx index 9fbdffd61c32..40cf3c12332b 100644 --- a/cui/source/customize/acccfg.cxx +++ b/cui/source/customize/acccfg.cxx @@ -1121,7 +1121,7 @@ IMPL_LINK_TYPED( SfxAcceleratorConfigPage, SelectHdl, SvTreeListBox*, pListBox, if (pEntry->m_bIsConfigurable) { if (pEntry->isConfigured()) - m_pRemoveButton->Enable( true ); + m_pRemoveButton->Enable(); m_pChangeButton->Enable( pEntry->m_sCommand != sPossibleNewCommand ); } } @@ -1147,7 +1147,7 @@ IMPL_LINK_TYPED( SfxAcceleratorConfigPage, SelectHdl, SvTreeListBox*, pListBox, if (pEntry->m_bIsConfigurable) { if (pEntry->isConfigured()) - m_pRemoveButton->Enable( true ); + m_pRemoveButton->Enable(); m_pChangeButton->Enable( pEntry->m_sCommand != sPossibleNewCommand ); } @@ -1160,7 +1160,7 @@ IMPL_LINK_TYPED( SfxAcceleratorConfigPage, SelectHdl, SvTreeListBox*, pListBox, if ( pUserData && pUserData->m_sCommand == sPossibleNewCommand ) { TAccInfo* pU1 = new TAccInfo(-1, -1, pUserData->m_aKey); - SvTreeListEntry* pE1 = m_pKeyBox->InsertEntry( pUserData->m_aKey.GetName(), 0L, true, TREELIST_APPEND ); + SvTreeListEntry* pE1 = m_pKeyBox->InsertEntry( pUserData->m_aKey.GetName(), 0L, true ); pE1->SetUserData(pU1); pE1->EnableChildrenOnDemand( false ); } diff --git a/cui/source/customize/cfg.cxx b/cui/source/customize/cfg.cxx index 0d68bfa8ad26..ea27a2a5e284 100644 --- a/cui/source/customize/cfg.cxx +++ b/cui/source/customize/cfg.cxx @@ -2882,8 +2882,8 @@ SvxToolbarConfigPage::SvxToolbarConfigPage(vcl::Window *pParent, const SfxItemSe m_pMoveDownButton->SetClickHdl ( LINK( this, SvxToolbarConfigPage, MoveHdl) ); // Always enable Up and Down buttons // added for issue i53677 by shizhoubo - m_pMoveDownButton->Enable( true ); - m_pMoveUpButton->Enable( true ); + m_pMoveDownButton->Enable(); + m_pMoveUpButton->Enable(); PopupMenu* pMenu = new PopupMenu( CUI_RES( MODIFY_TOOLBAR ) ); pMenu->SetMenuFlags( @@ -4322,9 +4322,9 @@ IMPL_LINK_NOARG_TYPED( SvxToolbarConfigPage, SelectToolbar, ListBox&, void ) return; } - m_pModifyTopLevelButton->Enable( true ); - m_pModifyCommandButton->Enable( true ); - m_pAddCommandsButton->Enable( true ); + m_pModifyTopLevelButton->Enable(); + m_pModifyCommandButton->Enable(); + m_pAddCommandsButton->Enable(); PopupMenu* pPopup = m_pModifyTopLevelButton->GetPopupMenu(); @@ -4967,7 +4967,7 @@ IMPL_LINK_TYPED( SvxIconSelectorDialog, SelectHdl, ToolBox *, pToolBox, void ) OUString aSelImageText = pTbSymbol->GetItemText( nId ); if ( m_xImportedImageManager->hasImage( GetImageType(), aSelImageText ) ) { - pBtnDelete->Enable( true ); + pBtnDelete->Enable(); } else { diff --git a/cui/source/customize/macropg.cxx b/cui/source/customize/macropg.cxx index b06a5323e64e..30dda46e85fa 100644 --- a/cui/source/customize/macropg.cxx +++ b/cui/source/customize/macropg.cxx @@ -745,7 +745,7 @@ void _SvxMacroTabPage::InitAndSetHandler( Reference< container::XNameReplace> xA mpImpl->pEventLB->GetListBox().SetEntryHeight( sal::static_int_cast< short >(nMinLineHeight) ); - mpImpl->pEventLB->Enable( true ); + mpImpl->pEventLB->Enable(); if(!m_xAppEvents.is()) { diff --git a/cui/source/customize/selector.cxx b/cui/source/customize/selector.cxx index 07a45a234f43..b46fd23810f5 100644 --- a/cui/source/customize/selector.cxx +++ b/cui/source/customize/selector.cxx @@ -968,7 +968,7 @@ SvxScriptSelectorDialog::UpdateUI() m_pCommands->GetHelpText(m_pCommands->FirstSelected()); m_pDescriptionText->SetText(sMessage.isEmpty() ? m_sDefaultDesc : sMessage); - m_pOKButton->Enable( true ); + m_pOKButton->Enable(); } else { diff --git a/cui/source/dialogs/SpellDialog.cxx b/cui/source/dialogs/SpellDialog.cxx index 9134302d55c1..b327639249e3 100644 --- a/cui/source/dialogs/SpellDialog.cxx +++ b/cui/source/dialogs/SpellDialog.cxx @@ -424,7 +424,7 @@ void SpellDialog::SpellContinue_Impl(bool bUseSavedSentence, bool bIgnoreCurrent sal_Int32 nIdx = 0; do { - aControls[nIdx]->Enable(true); + aControls[nIdx]->Enable(); } while(aControls[++nIdx]); diff --git a/cui/source/dialogs/colorpicker.cxx b/cui/source/dialogs/colorpicker.cxx index 5cb95937bbfd..420d12972f83 100644 --- a/cui/source/dialogs/colorpicker.cxx +++ b/cui/source/dialogs/colorpicker.cxx @@ -1102,7 +1102,7 @@ ColorPickerDialog::ColorPickerDialog( vcl::Window* pParent, sal_Int32 nColor, sa Image aSliderImage( maSliderImage ); mpFISliderLeft->SetImage( aSliderImage ); - mpFISliderLeft->Show(true); + mpFISliderLeft->Show(); BitmapEx aTmpBmp( maSliderImage.GetBitmapEx() ); aTmpBmp.Mirror( BmpMirrorFlags::Horizontal ); @@ -1128,7 +1128,7 @@ ColorPickerDialog::ColorPickerDialog( vcl::Window* pParent, sal_Int32 nColor, sa { mpColorPreview->SetSizePixel( mpColorPrevious->GetSizePixel() ); mpColorPrevious->SetColor( aColor ); - mpColorPrevious->Show( true ); + mpColorPrevious->Show(); } mdRed = ((double)aColor.GetRed()) / 255.0; diff --git a/cui/source/dialogs/cuifmsearch.cxx b/cui/source/dialogs/cuifmsearch.cxx index 268dbe98b054..cbca73e7d872 100644 --- a/cui/source/dialogs/cuifmsearch.cxx +++ b/cui/source/dialogs/cuifmsearch.cxx @@ -603,7 +603,7 @@ void FmSearchDialog::EnableSearchUI(bool bEnable) { // this means we're preparing for starting a search // In this case, EnableSearchForDependees disabled the search button // But as we're about to use it for cancelling the search, we really need to enable it, again - m_pbSearchAgain->Enable( true ); + m_pbSearchAgain->Enable(); } } diff --git a/cui/source/dialogs/hangulhanjadlg.cxx b/cui/source/dialogs/hangulhanjadlg.cxx index 1a8092bec5f5..91a97fd33262 100644 --- a/cui/source/dialogs/hangulhanjadlg.cxx +++ b/cui/source/dialogs/hangulhanjadlg.cxx @@ -851,9 +851,9 @@ namespace svx { // default state: try both direction m_pHangulOnly->Check( false ); - m_pHangulOnly->Enable( true ); + m_pHangulOnly->Enable(); m_pHanjaOnly->Check( false ); - m_pHanjaOnly->Enable( true ); + m_pHanjaOnly->Enable(); if (!_bTryBothDirections) { diff --git a/cui/source/dialogs/hlinettp.cxx b/cui/source/dialogs/hlinettp.cxx index f73027328ce3..f406ee35cf6a 100644 --- a/cui/source/dialogs/hlinettp.cxx +++ b/cui/source/dialogs/hlinettp.cxx @@ -66,7 +66,7 @@ SvxHyperlinkInternetTp::SvxHyperlinkInternetTp ( vcl::Window *pParent, // set defaults m_pRbtLinktypInternet->Check (); - m_pBtBrowse->Enable( true ); + m_pBtBrowse->Enable(); // set handlers diff --git a/cui/source/dialogs/hlmailtp.cxx b/cui/source/dialogs/hlmailtp.cxx index 567a47801ba4..d511cc5663fb 100644 --- a/cui/source/dialogs/hlmailtp.cxx +++ b/cui/source/dialogs/hlmailtp.cxx @@ -192,15 +192,13 @@ void SvxHyperlinkMailTp::SetInitFocus() void SvxHyperlinkMailTp::SetScheme(const OUString& rScheme) { - const bool bMail = true; - //update target: RemoveImproperProtocol(rScheme); m_pCbbReceiver->SetSmartProtocol( INetProtocol::Mailto ); //show/hide special fields for MAIL: - m_pBtAdrBook->Enable(bMail); - m_pEdSubject->Enable(bMail); + m_pBtAdrBook->Enable(); + m_pEdSubject->Enable(); } /************************************************************************* diff --git a/cui/source/dialogs/hltpbase.cxx b/cui/source/dialogs/hltpbase.cxx index 966947a98bff..69c94924a15d 100644 --- a/cui/source/dialogs/hltpbase.cxx +++ b/cui/source/dialogs/hltpbase.cxx @@ -379,7 +379,7 @@ IMPL_LINK_NOARG_TYPED(SvxHyperlinkTabPageBase, ClickScriptHdl_Impl, Button*, voi SFX_EVENT_MOUSEOUT_OBJECT); if ( bIsInputEnabled ) - GetParent()->EnableInput( true ); + GetParent()->EnableInput(); // execute dlg DisableClose( true ); short nRet = aDlg->Execute(); diff --git a/cui/source/dialogs/postdlg.cxx b/cui/source/dialogs/postdlg.cxx index a2ab2ec0efe2..7af081aa2d98 100644 --- a/cui/source/dialogs/postdlg.cxx +++ b/cui/source/dialogs/postdlg.cxx @@ -73,7 +73,7 @@ SvxPostItDialog::SvxPostItDialog(vcl::Window* pParent, const SfxItemSet& rCoreSe nWhich = rSet.GetPool()->GetWhich( SID_ATTR_POSTIT_AUTHOR ); OUString aAuthorStr, aDateStr; - if ( rSet.GetItemState( nWhich, true ) >= SfxItemState::DEFAULT ) + if ( rSet.GetItemState( nWhich ) >= SfxItemState::DEFAULT ) { bNew = false; const SvxPostItAuthorItem& rAuthor = @@ -85,7 +85,7 @@ SvxPostItDialog::SvxPostItDialog(vcl::Window* pParent, const SfxItemSet& rCoreSe nWhich = rSet.GetPool()->GetWhich( SID_ATTR_POSTIT_DATE ); - if ( rSet.GetItemState( nWhich, true ) >= SfxItemState::DEFAULT ) + if ( rSet.GetItemState( nWhich ) >= SfxItemState::DEFAULT ) { const SvxPostItDateItem& rDate = static_cast<const SvxPostItDateItem&>(rSet.Get( nWhich )); @@ -100,7 +100,7 @@ SvxPostItDialog::SvxPostItDialog(vcl::Window* pParent, const SfxItemSet& rCoreSe nWhich = rSet.GetPool()->GetWhich( SID_ATTR_POSTIT_TEXT ); OUString aTextStr; - if ( rSet.GetItemState( nWhich, true ) >= SfxItemState::DEFAULT ) + if ( rSet.GetItemState( nWhich ) >= SfxItemState::DEFAULT ) { const SvxPostItTextItem& rText = static_cast<const SvxPostItTextItem&>(rSet.Get( nWhich )); diff --git a/cui/source/options/optasian.cxx b/cui/source/options/optasian.cxx index 4f21f18fe5e6..260187a15184 100644 --- a/cui/source/options/optasian.cxx +++ b/cui/source/options/optasian.cxx @@ -146,7 +146,7 @@ SvxAsianLayoutPage::SvxAsianLayoutPage( vcl::Window* pParent, const SfxItemSet& m_pStartED->SetModifyHdl(aLk); m_pEndED->SetModifyHdl(aLk); - m_pLanguageLB->SetLanguageList( SvxLanguageListFlags::FBD_CHARS, false, false ); + m_pLanguageLB->SetLanguageList( SvxLanguageListFlags::FBD_CHARS, false ); } SvxAsianLayoutPage::~SvxAsianLayoutPage() diff --git a/cui/source/options/optchart.cxx b/cui/source/options/optchart.cxx index 555915e9743d..44624ddcecb0 100644 --- a/cui/source/options/optchart.cxx +++ b/cui/source/options/optchart.cxx @@ -190,7 +190,7 @@ IMPL_LINK_NOARG_TYPED(SvxDefaultColorOptPage, ResetToDefaults, Button*, void) m_pLbChartColors->GetFocus(); m_pLbChartColors->SelectEntryPos( 0 ); - m_pPBRemove->Enable( true ); + m_pPBRemove->Enable(); } } @@ -209,7 +209,7 @@ IMPL_LINK_NOARG_TYPED(SvxDefaultColorOptPage, AddChartColor, Button*, void) m_pLbChartColors->GetFocus(); m_pLbChartColors->SelectEntryPos( pColorConfig->GetColorList().size() - 1 ); - m_pPBRemove->Enable( true ); + m_pPBRemove->Enable(); } } @@ -242,7 +242,7 @@ IMPL_LINK_TYPED( SvxDefaultColorOptPage, RemoveChartColor, Button*, pButton, voi else if (m_pLbChartColors->GetEntryCount() > 0) m_pLbChartColors->SelectEntryPos( nIndex ); else - m_pPBRemove->Enable(true); + m_pPBRemove->Enable(); } } } diff --git a/cui/source/options/optgdlg.cxx b/cui/source/options/optgdlg.cxx index 6f14ea8c2814..0065db174704 100644 --- a/cui/source/options/optgdlg.cxx +++ b/cui/source/options/optgdlg.cxx @@ -1105,7 +1105,7 @@ OfaLanguagesTabPage::OfaLanguagesTabPage(vcl::Window* pParent, const SfxItemSet& m_pComplexLanguageLB->SetLanguageList( SvxLanguageListFlags::CTL | SvxLanguageListFlags::ONLY_KNOWN, true, false, true ); m_pComplexLanguageLB->InsertDefaultLanguage( css::i18n::ScriptType::COMPLEX ); - m_pLocaleSettingLB->SetLanguageList( SvxLanguageListFlags::ALL | SvxLanguageListFlags::ONLY_KNOWN, false, false); + m_pLocaleSettingLB->SetLanguageList( SvxLanguageListFlags::ALL | SvxLanguageListFlags::ONLY_KNOWN, false ); m_pLocaleSettingLB->InsertSystemLanguage( ); const NfCurrencyTable& rCurrTab = SvNumberFormatter::GetTheCurrencyTable(); @@ -1538,7 +1538,7 @@ void OfaLanguagesTabPage::Reset( const SfxItemSet* rSet ) //overwrite them by the values provided by the DocShell if(pCurrentDocShell) { - m_pCurrentDocCB->Enable(true); + m_pCurrentDocCB->Enable(); m_pCurrentDocCB->Check(bLanguageCurrentDoc_Impl); const SfxPoolItem* pLang; if( SfxItemState::SET == rSet->GetItemState(SID_ATTR_LANGUAGE, false, &pLang)) @@ -1603,7 +1603,7 @@ void OfaLanguagesTabPage::Reset( const SfxItemSet* rSet ) if ( SfxItemState::SET == rSet->GetItemState(SID_SET_DOCUMENT_LANGUAGE, false, &pLang ) && static_cast<const SfxBoolItem*>(pLang)->GetValue() ) { m_pWesternLanguageLB->GrabFocus(); - m_pCurrentDocCB->Enable(true); + m_pCurrentDocCB->Enable(); m_pCurrentDocCB->Check(); } } diff --git a/cui/source/options/optinet2.cxx b/cui/source/options/optinet2.cxx index 345468494451..c46e713fe1ed 100644 --- a/cui/source/options/optinet2.cxx +++ b/cui/source/options/optinet2.cxx @@ -684,11 +684,11 @@ IMPL_LINK_NOARG_TYPED(SvxSecurityTabPage, SavePasswordHdl, Button*, void) xMasterPasswd->removeMasterPassword(); if ( xMasterPasswd->changeMasterPassword( Reference< task::XInteractionHandler >() ) ) { - m_pMasterPasswordPB->Enable( true ); + m_pMasterPasswordPB->Enable(); m_pMasterPasswordCB->Check(); - m_pMasterPasswordCB->Enable( true ); - m_pMasterPasswordFT->Enable( true ); - m_pShowConnectionsPB->Enable( true ); + m_pMasterPasswordCB->Enable(); + m_pMasterPasswordFT->Enable(); + m_pShowConnectionsPB->Enable(); } else { @@ -713,8 +713,8 @@ IMPL_LINK_NOARG_TYPED(SvxSecurityTabPage, SavePasswordHdl, Button*, void) else { m_pSavePasswordsCB->Check(); - m_pMasterPasswordPB->Enable( true ); - m_pShowConnectionsPB->Enable( true ); + m_pMasterPasswordPB->Enable(); + m_pShowConnectionsPB->Enable(); } } } @@ -749,14 +749,14 @@ IMPL_LINK_NOARG_TYPED(SvxSecurityTabPage, MasterPasswordCBHdl, Button*, void) { if ( xMasterPasswd->isPersistentStoringAllowed() && xMasterPasswd->changeMasterPassword( Reference< task::XInteractionHandler >() ) ) { - m_pMasterPasswordPB->Enable( true ); - m_pMasterPasswordFT->Enable( true ); + m_pMasterPasswordPB->Enable(); + m_pMasterPasswordFT->Enable(); } else { m_pMasterPasswordCB->Check( false ); - m_pMasterPasswordPB->Enable( true ); - m_pMasterPasswordFT->Enable( true ); + m_pMasterPasswordPB->Enable(); + m_pMasterPasswordFT->Enable(); } } else @@ -769,8 +769,8 @@ IMPL_LINK_NOARG_TYPED(SvxSecurityTabPage, MasterPasswordCBHdl, Button*, void) else { m_pMasterPasswordCB->Check(); - m_pMasterPasswordPB->Enable( true ); - m_pShowConnectionsPB->Enable( true ); + m_pMasterPasswordPB->Enable(); + m_pShowConnectionsPB->Enable(); } } } @@ -872,17 +872,17 @@ void SvxSecurityTabPage::InitControls() if ( xMasterPasswd->isPersistentStoringAllowed() ) { - m_pMasterPasswordCB->Enable( true ); - m_pShowConnectionsPB->Enable( true ); + m_pMasterPasswordCB->Enable(); + m_pShowConnectionsPB->Enable(); m_pSavePasswordsCB->Check(); if ( xMasterPasswd->isDefaultMasterPasswordUsed() ) m_pMasterPasswordCB->Check( false ); else { - m_pMasterPasswordPB->Enable( true ); + m_pMasterPasswordPB->Enable(); m_pMasterPasswordCB->Check(); - m_pMasterPasswordFT->Enable( true ); + m_pMasterPasswordFT->Enable(); } } } @@ -1040,8 +1040,8 @@ bool SvxEMailTabPage::FillItemSet( SfxItemSet* ) void SvxEMailTabPage::Reset( const SfxItemSet* ) { - m_pMailerURLED->Enable(true ); - m_pMailerURLPB->Enable(true ); + m_pMailerURLED->Enable(); + m_pMailerURLPB->Enable(); if (pImpl->bROProgram) m_pMailerURLFI->Show(); diff --git a/cui/source/options/optlingu.cxx b/cui/source/options/optlingu.cxx index 814d5d8f8967..af8ce6356245 100644 --- a/cui/source/options/optlingu.cxx +++ b/cui/source/options/optlingu.cxx @@ -1346,7 +1346,7 @@ void SvxLinguTabPage::AddDicBoxEntry( OUString aTxt( ::GetDicInfoStr( rxDic->getName(), LanguageTag( rxDic->getLocale() ).getLanguageType(), DictionaryType_NEGATIVE == rxDic->getDictionaryType() ) ); - m_pLinguDicsCLB->InsertEntry( aTxt, TREELIST_APPEND ); // append at end + m_pLinguDicsCLB->InsertEntry( aTxt ); // append at end SvTreeListEntry* pEntry = m_pLinguDicsCLB->GetEntry( m_pLinguDicsCLB->GetEntryCount() - 1 ); DBG_ASSERT( pEntry, "failed to add entry" ); if (pEntry) @@ -1392,7 +1392,7 @@ void SvxLinguTabPage::UpdateModulesBox_Impl() for (sal_uLong i = 0; i < nDispSrvcCount; ++i) { const ServiceInfo_Impl &rInfo = rAllDispSrvcArr[i]; - m_pLinguModulesCLB->InsertEntry( rInfo.sDisplayName, TREELIST_APPEND ); + m_pLinguModulesCLB->InsertEntry( rInfo.sDisplayName ); SvTreeListEntry* pEntry = m_pLinguModulesCLB->GetEntry(i); pEntry->SetUserData( const_cast<ServiceInfo_Impl *>(&rInfo) ); m_pLinguModulesCLB->CheckEntryPos( i, rInfo.bConfigured ); @@ -1771,7 +1771,7 @@ IMPL_LINK_TYPED( SvxLinguTabPage, SelectHdl_Impl, SvTreeListBox*, pBox, void ) DicUserData aData( reinterpret_cast<sal_uLong>( pEntry->GetUserData() ) ); // always allow to edit (i.e. at least view the content of the dictionary) - m_pLinguDicsEditPB->Enable( true/*aData.IsEditable()*/ ); + m_pLinguDicsEditPB->Enable( /*aData.IsEditable()*/ ); m_pLinguDicsDelPB->Enable( aData.IsDeletable() ); } } diff --git a/cui/source/options/optupdt.cxx b/cui/source/options/optupdt.cxx index 741a2ecca292..06a9fe3b255a 100644 --- a/cui/source/options/optupdt.cxx +++ b/cui/source/options/optupdt.cxx @@ -256,9 +256,9 @@ void SvxOnlineUpdateTabPage::Reset( const SfxItemSet* ) m_xUpdateAccess->getByName( "AutoDownloadEnabled" ) >>= bValue; m_pAutoDownloadCheckBox->Check(bValue); - m_pDestPathLabel->Enable(true); - m_pDestPath->Enable(true); - m_pChangePathButton->Enable(true); + m_pDestPathLabel->Enable(); + m_pDestPath->Enable(); + m_pChangePathButton->Enable(); OUString sValue, aPath; m_xUpdateAccess->getByName( "DownloadDestination" ) >>= sValue; diff --git a/cui/source/options/webconninfo.cxx b/cui/source/options/webconninfo.cxx index d575601d3df7..c70479645811 100644 --- a/cui/source/options/webconninfo.cxx +++ b/cui/source/options/webconninfo.cxx @@ -304,7 +304,7 @@ IMPL_LINK_NOARG_TYPED(WebConnectionInfoDialog, EntrySelectedHdl, SvTreeListBox*, } else { - m_pRemoveBtn->Enable( true ); + m_pRemoveBtn->Enable(); // url container entries (-> use system credentials) have // no password diff --git a/cui/source/tabpages/autocdlg.cxx b/cui/source/tabpages/autocdlg.cxx index b8bab0c96328..4f744421b9a4 100644 --- a/cui/source/tabpages/autocdlg.cxx +++ b/cui/source/tabpages/autocdlg.cxx @@ -2633,7 +2633,7 @@ IMPL_LINK_NOARG_TYPED(OfaSmartTagOptionsTabPage, SelectHdl, SvTreeListBox*, void const lang::Locale aLocale( LanguageTag::convertToLocale( eLastDialogLanguage ) ); if ( xRec->hasPropertyPage( nSmartTagIdx, aLocale ) ) - m_pPropertiesPB->Enable( true ); + m_pPropertiesPB->Enable(); else m_pPropertiesPB->Enable( false ); } diff --git a/cui/source/tabpages/border.cxx b/cui/source/tabpages/border.cxx index 36ec95e18ded..3b432e17fe90 100644 --- a/cui/source/tabpages/border.cxx +++ b/cui/source/tabpages/border.cxx @@ -156,7 +156,7 @@ SvxBorderTabPage::SvxBorderTabPage(vcl::Window* pParent, const SfxItemSet& rCore /* Use SvxMarginItem instead of margins from SvxBoxItem, if present. -> Remember this state in mbUseMarginItem, because other special handling is needed across various functions... */ - mbUseMarginItem = rCoreAttrs.GetItemState(GetWhich(SID_ATTR_ALIGN_MARGIN),true) != SfxItemState::UNKNOWN; + mbUseMarginItem = rCoreAttrs.GetItemState(GetWhich(SID_ATTR_ALIGN_MARGIN)) != SfxItemState::UNKNOWN; const SfxPoolItem* pItem = NULL; if (rCoreAttrs.HasItem(SID_ATTR_BORDER_STYLES, &pItem)) @@ -217,7 +217,7 @@ SvxBorderTabPage::SvxBorderTabPage(vcl::Window* pParent, const SfxItemSet& rCore sal_uInt16 nWhich = GetWhich( SID_ATTR_BORDER_INNER, false ); bool bIsDontCare = true; - if ( rCoreAttrs.GetItemState( nWhich, true ) >= SfxItemState::DEFAULT ) + if ( rCoreAttrs.GetItemState( nWhich ) >= SfxItemState::DEFAULT ) { // paragraph or table const SvxBoxInfoItem* pBoxInfo = @@ -445,7 +445,7 @@ void SvxBorderTabPage::Reset( const SfxItemSet* rSet ) if ( pBoxInfoItem->IsDist() ) { - if( rSet->GetItemState( nWhichBox, true ) >= SfxItemState::DEFAULT ) + if( rSet->GetItemState( nWhichBox ) >= SfxItemState::DEFAULT ) { bool bIsAnyBorderVisible = m_pFrameSel->IsAnyBorderVisible(); if( !bIsAnyBorderVisible || !pBoxInfoItem->IsMinDist() ) diff --git a/cui/source/tabpages/chardlg.cxx b/cui/source/tabpages/chardlg.cxx index 3f0f0043b2b1..e048957047a8 100644 --- a/cui/source/tabpages/chardlg.cxx +++ b/cui/source/tabpages/chardlg.cxx @@ -352,7 +352,7 @@ SvxCharNamePage::SvxCharNamePage( vcl::Window* pParent, const SfxItemSet& rInSet m_pEastFontStyleFT->SetText(sFontStyleString); m_pCTLFontStyleFT->SetText(sFontStyleString); - m_pWestFrame->Show(true); + m_pWestFrame->Show(); m_pEastFrame->Show(bShowCJK); m_pCTLFrame->Show(bShowCTL); @@ -2281,7 +2281,7 @@ bool SvxCharEffectsPage::FillItemSet( SfxItemSet* rSet ) //! changing the underline style will be allowed if a style is //! selected in the listbox. bool bAllowChg = LISTBOX_ENTRY_NOTFOUND != nPos && - SfxItemState::DEFAULT > rOldSet.GetItemState( nWhich, true ); + SfxItemState::DEFAULT > rOldSet.GetItemState( nWhich ); const SvxUnderlineItem& rItem = *static_cast<const SvxUnderlineItem*>(pOld); if ( (FontUnderline)rItem.GetValue() == eUnder && @@ -2315,7 +2315,7 @@ bool SvxCharEffectsPage::FillItemSet( SfxItemSet* rSet ) //! changing the underline style will be allowed if a style is //! selected in the listbox. bool bAllowChg = LISTBOX_ENTRY_NOTFOUND != nPos && - SfxItemState::DEFAULT > rOldSet.GetItemState( nWhich, true ); + SfxItemState::DEFAULT > rOldSet.GetItemState( nWhich ); const SvxOverlineItem& rItem = *static_cast<const SvxOverlineItem*>(pOld); if ( (FontUnderline)rItem.GetValue() == eOver && @@ -2349,7 +2349,7 @@ bool SvxCharEffectsPage::FillItemSet( SfxItemSet* rSet ) //! changing the strikeout style will be allowed if a style is //! selected in the listbox. bool bAllowChg = LISTBOX_ENTRY_NOTFOUND != nPos && - SfxItemState::DEFAULT > rOldSet.GetItemState( nWhich, true ); + SfxItemState::DEFAULT > rOldSet.GetItemState( nWhich ); const SvxCrossedOutItem& rItem = *static_cast<const SvxCrossedOutItem*>(pOld); if ( !m_pStrikeoutLB->IsEnabled() @@ -2447,7 +2447,7 @@ bool SvxCharEffectsPage::FillItemSet( SfxItemSet* rSet ) //! changing the effect style will be allowed if a style is //! selected in the listbox. bool bAllowChg = LISTBOX_ENTRY_NOTFOUND != nPos && - SfxItemState::DEFAULT > rOldSet.GetItemState( nWhich, true ); + SfxItemState::DEFAULT > rOldSet.GetItemState( nWhich ); const SvxCaseMapItem& rItem = *static_cast<const SvxCaseMapItem*>(pOld); if ( (SvxCaseMap)rItem.GetValue() == eCaseMap && !bAllowChg ) diff --git a/cui/source/tabpages/macroass.cxx b/cui/source/tabpages/macroass.cxx index 0379d687e96b..6c3a2480ad6f 100644 --- a/cui/source/tabpages/macroass.cxx +++ b/cui/source/tabpages/macroass.cxx @@ -363,7 +363,7 @@ IMPL_LINK_TYPED( _SfxMacroTabPage, TimeOut_Impl, Idle*,, void ) FillMacroList(); if ( pTabDlg ) { - pTabDlg->EnableInput( true ); + pTabDlg->EnableInput(); pTabDlg->LeaveWait(); } } @@ -393,9 +393,9 @@ void _SfxMacroTabPage::InitAndSetHandler() mpImpl->pEventLB->Show(); mpImpl->pEventLB->ConnectElements(); - mpImpl->pEventLB->Enable( true ); - mpImpl->pGroupLB->Enable( true ); - mpImpl->pMacroLB->Enable( true ); + mpImpl->pEventLB->Enable(); + mpImpl->pGroupLB->Enable(); + mpImpl->pMacroLB->Enable(); mpImpl->pGroupLB->SetFunctionListBox( mpImpl->pMacroLB ); diff --git a/cui/source/tabpages/numfmt.cxx b/cui/source/tabpages/numfmt.cxx index cdc03e30e096..9f22d4c43aed 100644 --- a/cui/source/tabpages/numfmt.cxx +++ b/cui/source/tabpages/numfmt.cxx @@ -1274,8 +1274,8 @@ void SvxNumberFormatTabPage::SelFormatHdl_Impl(void * pLb ) } else { - m_pIbAdd->Enable(true ); - m_pIbInfo->Enable(true ); + m_pIbAdd->Enable(); + m_pIbInfo->Enable(); m_pIbRemove->Enable(false ); m_pFtComment->SetText(m_pEdComment->GetText()); @@ -1541,8 +1541,8 @@ void SvxNumberFormatTabPage::EditHdl_Impl( Edit* pEdFormat ) else { - m_pIbAdd->Enable(true ); - m_pIbInfo->Enable(true); + m_pIbAdd->Enable(); + m_pIbInfo->Enable(); m_pIbRemove->Enable(false ); m_pFtComment->SetText(m_pEdComment->GetText()); diff --git a/cui/source/tabpages/numpages.cxx b/cui/source/tabpages/numpages.cxx index 094f6221045d..d04334effe60 100644 --- a/cui/source/tabpages/numpages.cxx +++ b/cui/source/tabpages/numpages.cxx @@ -3038,8 +3038,8 @@ void SvxNumPositionTabPage::InitControls() if ( aNumFmtArr[nLvl]->GetLabelFollowedBy() == SvxNumberFormat::LISTTAB ) { - m_pListtabFT->Enable( true ); - m_pListtabMF->Enable( true ); + m_pListtabFT->Enable(); + m_pListtabMF->Enable(); if ( bSameListtab ) { SetMetricValue(*m_pListtabMF, aNumFmtArr[nLvl]->GetListtabPos(), eCoreUnit); diff --git a/cui/source/tabpages/paragrph.cxx b/cui/source/tabpages/paragrph.cxx index 63f10bb4d408..ecbc0ca42c9b 100644 --- a/cui/source/tabpages/paragrph.cxx +++ b/cui/source/tabpages/paragrph.cxx @@ -2250,7 +2250,7 @@ bool SvxAsianTabPage::FillItemSet( SfxItemSet* rSet ) static void lcl_SetBox(const SfxItemSet& rSet, sal_uInt16 nSlotId, CheckBox& rBox) { sal_uInt16 _nWhich = rSet.GetPool()->GetWhich(nSlotId); - SfxItemState eState = rSet.GetItemState(_nWhich, true); + SfxItemState eState = rSet.GetItemState(_nWhich); if( eState == SfxItemState::UNKNOWN || eState == SfxItemState::DISABLED ) rBox.Enable(false); else if(eState >= SfxItemState::DEFAULT) diff --git a/cui/source/tabpages/tabstpge.cxx b/cui/source/tabpages/tabstpge.cxx index 4603f6b1e777..25ede825288f 100644 --- a/cui/source/tabpages/tabstpge.cxx +++ b/cui/source/tabpages/tabstpge.cxx @@ -115,8 +115,8 @@ SvxTabulatorTabPage::SvxTabulatorTabPage(vcl::Window* pParent, const SfxItemSet& SvtCJKOptions aCJKOptions; get(m_pLeftTab, aCJKOptions.IsAsianTypographyEnabled() ? "radiobuttonST_LEFTTAB_ASIAN" : "radiobuttonBTN_TABTYPE_LEFT"); get(m_pRightTab, aCJKOptions.IsAsianTypographyEnabled() ? "radiobuttonST_RIGHTTAB_ASIAN" : "radiobuttonBTN_TABTYPE_RIGHT"); - m_pLeftTab->Show(true); - m_pRightTab->Show(true); + m_pLeftTab->Show(); + m_pRightTab->Show(); get(m_pCenterTab,"radiobuttonBTN_TABTYPE_CENTER"); get(m_pDezTab,"radiobuttonBTN_TABTYPE_DECIMAL"); get(m_pDezChar,"entryED_TABTYPE_DECCHAR"); diff --git a/cui/source/tabpages/tparea.cxx b/cui/source/tabpages/tparea.cxx index d35ef405523d..d1d4f6c62cd3 100644 --- a/cui/source/tabpages/tparea.cxx +++ b/cui/source/tabpages/tparea.cxx @@ -1094,7 +1094,7 @@ bool SvxAreaTabPage::FillItemSet( SfxItemSet* rAttrs ) // NEW if( (eSavedStyle != eStyle) && ( bModified || - SfxItemState::SET == m_rOutAttrs.GetItemState( GetWhich( XATTR_FILLCOLOR ), true ) ) ) + SfxItemState::SET == m_rOutAttrs.GetItemState( GetWhich( XATTR_FILLCOLOR ) ) ) ) { XFillStyleItem aStyleItem( drawing::FillStyle_SOLID ); pOld = GetOldItem( *rAttrs, XATTR_FILLSTYLE ); @@ -1125,7 +1125,7 @@ bool SvxAreaTabPage::FillItemSet( SfxItemSet* rAttrs ) // NEW if( (eSavedStyle != eStyle) && ( bModified || - SfxItemState::SET == m_rOutAttrs.GetItemState( GetWhich( XATTR_FILLGRADIENT ), true ) ) ) + SfxItemState::SET == m_rOutAttrs.GetItemState( GetWhich( XATTR_FILLGRADIENT ) ) ) ) { XFillStyleItem aStyleItem( drawing::FillStyle_GRADIENT ); pOld = GetOldItem( *rAttrs, XATTR_FILLSTYLE ); @@ -1171,7 +1171,7 @@ bool SvxAreaTabPage::FillItemSet( SfxItemSet* rAttrs ) // NEW if( (eSavedStyle != eStyle) && ( bModified || - SfxItemState::SET == m_rOutAttrs.GetItemState( GetWhich( XATTR_FILLHATCH ), true ) ) ) + SfxItemState::SET == m_rOutAttrs.GetItemState( GetWhich( XATTR_FILLHATCH ) ) ) ) { XFillStyleItem aStyleItem( drawing::FillStyle_HATCH ); pOld = GetOldItem( *rAttrs, XATTR_FILLSTYLE ); @@ -1212,7 +1212,7 @@ bool SvxAreaTabPage::FillItemSet( SfxItemSet* rAttrs ) // NEW if( (eSavedStyle != eStyle) && ( bModified || - SfxItemState::SET == m_rOutAttrs.GetItemState( GetWhich( XATTR_FILLBITMAP ), true ) ) ) + SfxItemState::SET == m_rOutAttrs.GetItemState( GetWhich( XATTR_FILLBITMAP ) ) ) ) { XFillStyleItem aStyleItem( drawing::FillStyle_BITMAP ); pOld = GetOldItem( *rAttrs, XATTR_FILLSTYLE ); |