cui/source/customize/cfg.cxx | 11 --- cui/source/dialogs/about.cxx | 10 -- cui/source/dialogs/cuigaldlg.cxx | 12 --- cui/source/dialogs/hlmailtp.cxx | 3 cui/source/dialogs/linkdlg.cxx | 8 -- cui/source/dialogs/postdlg.cxx | 13 --- cui/source/dialogs/scriptdlg.cxx | 20 +---- cui/source/dialogs/thesdlg.cxx | 4 - cui/source/options/dbregister.cxx | 4 - cui/source/options/fontsubs.cxx | 4 - cui/source/options/optdict.cxx | 12 +-- cui/source/options/personalization.cxx | 3 cui/source/tabpages/autocdlg.cxx | 16 +--- cui/source/tabpages/grfpage.cxx | 6 - cui/source/tabpages/macroass.cxx | 9 -- cui/source/tabpages/numpages.cxx | 22 ++---- cui/source/tabpages/page.cxx | 4 - cui/source/tabpages/tpgradnt.cxx | 3 cui/source/tabpages/tphatch.cxx | 3 cui/source/tabpages/tplnedef.cxx | 4 - cui/source/tabpages/tplneend.cxx | 4 - cui/source/tabpages/tppattern.cxx | 7 -- dbaccess/source/core/api/KeySet.cxx | 5 - dbaccess/source/core/dataaccess/documentcontainer.cxx | 3 dbaccess/source/filter/xml/xmlFileBasedDatabase.cxx | 3 dbaccess/source/ui/dlg/DBSetupConnectionPages.cxx | 3 dbaccess/source/ui/dlg/directsql.cxx | 6 - dbaccess/source/ui/dlg/indexdialog.cxx | 3 dbaccess/source/ui/dlg/sqlmessage.cxx | 10 -- dbaccess/source/ui/dlg/tablespage.cxx | 9 -- dbaccess/source/ui/misc/WNameMatch.cxx | 12 +-- dbaccess/source/ui/querydesign/QueryDesignView.cxx | 44 +++---------- dbaccess/source/ui/querydesign/SelectionBrowseBox.cxx | 9 -- dbaccess/source/ui/relationdesign/RTableConnectionData.cxx | 3 dbaccess/source/ui/tabledesign/TableController.cxx | 3 35 files changed, 85 insertions(+), 210 deletions(-)
New commits: commit bd4c80b856a145563ba38242705e87028b1e0ed6 Author: nadith <nadmali...@gmail.com> Date: Mon Aug 1 14:51:58 2016 +0530 tdf#100726: Improve readability of OUString concatenation module cui and dbaccess fixed Change-Id: Ief6fca46622fe4e0da804c9a874f7ec93a0fd7af Reviewed-on: https://gerrit.libreoffice.org/27770 Tested-by: Jenkins <c...@libreoffice.org> Reviewed-by: Noel Grandin <noelgran...@gmail.com> diff --git a/cui/source/customize/cfg.cxx b/cui/source/customize/cfg.cxx index ada85af..a97db75 100644 --- a/cui/source/customize/cfg.cxx +++ b/cui/source/customize/cfg.cxx @@ -337,9 +337,7 @@ generateCustomMenuURL( SvxEntries* entries, sal_Int32 suffix = 1 ) { - OUString url(CUSTOM_MENU_STR ); - url += OUString::number( suffix ); - + OUString url = CUSTOM_MENU_STR + OUString::number( suffix ); if (!entries) return url; @@ -2163,9 +2161,7 @@ void SvxConfigPage::AddSubMenusToUI( if ( pEntryData->IsPopup() ) { - OUString subMenuTitle( rBaseTitle ); - subMenuTitle += aMenuSeparatorStr; - subMenuTitle += stripHotKey( pEntryData->GetName() ); + OUString subMenuTitle = rBaseTitle + aMenuSeparatorStr + stripHotKey( pEntryData->GetName() ); const sal_Int32 nPos = m_pTopLevelListBox->InsertEntry( subMenuTitle ); m_pTopLevelListBox->SetEntryData( nPos, pEntryData ); @@ -5429,8 +5425,7 @@ void SvxIconSelectorDialog::ImportGraphics( fPath = rPaths[0].copy(8) + "/"; for ( sal_Int32 i = 0; i < rejectedCount; ++i ) { - message += fPath + rejected[i]; - message += newLine; + message += fPath + rejected[i] + newLine; } ScopedVclPtrInstance< SvxIconChangeDialog > aDialog(this, message); diff --git a/cui/source/dialogs/about.cxx b/cui/source/dialogs/about.cxx index 22bb659..6826c51 100644 --- a/cui/source/dialogs/about.cxx +++ b/cui/source/dialogs/about.cxx @@ -287,8 +287,7 @@ OUString AboutDialog::GetVersionString() sVersion += m_sBuildStr.replaceAll("$BUILDID", sBuildId); } - sVersion += "\n"; - sVersion += Application::GetHWOSConfInfo(); + sVersion += "\n" + Application::GetHWOSConfInfo(); if (EXTRA_BUILDID[0] != '\0') { @@ -324,11 +323,8 @@ OUString AboutDialog::GetVersionString() OUString AboutDialog::GetCopyrightString() { - OUString aCopyrightString = m_aVendorTextStr; - aCopyrightString += "\n"; - - aCopyrightString += m_aCopyrightTextStr; - aCopyrightString += "\n"; + OUString aCopyrightString = m_aVendorTextStr + "\n" + + m_aCopyrightTextStr + "\n"; if (utl::ConfigManager::getProductName() == "LibreOffice") aCopyrightString += m_aBasedTextStr; diff --git a/cui/source/dialogs/cuigaldlg.cxx b/cui/source/dialogs/cuigaldlg.cxx index 9e6c715..f12d42b 100644 --- a/cui/source/dialogs/cuigaldlg.cxx +++ b/cui/source/dialogs/cuigaldlg.cxx @@ -572,9 +572,7 @@ IMPL_LINK_NOARG_TYPED(GalleryIdDialog, ClickOkHdl, Button*, void) { OUString aStr( CUI_RES( RID_SVXSTR_GALLERY_ID_EXISTS ) ); - aStr += " ("; - aStr += pInfo->GetThemeName(); - aStr += ")"; + aStr += " (" + pInfo->GetThemeName() + ")"; ScopedVclPtrInstance< InfoBox > aBox( this, aStr ); aBox->Execute(); @@ -602,9 +600,7 @@ GalleryThemeProperties::GalleryThemeProperties(vcl::Window* pParent, if( pData->pTheme->IsReadOnly() ) RemoveTabPage(m_nFilesPageId); - OUString aText( GetText() ); - - aText += pData->pTheme->GetName(); + OUString aText = GetText() + pData->pTheme->GetName(); if( pData->pTheme->IsReadOnly() ) aText += CUI_RES( RID_SVXSTR_GALLERY_READONLY ); @@ -831,9 +827,7 @@ OUString TPGalleryThemeProperties::addExtension( const OUString& _rDisplayText, if ( sRet.indexOf( sAllFilter ) == -1 ) { - sRet += sOpenBracket; - sRet += _rExtension; - sRet += sCloseBracket; + sRet += sOpenBracket + _rExtension + sCloseBracket; } return sRet; } diff --git a/cui/source/dialogs/hlmailtp.cxx b/cui/source/dialogs/hlmailtp.cxx index cb40ed8..1d8387b 100644 --- a/cui/source/dialogs/hlmailtp.cxx +++ b/cui/source/dialogs/hlmailtp.cxx @@ -155,8 +155,7 @@ OUString SvxHyperlinkMailTp::CreateAbsoluteURL() const { if ( !m_pEdSubject->GetText().isEmpty() ) { - OUString aQuery("subject="); - aQuery += m_pEdSubject->GetText(); + OUString aQuery = "subject=" + m_pEdSubject->GetText(); aURL.SetParam(aQuery); } } diff --git a/cui/source/dialogs/linkdlg.cxx b/cui/source/dialogs/linkdlg.cxx index 75a9cda..48e0960 100644 --- a/cui/source/dialogs/linkdlg.cxx +++ b/cui/source/dialogs/linkdlg.cxx @@ -639,16 +639,12 @@ void SvBaseLinksDlg::InsertEntry( const SvBaseLink& rLink, sal_uLong nPos, bool // filename not in string aTxt = aFileName; - aEntry = aTxt; - aEntry += "\t"; + aEntry = aTxt + "\t"; if( OBJECT_CLIENT_GRF == rLink.GetObjType() ) aEntry += sFilter; else aEntry += sLinkNm; - aEntry += "\t"; - aEntry += sTypeNm; - aEntry += "\t"; - aEntry += ImplGetStateStr( rLink ); + aEntry += "\t" + sTypeNm + "\t" + ImplGetStateStr( rLink ); SvTreeListEntry * pE = m_pTbLinks->InsertEntryToColumn( aEntry, nPos ); pE->SetUserData( const_cast<SvBaseLink *>(&rLink) ); diff --git a/cui/source/dialogs/postdlg.cxx b/cui/source/dialogs/postdlg.cxx index d5c1f44..35c3500 100644 --- a/cui/source/dialogs/postdlg.cxx +++ b/cui/source/dialogs/postdlg.cxx @@ -142,9 +142,7 @@ void SvxPostItDialog::dispose() void SvxPostItDialog::ShowLastAuthor(const OUString& rAuthor, const OUString& rDate) { - OUString sTxt( rAuthor ); - sTxt += ", "; - sTxt += rDate; + OUString sTxt = rAuthor + ", " + rDate; m_pLastEditFT->SetText( sTxt ); } @@ -189,14 +187,9 @@ IMPL_LINK_NOARG_TYPED(SvxPostItDialog, Stamp, Button*, void) if ( !aTmp.isEmpty() ) { - aStr += aTmp; - aStr += ", "; + aStr += aTmp + ", "; } - aStr += rLocaleWrapper.getDate(aDate); - aStr += ", "; - aStr += rLocaleWrapper.getTime(aTime, false); - aStr += " ----\n"; - + aStr += rLocaleWrapper.getDate(aDate) + ", " + rLocaleWrapper.getTime(aTime, false) + " ----\n"; aStr = convertLineEnd(aStr, GetSystemLineEnd()); m_pEditED->SetText(aStr); diff --git a/cui/source/dialogs/scriptdlg.cxx b/cui/source/dialogs/scriptdlg.cxx index c68e5d1..95e4fbc3 100644 --- a/cui/source/dialogs/scriptdlg.cxx +++ b/cui/source/dialogs/scriptdlg.cxx @@ -906,8 +906,7 @@ void SvxScriptOrgDialog::createEntry( SvTreeListEntry* pEntry ) if (aUserSuppliedName+extn == childNodes[index]->getName()) { bValid = false; - OUString aError( m_createErrStr ); - aError += m_createDupStr; + OUString aError = m_createErrStr + m_createDupStr; ScopedVclPtrInstance< MessageDialog > aErrorBox(static_cast<vcl::Window*>(this), aError); aErrorBox->SetText( m_createErrTitleStr ); aErrorBox->Execute(); @@ -1241,18 +1240,12 @@ OUString FormatErrorString( if ( !type.isEmpty() ) { - result += "\n\n" + - OUString(CUI_RES(RID_SVXSTR_ERROR_TYPE_LABEL)) + - " " + - type; + result += "\n\n" + OUString(CUI_RES(RID_SVXSTR_ERROR_TYPE_LABEL)) + " " + type; } if ( !message.isEmpty() ) { - result += "\n\n" + - OUString(CUI_RES(RID_SVXSTR_ERROR_MESSAGE_LABEL)) + - " " + - message; + result += "\n\n" + OUString(CUI_RES(RID_SVXSTR_ERROR_MESSAGE_LABEL)) + " " + message; } return result; @@ -1380,8 +1373,7 @@ OUString GetErrorMessage( OUString GetErrorMessage( const RuntimeException& re ) { Type t = cppu::UnoType<decltype(re)>::get(); - OUString message = t.getTypeName(); - message += re.Message; + OUString message = t.getTypeName() + re.Message; return message; } @@ -1389,9 +1381,7 @@ OUString GetErrorMessage( const RuntimeException& re ) OUString GetErrorMessage( const Exception& e ) { Type t = cppu::UnoType<decltype(e)>::get(); - OUString message = t.getTypeName(); - message += e.Message; - + OUString message = t.getTypeName() + e.Message; return message; } diff --git a/cui/source/dialogs/thesdlg.cxx b/cui/source/dialogs/thesdlg.cxx index ba03f6c..511ff0c 100644 --- a/cui/source/dialogs/thesdlg.cxx +++ b/cui/source/dialogs/thesdlg.cxx @@ -546,9 +546,7 @@ void SvxThesaurusDialog::SetWindowTitle( LanguageType nLanguage ) sal_Int32 nIndex = aStr.indexOf( '(' ); if( nIndex != -1 ) aStr = aStr.copy( 0, nIndex - 1 ); - aStr += " ("; - aStr += SvtLanguageTable::GetLanguageString( nLanguage ); - aStr += ")"; + aStr += " (" + SvtLanguageTable::GetLanguageString( nLanguage ) + ")"; SetText( aStr ); // set window title } diff --git a/cui/source/options/dbregister.cxx b/cui/source/options/dbregister.cxx index 14420b0..a8cd6ea 100644 --- a/cui/source/options/dbregister.cxx +++ b/cui/source/options/dbregister.cxx @@ -387,9 +387,7 @@ IMPL_LINK_NOARG_TYPED(DbRegistrationOptionsPage, PathSelect_Impl, SvTreeListBox* void DbRegistrationOptionsPage::insertNewEntry( const OUString& _sName,const OUString& _sLocation, const bool _bReadOnly ) { - OUString aStr( _sName ); - aStr += "\t"; - aStr += _sLocation; + OUString aStr = _sName + "\t" + _sLocation; SvTreeListEntry* pEntry = nullptr; if ( _bReadOnly ) diff --git a/cui/source/options/fontsubs.cxx b/cui/source/options/fontsubs.cxx index 8c0a638..4580275 100644 --- a/cui/source/options/fontsubs.cxx +++ b/cui/source/options/fontsubs.cxx @@ -371,9 +371,7 @@ void SvxFontSubstTabPage::CheckEnable() SvTreeListEntry* pEntry = m_pCheckLB->FirstSelected(); - OUString sEntry = m_pFont1CB->GetText(); - sEntry += "\t"; - sEntry += m_pFont2CB->GetText(); + OUString sEntry = m_pFont1CB->GetText() + "\t" + m_pFont2CB->GetText(); // because of OS/2 optimization error (Bug #56267) a bit more intricate: if (m_pFont1CB->GetText().isEmpty() || m_pFont2CB->GetText().isEmpty()) diff --git a/cui/source/options/optdict.cxx b/cui/source/options/optdict.cxx index 40ab495..b3db8c2 100644 --- a/cui/source/options/optdict.cxx +++ b/cui/source/options/optdict.cxx @@ -131,9 +131,9 @@ void SvxNewDictionaryDialog::dispose() IMPL_LINK_NOARG_TYPED(SvxNewDictionaryDialog, OKHdl_Impl, Button*, void) { - OUString sDict = comphelper::string::stripEnd(pNameEdit->GetText(), ' '); - // add extension for personal dictionaries - sDict += ".dic"; + + // add extension for personal dictionaries + OUString sDict = comphelper::string::stripEnd(pNameEdit->GetText(), ' ') + ".dic"; Reference< XSearchableDictionaryList > xDicList( LinguMgr::GetDictionaryList() ); @@ -532,8 +532,7 @@ void SvxEditDictionaryDialog::ShowWords_Impl( sal_uInt16 nId ) sal_uLong nPos = GetLBInsertPos( aStr ); if(pEntry[i]->isNegative()) { - aStr += "\t"; - aStr += pEntry[i]->getReplacementText(); + aStr += "\t" + pEntry[i]->getReplacementText(); } pWordsLB->InsertEntry(aStr, nullptr, false, nPos == TREELIST_ENTRY_NOTFOUND ? TREELIST_APPEND : nPos); } @@ -643,8 +642,7 @@ bool SvxEditDictionaryDialog::NewDelHdl(void* pBtn) if(pReplaceFT->IsVisible()) { - sEntry += "\t"; - sEntry += aReplaceStr; + sEntry += "\t" + aReplaceStr; } SvTreeListEntry* pNewEntry = nullptr; diff --git a/cui/source/options/personalization.cxx b/cui/source/options/personalization.cxx index ad275ef..b570c0b 100644 --- a/cui/source/options/personalization.cxx +++ b/cui/source/options/personalization.cxx @@ -656,8 +656,7 @@ bool getPreviewFile( const OUString& rURL, OUString *pPreviewFile, OUString *pPe // copy the images to the user's gallery OUString gallery = "${$BRAND_BASE_DIR/" LIBO_ETC_FOLDER "/" SAL_CONFIGFILE( "bootstrap") "::UserInstallation}"; rtl::Bootstrap::expandMacros( gallery ); - gallery += "/user/gallery/personas/"; - gallery += aName + "/"; + gallery += "/user/gallery/personas/" + aName + "/"; osl::Directory::createPath( gallery ); OUString aPreviewFile( INetURLObject( aPreviewURL ).getName() ); diff --git a/cui/source/tabpages/autocdlg.cxx b/cui/source/tabpages/autocdlg.cxx index f442306..f19be92 100644 --- a/cui/source/tabpages/autocdlg.cxx +++ b/cui/source/tabpages/autocdlg.cxx @@ -1036,9 +1036,7 @@ void OfaAutocorrReplacePage::RefillReplaceBox(bool bFromReset, // formatted text is only in Writer if(bSWriter || bTextOnly) { - OUString sEntry(rDouble.sShort); - sEntry += "\t"; - sEntry += rDouble.sLong; + OUString sEntry = rDouble.sShort + "\t" + rDouble.sLong; SvTreeListEntry* pEntry = m_pReplaceTLB->InsertEntry(sEntry); m_pTextOnlyCB->Check(bTextOnly); if(!bTextOnly) @@ -1064,9 +1062,7 @@ void OfaAutocorrReplacePage::RefillReplaceBox(bool bFromReset, // formatted text is only in Writer if(bSWriter || bTextOnly) { - OUString sEntry(pWordPtr->GetShort()); - sEntry += "\t"; - sEntry += pWordPtr->GetLong(); + OUString sEntry = pWordPtr->GetShort() + "\t" + pWordPtr->GetLong(); SvTreeListEntry* pEntry = m_pReplaceTLB->InsertEntry(sEntry); m_pTextOnlyCB->Check(pWordPtr->IsTextOnly()); if(!bTextOnly) @@ -1246,8 +1242,7 @@ bool OfaAutocorrReplacePage::NewDelHdl(void* pBtn) NewEntry(m_pShortED->GetText(), m_pReplaceED->GetText(), bKeepSourceFormatting); m_pReplaceTLB->SetUpdateMode(false); sal_uLong nPos = TREELIST_ENTRY_NOTFOUND; - sEntry += "\t"; - sEntry += m_pReplaceED->GetText(); + sEntry += "\t" + m_pReplaceED->GetText(); if(_pNewEntry) { nPos = m_pReplaceTLB->GetModel()->GetAbsPos(_pNewEntry); @@ -1867,10 +1862,7 @@ OfaQuoteTabPage::OfaQuoteTabPage(vcl::Window* pParent, const SfxItemSet& rSet) m_pSwCheckLB->SetStyle(m_pSwCheckLB->GetStyle() | WB_HSCROLL| WB_VSCROLL); m_pSwCheckLB->SvSimpleTable::SetTabs(aStaticTabs); - OUString sHeader(get<vcl::Window>("m")->GetText()); - sHeader += "\t"; - sHeader += get<vcl::Window>("t")->GetText(); - sHeader += "\t"; + OUString sHeader = get<vcl::Window>("m")->GetText() + "\t" + get<vcl::Window>("t")->GetText() + "\t"; m_pSwCheckLB->InsertHeaderEntry( sHeader, HEADERBAR_APPEND, HeaderBarItemBits::CENTER | HeaderBarItemBits::VCENTER | HeaderBarItemBits::FIXEDPOS | HeaderBarItemBits::FIXED); m_pCheckLB->Hide(); diff --git a/cui/source/tabpages/grfpage.cxx b/cui/source/tabpages/grfpage.cxx index 4161199..6eefbce 100644 --- a/cui/source/tabpages/grfpage.cxx +++ b/cui/source/tabpages/grfpage.cxx @@ -699,8 +699,7 @@ void SvxGrfCropPage::GraphicHasChanged( bool bFound ) OUString sTemp = aFld->GetText(); aFld->SetValue( aFld->Normalize( aOrigSize.Height() ), eUnit ); // multiplication sign (U+00D7) - sTemp += OUString( sal_Unicode (0x00D7) ); - sTemp += aFld->GetText(); + sTemp += OUString( sal_Unicode (0x00D7) ) + aFld->GetText(); if ( aOrigPixelSize.Width() && aOrigPixelSize.Height() ) { sal_Int32 ax = sal_Int32(floor((float)aOrigPixelSize.Width() / @@ -711,8 +710,7 @@ void SvxGrfCropPage::GraphicHasChanged( bool bFound ) sTemp += CUI_RESSTR( RID_SVXSTR_PPI ); OUString sPPI = OUString::number(ax); if (abs(ax - ay) > 1) { - sPPI += OUString( sal_Unicode (0x00D7) ); - sPPI += OUString::number(ay); + sPPI += OUString( sal_Unicode (0x00D7) ) + OUString::number(ay); } sTemp = sTemp.replaceAll("%1", sPPI); } diff --git a/cui/source/tabpages/macroass.cxx b/cui/source/tabpages/macroass.cxx index 973b6b1..7972967 100644 --- a/cui/source/tabpages/macroass.cxx +++ b/cui/source/tabpages/macroass.cxx @@ -105,11 +105,7 @@ OUString ConvertToUIName_Impl( SvxMacro *pMacro ) aEntry = aName.getToken( nCount-1, '.' ); if ( nCount > 2 ) { - aEntry += "("; - aEntry += aName.getToken( 0, '.' ); - aEntry += "."; - aEntry += aName.getToken( nCount-2, '.' ); - aEntry += ")"; + aEntry += "(" + aName.getToken( 0, '.' ) + "." + aName.getToken( nCount-2, '.' ) + ")"; } return aEntry; } @@ -172,8 +168,7 @@ void SfxMacroTabPage::dispose() void SfxMacroTabPage::AddEvent( const OUString & rEventName, sal_uInt16 nEventId ) { - OUString sTmp( rEventName ); - sTmp += "\t"; + OUString sTmp = rEventName + "\t"; // if the table is valid already SvxMacro* pM = aTbl.Get( nEventId ); diff --git a/cui/source/tabpages/numpages.cxx b/cui/source/tabpages/numpages.cxx index 1d9d220..e3795cd 100644 --- a/cui/source/tabpages/numpages.cxx +++ b/cui/source/tabpages/numpages.cxx @@ -1345,8 +1345,7 @@ void SvxNumOptionsTabPage::Reset( const SfxItemSet* rSet ) m_pLevelLB->InsertEntry( OUString::number(i)); if(pSaveNum->GetLevelCount() > 1) { - OUString sEntry( "1 - " ); - sEntry += OUString::number( pSaveNum->GetLevelCount() ); + OUString sEntry = "1 - " + OUString::number( pSaveNum->GetLevelCount() ); m_pLevelLB->InsertEntry(sEntry); m_pLevelLB->SelectEntry(sEntry); } @@ -2811,20 +2810,16 @@ void lcl_PrintDebugOutput(FixedText& rFixed, const SvxNumberFormat& rNumFmt) OUString const sHash( " # " ); if ( rNumFmt.GetPositionAndSpaceMode() == SvxNumberFormat::LABEL_WIDTH_AND_POSITION ) { - OUString sDebugText( OUString::number( convertTwipToMm100(rNumFmt.GetAbsLSpace() ) ) ); - sDebugText += sHash; - sDebugText += OUString::number( convertTwipToMm100(rNumFmt.GetCharTextDistance() ) ); - sDebugText += sHash; - sDebugText += OUString::number( convertTwipToMm100(rNumFmt.GetFirstLineOffset() ) ); + OUString sDebugText = OUString::number( convertTwipToMm100(rNumFmt.GetAbsLSpace() ) ) + sHash + + OUString::number( convertTwipToMm100(rNumFmt.GetCharTextDistance() ) + sHash + + OUString::number( convertTwipToMm100(rNumFmt.GetFirstLineOffset() ) ); rFixed.SetText(sDebugText); } else if ( rNumFmt.GetPositionAndSpaceMode() == SvxNumberFormat::LABEL_ALIGNMENT ) { - OUString sDebugText( OUString::number( convertTwipToMm100(rNumFmt.GetListtabPos() ) ) ); - sDebugText += sHash; - sDebugText += OUString::number( convertTwipToMm100(rNumFmt.GetFirstLineIndent() ) ); - sDebugText += sHash; - sDebugText += OUString::number( convertTwipToMm100(rNumFmt.GetIndentAt() ) ); + OUString sDebugText = OUString::number( convertTwipToMm100(rNumFmt.GetListtabPos() ) ) + sHash; + + OUString::number( convertTwipToMm100(rNumFmt.GetFirstLineIndent() ) ) + sHash + + OUString::number( convertTwipToMm100(rNumFmt.GetIndentAt() ) ); rFixed.SetText(sDebugText); } @@ -3123,8 +3118,7 @@ void SvxNumPositionTabPage::Reset( const SfxItemSet* rSet ) m_pLevelLB->InsertEntry( OUString::number(i) ); if(pSaveNum->GetLevelCount() > 1) { - OUString sEntry( "1 - " ); - sEntry += OUString::number( pSaveNum->GetLevelCount() ); + OUString sEntry = "1 - " + OUString::number( pSaveNum->GetLevelCount() ); m_pLevelLB->InsertEntry(sEntry); m_pLevelLB->SelectEntry(sEntry); } diff --git a/cui/source/tabpages/page.cxx b/cui/source/tabpages/page.cxx index f6fea29..2fdee55 100644 --- a/cui/source/tabpages/page.cxx +++ b/cui/source/tabpages/page.cxx @@ -936,9 +936,7 @@ IMPL_LINK_NOARG_TYPED(SvxPageDescPage, PaperBinHdl_Impl, Control&, void) if ( aName.isEmpty() ) { - aName = aPaperBin; - aName += " "; - aName += OUString::number( i+1 ); + aName = aPaperBin + " " + OUString::number( i+1 ); } nEntryPos = m_pPaperTrayBox->InsertEntry( aName ); m_pPaperTrayBox->SetEntryData( nEntryPos, reinterpret_cast<void*>((sal_uLong)i) ); diff --git a/cui/source/tabpages/tpgradnt.cxx b/cui/source/tabpages/tpgradnt.cxx index 7e9eba6..d993c3e 100644 --- a/cui/source/tabpages/tpgradnt.cxx +++ b/cui/source/tabpages/tpgradnt.cxx @@ -228,8 +228,7 @@ void SvxGradientTabPage::ActivatePage( const SfxItemSet& ) if ( aURL.getBase().getLength() > 18 ) { - aString += aURL.getBase().copy( 0, 15 ); - aString += "..."; + aString += aURL.getBase().copy( 0, 15 ) + "..."; } else aString += aURL.getBase(); diff --git a/cui/source/tabpages/tphatch.cxx b/cui/source/tabpages/tphatch.cxx index 33eb0d3..ba7749b 100644 --- a/cui/source/tabpages/tphatch.cxx +++ b/cui/source/tabpages/tphatch.cxx @@ -204,8 +204,7 @@ void SvxHatchTabPage::ActivatePage( const SfxItemSet& rSet ) if ( aURL.getBase().getLength() > 18 ) { - aString += aURL.getBase().copy( 0, 15 ); - aString += "..."; + aString += aURL.getBase().copy( 0, 15 ) + "..."; } else aString += aURL.getBase(); diff --git a/cui/source/tabpages/tplnedef.cxx b/cui/source/tabpages/tplnedef.cxx index 499dfb5..d689690 100644 --- a/cui/source/tabpages/tplnedef.cxx +++ b/cui/source/tabpages/tplnedef.cxx @@ -543,9 +543,7 @@ IMPL_LINK_NOARG_TYPED(SvxLineDefTabPage, ClickAddHdl_Impl, Button*, void) while ( !bDifferent ) { - aName = aNewName; - aName += " "; - aName += OUString::number( j++ ); + aName = aNewName + " " + OUString::number( j++ ); bDifferent = true; for ( long i = 0; i < nCount && bDifferent; i++ ) diff --git a/cui/source/tabpages/tplneend.cxx b/cui/source/tabpages/tplneend.cxx index 511b973..c52ebf0 100644 --- a/cui/source/tabpages/tplneend.cxx +++ b/cui/source/tabpages/tplneend.cxx @@ -435,9 +435,7 @@ IMPL_LINK_NOARG_TYPED(SvxLineEndDefTabPage, ClickAddHdl_Impl, Button*, void) while ( !bDifferent ) { - aName = aNewName; - aName += " "; - aName += OUString::number( j++ ); + aName = aNewName + " " + OUString::number( j++ ); bDifferent = true; for( long i = 0; i < nCount && bDifferent; i++ ) diff --git a/cui/source/tabpages/tppattern.cxx b/cui/source/tabpages/tppattern.cxx index 4b2570b..10fcf5d 100644 --- a/cui/source/tabpages/tppattern.cxx +++ b/cui/source/tabpages/tppattern.cxx @@ -219,8 +219,7 @@ void SvxPatternTabPage::ActivatePage( const SfxItemSet& ) if( aURL.getBase().getLength() > 18 ) { - aString += aURL.getBase().copy( 0, 15 ); - aString += "..."; + aString += aURL.getBase().copy( 0, 15 ) + "..."; } else aString += aURL.getBase(); @@ -478,9 +477,7 @@ IMPL_LINK_NOARG_TYPED(SvxPatternTabPage, ClickAddHdl_Impl, Button*, void) while( !bDifferent ) { - aName = aNewName; - aName += " "; - aName += OUString::number( j++ ); + aName = aNewName + " " + OUString::number( j++ ); bDifferent = true; for( long i = 0; i < nCount && bDifferent; i++ ) diff --git a/dbaccess/source/core/api/KeySet.cxx b/dbaccess/source/core/api/KeySet.cxx index 28a1f1c..3af8899 100644 --- a/dbaccess/source/core/api/KeySet.cxx +++ b/dbaccess/source/core/api/KeySet.cxx @@ -756,10 +756,7 @@ void OKeySet::executeInsert( const ORowSetRow& _rInsertRow,const OUString& i_sSQ SelectColumnsMetaData::const_iterator aFind = m_pKeyColumnNames->find(*aAutoIter); if ( aFind != aEnd ) { - sMaxStmt += sMax; - sMaxStmt += ::dbtools::quoteName( sQuote,aFind->second.sRealName -); - sMaxStmt += sMaxEnd; + sMaxStmt += sMax + ::dbtools::quoteName( sQuote,aFind->second.sRealName) + sMaxEnd; } } diff --git a/dbaccess/source/core/dataaccess/documentcontainer.cxx b/dbaccess/source/core/dataaccess/documentcontainer.cxx index aaff380..9770c41 100644 --- a/dbaccess/source/core/dataaccess/documentcontainer.cxx +++ b/dbaccess/source/core/dataaccess/documentcontainer.cxx @@ -220,8 +220,7 @@ Reference< XInterface > SAL_CALL ODocumentContainer::createInstanceWithArguments { static const char sBaseName[] = "Obj"; - sPersistentName = sBaseName; - sPersistentName += OUString::number(rDefinitions.size() + 1); + sPersistentName = sBaseName + OUString::number(rDefinitions.size() + 1); Reference<XNameAccess> xElements(getContainerStorage(),UNO_QUERY); if ( xElements.is() ) sPersistentName = ::dbtools::createUniqueName(xElements,sPersistentName); diff --git a/dbaccess/source/filter/xml/xmlFileBasedDatabase.cxx b/dbaccess/source/filter/xml/xmlFileBasedDatabase.cxx index 90f0b1f..458a26b 100644 --- a/dbaccess/source/filter/xml/xmlFileBasedDatabase.cxx +++ b/dbaccess/source/filter/xml/xmlFileBasedDatabase.cxx @@ -99,8 +99,7 @@ OXMLFileBasedDatabase::OXMLFileBasedDatabase( ODBFilter& rImport, if ( !(sLocation.isEmpty() || sMediaType.isEmpty()) ) { ::dbaccess::ODsnTypeCollection aTypeCollection(rImport.GetComponentContext()); - OUString sURL(aTypeCollection.getDatasourcePrefixFromMediaType(sMediaType,sFileTypeExtension)); - sURL += sLocation; + OUString sURL = aTypeCollection.getDatasourcePrefixFromMediaType(sMediaType,sFileTypeExtension) + sLocation; try { xDataSource->setPropertyValue(PROPERTY_URL,makeAny(sURL)); diff --git a/dbaccess/source/ui/dlg/DBSetupConnectionPages.cxx b/dbaccess/source/ui/dlg/DBSetupConnectionPages.cxx index 565386d..8d6a040 100644 --- a/dbaccess/source/ui/dlg/DBSetupConnectionPages.cxx +++ b/dbaccess/source/ui/dlg/DBSetupConnectionPages.cxx @@ -189,8 +189,7 @@ using namespace ::com::sun::star; OSL_ENSURE(pCollection, "OLDAPConnectionPageSetup::FillItemSet : really need a DSN type collection !"); if (pCollection) { - OUString sUrl = pCollection->getPrefix( "sdbc:address:ldap:"); - sUrl += m_pETHostServer->GetText(); + OUString sUrl = pCollection->getPrefix( "sdbc:address:ldap:") + m_pETHostServer->GetText(); _rSet->Put(SfxStringItem(DSID_CONNECTURL, sUrl)); bChangedSomething = true; } diff --git a/dbaccess/source/ui/dlg/directsql.cxx b/dbaccess/source/ui/dlg/directsql.cxx index 7ae9087..97b79d2 100644 --- a/dbaccess/source/ui/dlg/directsql.cxx +++ b/dbaccess/source/ui/dlg/directsql.cxx @@ -258,11 +258,9 @@ namespace dbaui void DirectSQLDialog::addOutputText(const OUString& _rMessage) { - OUString sAppendMessage = _rMessage; - sAppendMessage += "\n"; + OUString sAppendMessage = _rMessage + "\n"; - OUString sCompleteMessage = m_pOutput->GetText(); - sCompleteMessage += sAppendMessage; + OUString sCompleteMessage = m_pOutput->GetText() + sAppendMessage; m_pOutput->SetText(sCompleteMessage); } diff --git a/dbaccess/source/ui/dlg/indexdialog.cxx b/dbaccess/source/ui/dlg/indexdialog.cxx index c864a2c..0661a95 100644 --- a/dbaccess/source/ui/dlg/indexdialog.cxx +++ b/dbaccess/source/ui/dlg/indexdialog.cxx @@ -376,8 +376,7 @@ namespace dbaui for ( i = 1; i < 0x7FFFFFFF; ++i ) { - sNewIndexName = sNewIndexNameBase; - sNewIndexName += OUString::number(i); + sNewIndexName = sNewIndexNameBase + OUString::number(i); if (m_pIndexes->end() == m_pIndexes->find(sNewIndexName)) break; } diff --git a/dbaccess/source/ui/dlg/sqlmessage.cxx b/dbaccess/source/ui/dlg/sqlmessage.cxx index 2df16e9..8d8c8c4 100644 --- a/dbaccess/source/ui/dlg/sqlmessage.cxx +++ b/dbaccess/source/ui/dlg/sqlmessage.cxx @@ -364,18 +364,12 @@ IMPL_LINK_NOARG_TYPED(OExceptionChainDialog, OnExceptionSelected, SvTreeListBox* if ( !aExceptionInfo.sSQLState.isEmpty() ) { - sText += m_sStatusLabel; - sText += ": "; - sText += aExceptionInfo.sSQLState; - sText += "\n"; + sText += m_sStatusLabel + ": " + aExceptionInfo.sSQLState + "\n"; } if ( !aExceptionInfo.sErrorCode.isEmpty() ) { - sText += m_sErrorCodeLabel; - sText += ": "; - sText += aExceptionInfo.sErrorCode; - sText += "\n"; + sText += m_sErrorCodeLabel + ": " + aExceptionInfo.sErrorCode + "\n"; } if ( !sText.isEmpty() ) diff --git a/dbaccess/source/ui/dlg/tablespage.cxx b/dbaccess/source/ui/dlg/tablespage.cxx index 9b682a6..53062db 100644 --- a/dbaccess/source/ui/dlg/tablespage.cxx +++ b/dbaccess/source/ui/dlg/tablespage.cxx @@ -479,8 +479,7 @@ namespace dbaui bCatalogWildcard = OTableTreeListBox::isWildcardChecked(pCatalog); if (m_bCatalogAtStart) { - sComposedName += m_pTablesList->GetEntryText( pCatalog ); - sComposedName += m_sCatalogSeparator; + sComposedName += m_pTablesList->GetEntryText( pCatalog ) + m_sCatalogSeparator; if (bCatalogWildcard) sComposedName += sWildcard; } @@ -490,14 +489,12 @@ namespace dbaui sCatalog = sWildcard; else sCatalog.clear(); - sCatalog += m_sCatalogSeparator; - sCatalog += m_pTablesList->GetEntryText( pCatalog ); + sCatalog += m_sCatalogSeparator + m_pTablesList->GetEntryText( pCatalog ); } } } bSchemaWildcard = OTableTreeListBox::isWildcardChecked(pSchema); - sComposedName += m_pTablesList->GetEntryText( pSchema ); - sComposedName += sDot; + sComposedName += m_pTablesList->GetEntryText( pSchema ) + sDot; } if (bSchemaWildcard) diff --git a/dbaccess/source/ui/misc/WNameMatch.cxx b/dbaccess/source/ui/misc/WNameMatch.cxx index e5a5e962..75d9a78 100644 --- a/dbaccess/source/ui/misc/WNameMatch.cxx +++ b/dbaccess/source/ui/misc/WNameMatch.cxx @@ -67,10 +67,8 @@ OWizNameMatching::OWizNameMatching( vcl::Window* pParent) m_pCTRL_LEFT->SetStyle( m_pCTRL_LEFT->GetStyle() | WB_FORCE_MAKEVISIBLE ); m_pCTRL_RIGHT->SetStyle( m_pCTRL_RIGHT->GetStyle() | WB_FORCE_MAKEVISIBLE ); - m_sSourceText = m_pTABLE_LEFT->GetText(); - m_sSourceText += "\n"; - m_sDestText = m_pTABLE_RIGHT->GetText(); - m_sDestText += "\n"; + m_sSourceText = m_pTABLE_LEFT->GetText() + "\n"; + m_sDestText = m_pTABLE_RIGHT->GetText() + "\n"; } OWizNameMatching::~OWizNameMatching() @@ -113,14 +111,12 @@ void OWizNameMatching::ActivatePage( ) { // set source table name - OUString aName = m_sSourceText; - aName += m_pParent->m_sSourceName; + OUString aName = m_sSourceText + m_pParent->m_sSourceName; m_pTABLE_LEFT->SetText(aName); // set dest table name - aName = m_sDestText; - aName += m_pParent->m_sName; + aName = m_sDestText + m_pParent->m_sName; m_pTABLE_RIGHT->SetText(aName); m_pCTRL_LEFT->FillListBox(m_pParent->getSrcVector()); diff --git a/dbaccess/source/ui/querydesign/QueryDesignView.cxx b/dbaccess/source/ui/querydesign/QueryDesignView.cxx index 8f8748c..1e08467 100644 --- a/dbaccess/source/ui/querydesign/QueryDesignView.cxx +++ b/dbaccess/source/ui/querydesign/QueryDesignView.cxx @@ -108,8 +108,7 @@ namespace OUString sRet; if ( _bQuote && !_sAliasName.isEmpty() ) { - sRet = ::dbtools::quoteName(_sQuote,_sAliasName); - sRet += "."; + sRet = ::dbtools::quoteName(_sQuote,_sAliasName) + "."; } return sRet; } @@ -408,12 +407,10 @@ namespace aErg += " FULL OUTER "; break; } - aErg += "JOIN "; - aErg += rRh; + aErg += "JOIN " + rRh; if ( CROSS_JOIN != pData->GetJoinType() && !pData->isNatural() ) { - aErg += " ON "; - aErg += BuildJoinCriteria(_xConnection,&pData->GetConnLineDataList(),pData); + aErg += " ON " + BuildJoinCriteria(_xConnection,&pData->GetConnLineDataList(),pData); } return aErg; @@ -776,10 +773,7 @@ namespace if ( pEntryField->isAggreateFunction() ) { OSL_ENSURE(!pEntryField->GetFunction().isEmpty(),"No function name for aggregate given!"); - aHavingStr += pEntryField->GetFunction(); - aHavingStr += "("; // bracket - aHavingStr += aWork; - aHavingStr += ")"; // bracket + aHavingStr += pEntryField->GetFunction() + "(" + aWork + ")"; // bracket } else aHavingStr += aWork; @@ -941,12 +935,9 @@ namespace } else { - aWorkStr += quoteTableAlias(bMulti,pEntryField->GetAlias(),aQuote); - aWorkStr += ::dbtools::quoteName(aQuote, aColumnName); + aWorkStr += quoteTableAlias(bMulti,pEntryField->GetAlias(),aQuote) + ::dbtools::quoteName(aQuote, aColumnName); } - aWorkStr += " "; - aWorkStr += OUString( ";ASC;DESC" ).getToken( (sal_uInt16)eOrder, ';' ); - aWorkStr += ","; + aWorkStr += " " + OUString( ";ASC;DESC" ).getToken( (sal_uInt16)eOrder, ';' ) + ","; } } @@ -1002,8 +993,7 @@ namespace if(_rTableNames.insert(sTabName).second) { - _rsTableListStr += sTabName; - _rsTableListStr += ","; + _rsTableListStr += sTabName + ","; } } OUString GenerateFromClause( const Reference< XConnection>& _xConnection, @@ -1113,8 +1103,7 @@ namespace const OQueryTableWindow* pEntryTab = static_cast<const OQueryTableWindow*>(aTabIter->second.get()); if(!pEntryTab->ExistsAConn()) { - aTableListStr += BuildTable(_xConnection,pEntryTab); - aTableListStr += ","; + aTableListStr += BuildTable(_xConnection,pEntryTab) + ","; } } @@ -1174,16 +1163,14 @@ namespace if ( aGroupByNames.find(sGroupByPart) == aGroupByNames.end() ) { aGroupByNames.insert(::std::map< OUString,bool>::value_type(sGroupByPart,true)); - aGroupByStr += sGroupByPart; - aGroupByStr += ","; + aGroupByStr += sGroupByPart + ","; } } } if ( !aGroupByStr.isEmpty() ) { aGroupByStr = aGroupByStr.replaceAt(aGroupByStr.getLength()-1,1, OUString(' ') ); - OUString aGroupByStr2(" GROUP BY "); - aGroupByStr2 += aGroupByStr; + OUString aGroupByStr2 = " GROUP BY " + aGroupByStr; aGroupByStr = aGroupByStr2; } } @@ -2736,8 +2723,7 @@ void OQueryDesignView::fillValidFields(const OUString& sAliasName, ComboBox* pFi OQueryTableWindow* pCurrentWin = static_cast<OQueryTableWindow*>(aIter->second.get()); if (bAllTables || (pCurrentWin->GetAliasName() == sAliasName)) { - strCurrentPrefix = pCurrentWin->GetAliasName(); - strCurrentPrefix += "."; + strCurrentPrefix = pCurrentWin->GetAliasName() + "."; pCurrentWin->EnumValidFields(aFields); @@ -2853,8 +2839,7 @@ OUString OQueryDesignView::getStatement() OUString aTmp = "( " + aJoinCrit + " )"; if(!aCriteriaListStr.isEmpty()) { - aTmp += C_AND; - aTmp += aCriteriaListStr.makeStringAndClear(); + aTmp += C_AND + aCriteriaListStr.makeStringAndClear(); } aCriteriaListStr = aTmp; } @@ -3019,10 +3004,7 @@ OSQLParseNode* OQueryDesignView::getPredicateTreeFromEntry(const OTableFieldDesc { // first try the international version OUString sSql; - sSql += "SELECT * "; - sSql += " FROM x WHERE "; - sSql += pEntry->GetField(); - sSql += _sCriteria; + sSql += "SELECT * FROM x WHERE " + pEntry->GetField() + _sCriteria; std::unique_ptr<OSQLParseNode> pParseNode( rParser.parseTree( _rsErrorMessage, sSql, true ) ); nType = DataType::DOUBLE; if ( pParseNode.get() ) diff --git a/dbaccess/source/ui/querydesign/SelectionBrowseBox.cxx b/dbaccess/source/ui/querydesign/SelectionBrowseBox.cxx index 65a7c5b..971e821 100644 --- a/dbaccess/source/ui/querydesign/SelectionBrowseBox.cxx +++ b/dbaccess/source/ui/querydesign/SelectionBrowseBox.cxx @@ -703,8 +703,7 @@ bool OSelectionBrowseBox::saveField(OUString& _sFieldName ,OTableFieldDescRef& _ sSql = "SELECT " + sSql; if ( !sFieldAlias.isEmpty() ) { // always quote the alias name: there cannot be a function in it - sSql += " "; - sSql += ::dbtools::quoteName( xMetaData->getIdentifierQuoteString(), sFieldAlias ); + sSql += " " + ::dbtools::quoteName( xMetaData->getIdentifierQuoteString(), sFieldAlias ); } sSql += " FROM x"; @@ -1767,11 +1766,7 @@ void OSelectionBrowseBox::AddCondition( const OTableFieldDescRef& rInfo, const O OUString sOldCriteria = pLastEntry->GetCriteria( nLevel ); if ( !sOldCriteria.isEmpty() ) { - sCriteria = "( "; - sCriteria += sOldCriteria; - sCriteria += " OR "; - sCriteria += rValue; - sCriteria += " )"; + sCriteria = "( " + sOldCriteria + " OR " + rValue + " )"; } pLastEntry->SetCriteria( nLevel, sCriteria); if(nLevel == (m_nVisibleCount-BROW_CRIT1_ROW-1)) diff --git a/dbaccess/source/ui/relationdesign/RTableConnectionData.cxx b/dbaccess/source/ui/relationdesign/RTableConnectionData.cxx index 3c1456d..88654ee 100644 --- a/dbaccess/source/ui/relationdesign/RTableConnectionData.cxx +++ b/dbaccess/source/ui/relationdesign/RTableConnectionData.cxx @@ -272,8 +272,7 @@ bool ORelationTableConnectionData::Update() // build a foreign key name OUString sSourceName; xTableProp->getPropertyValue(PROPERTY_NAME) >>= sSourceName; - OUString sKeyName = sSourceName; - sKeyName += getReferencedTable()->GetTableName(); + OUString sKeyName = sSourceName + getReferencedTable()->GetTableName(); xKey->setPropertyValue(PROPERTY_NAME,makeAny(sKeyName)); xKey->setPropertyValue(PROPERTY_TYPE,makeAny(KeyType::FOREIGN)); diff --git a/dbaccess/source/ui/tabledesign/TableController.cxx b/dbaccess/source/ui/tabledesign/TableController.cxx index adb008b..192b757 100644 --- a/dbaccess/source/ui/tabledesign/TableController.cxx +++ b/dbaccess/source/ui/tabledesign/TableController.cxx @@ -1493,8 +1493,7 @@ OUString OTableController::getPrivateTitle() const if ( sTitle.isEmpty() ) { OUString aName = ModuleRes(STR_TBL_TITLE); - sTitle = aName.getToken(0,' '); - sTitle += OUString::number(getCurrentStartNumber()); + sTitle = aName.getToken(0,' ') + OUString::number(getCurrentStartNumber()); } } catch( const Exception& ) _______________________________________________ Libreoffice-commits mailing list libreoffice-comm...@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits