[Libreoffice-commits] core.git: accessibility/source basic/source chart2/source connectivity/source dbaccess/source editeng/source helpcompiler/inc include/vcl lotuswordpro/source odk/examples reportdesign/source sax/qa sc/source sd/source setup_native/source sfx2/source stoc/source svx/qa svx/source sw/qa sw/source test/source vcl/source vcl/unx writerfilter/source xmlhelp/source xmloff/source
Noel Grandin (via logerrit)
logerrit at kemper.freedesktop.org
Fri Oct 11 05:19:03 UTC 2019
accessibility/source/standard/accessiblemenuitemcomponent.cxx | 2
basic/source/classes/sbxmod.cxx | 2
basic/source/runtime/iosys.cxx | 6
chart2/source/view/main/VLegend.cxx | 2
connectivity/source/drivers/ado/ADatabaseMetaData.cxx | 2
connectivity/source/drivers/firebird/DatabaseMetaData.cxx | 4
connectivity/source/drivers/odbc/OConnection.cxx | 4
dbaccess/source/ui/dlg/DbAdminImpl.cxx | 2
dbaccess/source/ui/misc/TokenWriter.cxx | 87 ++++------
editeng/source/items/frmitems.cxx | 58 ++----
editeng/source/items/optitems.cxx | 3
editeng/source/items/paraitem.cxx | 5
editeng/source/items/textitem.cxx | 7
editeng/source/rtf/svxrtf.cxx | 2
helpcompiler/inc/HelpCompiler.hxx | 2
include/vcl/weld.hxx | 2
lotuswordpro/source/filter/xfilter/xfstylecont.cxx | 2
odk/examples/DevelopersGuide/Database/DriverSkeleton/SConnection.cxx | 4
reportdesign/source/filter/xml/xmlExport.cxx | 2
sax/qa/cppunit/xmlimport.cxx | 12 -
sc/source/core/data/attrib.cxx | 2
sc/source/core/data/docpool.cxx | 8
sc/source/core/opencl/formulagroupcl.cxx | 2
sc/source/filter/excel/xecontent.cxx | 2
sc/source/ui/app/inputhdl.cxx | 2
sc/source/ui/view/gridwin.cxx | 2
sd/source/ui/accessibility/AccessibleDocumentViewBase.cxx | 2
sd/source/ui/annotations/annotationmanager.cxx | 2
sd/source/ui/dlg/custsdlg.cxx | 4
setup_native/source/win32/customactions/shellextensions/completeinstallpath.cxx | 2
sfx2/source/dialog/mailmodel.cxx | 2
sfx2/source/dialog/versdlg.cxx | 8
sfx2/source/doc/DocumentMetadataAccess.cxx | 2
sfx2/source/doc/sfxbasemodel.cxx | 2
sfx2/source/doc/templatedlg.cxx | 6
stoc/source/defaultregistry/defaultregistry.cxx | 6
stoc/source/implementationregistration/implreg.cxx | 4
stoc/source/javavm/javavm.cxx | 7
svx/qa/unit/customshapes.cxx | 14 -
svx/source/form/fmshimp.cxx | 8
svx/source/form/fmsrcimp.cxx | 2
svx/source/svdraw/presetooxhandleadjustmentrelations.cxx | 2
svx/source/svdraw/svdattr.cxx | 2
sw/qa/extras/globalfilter/globalfilter.cxx | 4
sw/source/core/access/acctable.cxx | 4
sw/source/filter/ww8/ww8par5.cxx | 2
sw/source/uibase/utlui/attrdesc.cxx | 44 ++---
test/source/bootstrapfixture.cxx | 3
vcl/source/gdi/mtfxmldump.cxx | 4
vcl/source/uitest/logger.cxx | 4
vcl/source/window/builder.cxx | 2
vcl/source/window/tabdlg.cxx | 2
vcl/unx/generic/printer/cupsmgr.cxx | 4
vcl/unx/gtk3/gtk3gtkinst.cxx | 2
vcl/unx/gtk3/gtk3gtksalmenu.cxx | 6
writerfilter/source/dmapper/DomainMapper_Impl.cxx | 2
xmlhelp/source/treeview/tvread.cxx | 2
xmloff/source/text/txtprhdl.cxx | 4
58 files changed, 184 insertions(+), 208 deletions(-)
New commits:
commit cdbac696fb0cbb1d09645bc02799eed5504b192b
Author: Noel Grandin <noel.grandin at collabora.co.uk>
AuthorDate: Thu Oct 10 16:23:37 2019 +0200
Commit: Noel Grandin <noel.grandin at collabora.co.uk>
CommitDate: Fri Oct 11 07:18:11 2019 +0200
simplify "a = a +" to "a +="
mostly so that my stringadd loplugin can point out places to improve
Change-Id: I9920ee1c99cdb6b811ba67ff9d8e32aa261884b5
Reviewed-on: https://gerrit.libreoffice.org/80618
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin at collabora.co.uk>
diff --git a/accessibility/source/standard/accessiblemenuitemcomponent.cxx b/accessibility/source/standard/accessiblemenuitemcomponent.cxx
index 1e1499734e9b..7160944ebf72 100644
--- a/accessibility/source/standard/accessiblemenuitemcomponent.cxx
+++ b/accessibility/source/standard/accessiblemenuitemcomponent.cxx
@@ -189,7 +189,7 @@ OUString OAccessibleMenuItemComponent::GetAccessibleName()
sName = OutputDevice::GetNonMnemonicString( sName );
#if defined(_WIN32)
if ( m_pParent->GetAccelKey( nItemId ).GetName().getLength() )
- sName = sName + "\t" + m_pParent->GetAccelKey(nItemId).GetName();
+ sName += "\t" + m_pParent->GetAccelKey(nItemId).GetName();
#endif
}
diff --git a/basic/source/classes/sbxmod.cxx b/basic/source/classes/sbxmod.cxx
index 72811562ac92..0e93da321ccd 100644
--- a/basic/source/classes/sbxmod.cxx
+++ b/basic/source/classes/sbxmod.cxx
@@ -2565,7 +2565,7 @@ void SbUserFormModule::InitObject()
}
catch(const Exception& ) {}
- sDialogUrl = sDialogUrl + sProjectName + "." + GetName() + "?location=document";
+ sDialogUrl += sProjectName + "." + GetName() + "?location=document";
uno::Reference< awt::XDialogProvider > xProvider = awt::DialogProvider::createWithModel( xContext, m_xModel );
m_xDialog = xProvider->createDialog( sDialogUrl );
diff --git a/basic/source/runtime/iosys.cxx b/basic/source/runtime/iosys.cxx
index cf433052014c..6797b41f7758 100644
--- a/basic/source/runtime/iosys.cxx
+++ b/basic/source/runtime/iosys.cxx
@@ -535,7 +535,7 @@ ErrCode const & SbiStream::Read( char& ch )
if (aLine.isEmpty())
{
Read( aLine );
- aLine = aLine + OString('\n');
+ aLine += OString('\n');
}
ch = aLine[0];
aLine = aLine.copy(1);
@@ -582,7 +582,7 @@ ErrCode SbiStream::Write( const OString& rBuf )
}
if( IsText() )
{
- aLine = aLine + rBuf;
+ aLine += rBuf;
// Get it out, if the end is an LF, but strip CRLF before,
// because the SvStream adds a CRLF!
sal_Int32 nLineLen = aLine.getLength();
@@ -729,7 +729,7 @@ char SbiIoSystem::Read()
if( aIn.isEmpty() )
{
ReadCon( aIn );
- aIn = aIn + OString('\n');
+ aIn += OString('\n');
}
ch = aIn[0];
aIn = aIn.copy(1);
diff --git a/chart2/source/view/main/VLegend.cxx b/chart2/source/view/main/VLegend.cxx
index cc7ff9a2d567..c8edeeed00a0 100644
--- a/chart2/source/view/main/VLegend.cxx
+++ b/chart2/source/view/main/VLegend.cxx
@@ -167,7 +167,7 @@ awt::Size lcl_createTextShapes(
if( i == 1 )
break;
- aLabelString = aLabelString + aLabelSeq[i]->getString();
+ aLabelString += aLabelSeq[i]->getString();
// workaround for Issue #i67540#
if( aLabelString.isEmpty())
aLabelString = " ";
diff --git a/connectivity/source/drivers/ado/ADatabaseMetaData.cxx b/connectivity/source/drivers/ado/ADatabaseMetaData.cxx
index 29871b110f13..8e46761e8c27 100644
--- a/connectivity/source/drivers/ado/ADatabaseMetaData.cxx
+++ b/connectivity/source/drivers/ado/ADatabaseMetaData.cxx
@@ -916,7 +916,7 @@ OUString SAL_CALL ODatabaseMetaData::getSQLKeywords( )
WpOLEAppendCollection<ADOFields, ADOField, WpADOField> aFields(aRecordset.GetFields());
WpADOField aField(aFields.GetItem(0));
aField.get_Value(aValue);
- aRet = aRet + aValue.getString() + ",";
+ aRet += aValue.getString() + ",";
aRecordset.MoveNext();
}
aRecordset.Close();
diff --git a/connectivity/source/drivers/firebird/DatabaseMetaData.cxx b/connectivity/source/drivers/firebird/DatabaseMetaData.cxx
index 74015586c093..b0344b5f8b8f 100644
--- a/connectivity/source/drivers/firebird/DatabaseMetaData.cxx
+++ b/connectivity/source/drivers/firebird/DatabaseMetaData.cxx
@@ -1502,9 +1502,9 @@ uno::Reference< XResultSet > ODatabaseMetaData::lcl_getKeys(const bool& bIsImpor
"ON FOREI.RDB$INDEX_NAME = FOREIGN_INDEX.RDB$INDEX_NAME "
"WHERE FOREI.RDB$CONSTRAINT_TYPE = 'FOREIGN KEY' ";
if (bIsImport)
- sSQL = sSQL + "AND FOREI.RDB$RELATION_NAME = '"+ table +"'";
+ sSQL += "AND FOREI.RDB$RELATION_NAME = '"+ table +"'";
else
- sSQL = sSQL + "AND PRIM.RDB$RELATION_NAME = '"+ table +"'";
+ sSQL += "AND PRIM.RDB$RELATION_NAME = '"+ table +"'";
uno::Reference< XResultSet > rs = statement->executeQuery(sSQL);
uno::Reference< XRow > xRow( rs, UNO_QUERY_THROW );
diff --git a/connectivity/source/drivers/odbc/OConnection.cxx b/connectivity/source/drivers/odbc/OConnection.cxx
index e879684b68fa..589697f7ff72 100644
--- a/connectivity/source/drivers/odbc/OConnection.cxx
+++ b/connectivity/source/drivers/odbc/OConnection.cxx
@@ -224,13 +224,13 @@ SQLRETURN OConnection::Construct(const OUString& url,const Sequence< PropertyVal
{
if( ! (pBegin->Value >>= aUID) )
SAL_WARN("connectivity.odbc", "Construct: unable to get property user");
- aDSN = aDSN + ";UID=" + aUID;
+ aDSN += ";UID=" + aUID;
}
else if( pBegin->Name == "password")
{
if( ! (pBegin->Value >>= aPWD) )
SAL_WARN("connectivity.odbc", "Construct: unable to get property password");
- aDSN = aDSN + ";PWD=" + aPWD;
+ aDSN += ";PWD=" + aPWD;
}
else if( pBegin->Name == "UseCatalog")
{
diff --git a/dbaccess/source/ui/dlg/DbAdminImpl.cxx b/dbaccess/source/ui/dlg/DbAdminImpl.cxx
index 30037f1bbdf4..04a4da3a4aa3 100644
--- a/dbaccess/source/ui/dlg/DbAdminImpl.cxx
+++ b/dbaccess/source/ui/dlg/DbAdminImpl.cxx
@@ -124,7 +124,7 @@ namespace
if ( _pPortNumber )
{
- sNewUrl = sNewUrl + ":" + OUString::number(_pPortNumber->GetValue());
+ sNewUrl += ":" + OUString::number(_pPortNumber->GetValue());
}
return sNewUrl;
diff --git a/dbaccess/source/ui/misc/TokenWriter.cxx b/dbaccess/source/ui/misc/TokenWriter.cxx
index d134f3207d09..29a606b765c8 100644
--- a/dbaccess/source/ui/misc/TokenWriter.cxx
+++ b/dbaccess/source/ui/misc/TokenWriter.cxx
@@ -690,11 +690,11 @@ void OHTMLImportExport::WriteBody()
void OHTMLImportExport::WriteTables()
{
- OString aStrOut = OOO_STRING_SVTOOLS_HTML_table;
- aStrOut = aStrOut + " ";
- aStrOut = aStrOut + OOO_STRING_SVTOOLS_HTML_frame;
- aStrOut = aStrOut + "=";
- aStrOut = aStrOut + OOO_STRING_SVTOOLS_HTML_TF_void;
+ OString aStrOut = OOO_STRING_SVTOOLS_HTML_table
+ " "
+ OOO_STRING_SVTOOLS_HTML_frame
+ "="
+ OOO_STRING_SVTOOLS_HTML_TF_void;
Sequence< OUString> aNames;
Reference<XNameAccess> xColumns;
@@ -714,21 +714,21 @@ void OHTMLImportExport::WriteTables()
}
}
- aStrOut = aStrOut + " ";
- aStrOut = aStrOut + OOO_STRING_SVTOOLS_HTML_O_align;
- aStrOut = aStrOut + "=";
- aStrOut = aStrOut + OOO_STRING_SVTOOLS_HTML_AL_left;
- aStrOut = aStrOut + " ";
- aStrOut = aStrOut + OOO_STRING_SVTOOLS_HTML_O_cellspacing;
- aStrOut = aStrOut + "=";
- aStrOut = aStrOut + OString::number(nCellSpacing);
- aStrOut = aStrOut + " ";
- aStrOut = aStrOut + OOO_STRING_SVTOOLS_HTML_O_cols;
- aStrOut = aStrOut + "=";
- aStrOut = aStrOut + OString::number(aNames.getLength());
- aStrOut = aStrOut + " ";
- aStrOut = aStrOut + OOO_STRING_SVTOOLS_HTML_O_border;
- aStrOut = aStrOut + "=1";
+ aStrOut += " "
+ OOO_STRING_SVTOOLS_HTML_O_align
+ "="
+ OOO_STRING_SVTOOLS_HTML_AL_left
+ " "
+ OOO_STRING_SVTOOLS_HTML_O_cellspacing
+ "=" +
+ OString::number(nCellSpacing) +
+ " "
+ OOO_STRING_SVTOOLS_HTML_O_cols
+ "=" +
+ OString::number(aNames.getLength()) +
+ " "
+ OOO_STRING_SVTOOLS_HTML_O_border
+ "=1";
IncIndent(1);
TAG_ON( aStrOut.getStr() );
@@ -861,20 +861,19 @@ void OHTMLImportExport::WriteCell( sal_Int32 nFormat, sal_Int32 nWidthPixel, sal
// despite the <TABLE COLS=n> and <COL WIDTH=x> designation necessary,
// as Netscape is not paying attention to them.
// column width
- aStrTD = aStrTD + " ";
- aStrTD = aStrTD + OOO_STRING_SVTOOLS_HTML_O_width;
- aStrTD = aStrTD + "=";
- aStrTD = aStrTD + OString::number(nWidthPixel);
+ aStrTD += " "
+ OOO_STRING_SVTOOLS_HTML_O_width
+ "=" +
+ OString::number(nWidthPixel) +
// line height
- aStrTD = aStrTD + " ";
- aStrTD = aStrTD + OOO_STRING_SVTOOLS_HTML_O_height;
- aStrTD = aStrTD + "=";
- aStrTD = aStrTD + OString::number(nHeightPixel);
-
- aStrTD = aStrTD + " ";
- aStrTD = aStrTD + OOO_STRING_SVTOOLS_HTML_O_align;
- aStrTD = aStrTD + "=";
- aStrTD = aStrTD + pChar;
+ " "
+ OOO_STRING_SVTOOLS_HTML_O_height
+ "=" +
+ OString::number(nHeightPixel) +
+ " "
+ OOO_STRING_SVTOOLS_HTML_O_align
+ "=" +
+ pChar;
SvNumberFormatsSupplierObj* pSupplierImpl = m_xFormatter.is() ? comphelper::getUnoTunnelImplementation<SvNumberFormatsSupplierObj>(m_xFormatter->getNumberFormatsSupplier()) : nullptr;
SvNumberFormatter* pFormatter = pSupplierImpl ? pSupplierImpl->GetNumberFormatter() : nullptr;
@@ -929,18 +928,18 @@ void OHTMLImportExport::FontOn()
#endif
// <FONT FACE="xxx">
- OString aStrOut = "<";
- aStrOut = aStrOut + OOO_STRING_SVTOOLS_HTML_font;
- aStrOut = aStrOut + " ";
- aStrOut = aStrOut + OOO_STRING_SVTOOLS_HTML_O_face;
- aStrOut = aStrOut + "=";
- aStrOut = aStrOut + "\"";
- aStrOut = aStrOut + OUStringToOString(m_aFont.Name,osl_getThreadTextEncoding());
+ OString aStrOut = "<"
+ OOO_STRING_SVTOOLS_HTML_font
+ " "
+ OOO_STRING_SVTOOLS_HTML_O_face
+ "="
+ "\"" +
+ OUStringToOString(m_aFont.Name,osl_getThreadTextEncoding()) +
// TODO : think about the encoding of the font name
- aStrOut = aStrOut + "\"";
- aStrOut = aStrOut + " ";
- aStrOut = aStrOut + OOO_STRING_SVTOOLS_HTML_O_color;
- aStrOut = aStrOut + "=";
+ "\""
+ " "
+ OOO_STRING_SVTOOLS_HTML_O_color
+ "=";
m_pStream->WriteOString( aStrOut );
sal_Int32 nColor = 0;
diff --git a/editeng/source/items/frmitems.cxx b/editeng/source/items/frmitems.cxx
index 4cb09608cf2e..1d74227313fe 100644
--- a/editeng/source/items/frmitems.cxx
+++ b/editeng/source/items/frmitems.cxx
@@ -556,8 +556,7 @@ bool SvxLRSpaceItem::GetPresentation
Application::GetSettings().GetUILanguageTag());
else
{
- rText = rText +
- GetMetricText( nLeftMargin, eCoreUnit, ePresUnit, &rIntl ) +
+ rText += GetMetricText( nLeftMargin, eCoreUnit, ePresUnit, &rIntl ) +
" " + EditResId(GetMetricId(ePresUnit));
}
rText += OUString(cpDelim);
@@ -565,12 +564,11 @@ bool SvxLRSpaceItem::GetPresentation
{
rText += EditResId(RID_SVXITEMS_LRSPACE_FLINE);
if ( 100 != nPropFirstLineOfst )
- rText = rText + unicode::formatPercent(nPropFirstLineOfst,
- Application::GetSettings().GetUILanguageTag());
+ rText += unicode::formatPercent(nPropFirstLineOfst,
+ Application::GetSettings().GetUILanguageTag());
else
{
- rText = rText +
- GetMetricText( static_cast<long>(nFirstLineOfst),
+ rText += GetMetricText( static_cast<long>(nFirstLineOfst),
eCoreUnit, ePresUnit, &rIntl ) +
" " + EditResId(GetMetricId(ePresUnit));
}
@@ -578,12 +576,11 @@ bool SvxLRSpaceItem::GetPresentation
}
rText += EditResId(RID_SVXITEMS_LRSPACE_RIGHT);
if ( 100 != nPropRightMargin )
- rText = rText + unicode::formatPercent(nPropRightMargin,
+ rText += unicode::formatPercent(nPropRightMargin,
Application::GetSettings().GetUILanguageTag());
else
{
- rText = rText +
- GetMetricText( nRightMargin,
+ rText += GetMetricText( nRightMargin,
eCoreUnit, ePresUnit, &rIntl ) +
" " + EditResId(GetMetricId(ePresUnit));
}
@@ -799,11 +796,10 @@ bool SvxULSpaceItem::GetPresentation
}
else
{
- rText = rText +
- GetMetricText( static_cast<long>(nUpper), eCoreUnit, ePresUnit, &rIntl ) +
+ rText += GetMetricText( static_cast<long>(nUpper), eCoreUnit, ePresUnit, &rIntl ) +
" " + EditResId(GetMetricId(ePresUnit));
}
- rText = rText + OUString(cpDelim) + EditResId(RID_SVXITEMS_ULSPACE_LOWER);
+ rText += OUString(cpDelim) + EditResId(RID_SVXITEMS_ULSPACE_LOWER);
if ( 100 != nPropLower )
{
rText += unicode::formatPercent(nPropLower,
@@ -811,8 +807,7 @@ bool SvxULSpaceItem::GetPresentation
}
else
{
- rText = rText +
- GetMetricText( static_cast<long>(nLower), eCoreUnit, ePresUnit, &rIntl ) +
+ rText += GetMetricText( static_cast<long>(nLower), eCoreUnit, ePresUnit, &rIntl ) +
" " + EditResId(GetMetricId(ePresUnit));
}
return true;
@@ -965,7 +960,7 @@ bool SvxProtectItem::GetPresentation
if ( bSize )
pId = RID_SVXITEMS_PROT_SIZE_TRUE;
- rText = rText + EditResId(pId) + OUString(cpDelim);
+ rText += EditResId(pId) + OUString(cpDelim);
pId = RID_SVXITEMS_PROT_POS_FALSE;
if ( bPos )
@@ -1176,8 +1171,7 @@ bool SvxShadowItem::GetPresentation
if ( aShadowColor.GetTransparency() )
pId = RID_SVXITEMS_TRANSPARENT_TRUE;
- rText = rText +
- EditResId(pId) +
+ rText += EditResId(pId) +
OUString(cpDelim) +
GetMetricText( static_cast<long>(nWidth), eCoreUnit, ePresUnit, &rIntl ) +
OUString(cpDelim) +
@@ -1193,8 +1187,7 @@ bool SvxShadowItem::GetPresentation
const char* pId = RID_SVXITEMS_TRANSPARENT_FALSE;
if ( aShadowColor.GetTransparency() )
pId = RID_SVXITEMS_TRANSPARENT_TRUE;
- rText = rText +
- EditResId(pId) +
+ rText += EditResId(pId) +
OUString(cpDelim) +
GetMetricText( static_cast<long>(nWidth), eCoreUnit, ePresUnit, &rIntl ) +
" " + EditResId(GetMetricId(ePresUnit)) +
@@ -1722,23 +1715,22 @@ bool SvxBoxItem::GetPresentation
{
if ( pBottom )
{
- rText = rText + pBottom->GetValueString( eCoreUnit, ePresUnit, &rIntl ) + cpDelimTmp;
+ rText += pBottom->GetValueString( eCoreUnit, ePresUnit, &rIntl ) + cpDelimTmp;
}
if ( pLeft )
{
- rText = rText + pLeft->GetValueString( eCoreUnit, ePresUnit, &rIntl ) + cpDelimTmp;
+ rText += pLeft->GetValueString( eCoreUnit, ePresUnit, &rIntl ) + cpDelimTmp;
}
if ( pRight )
{
- rText = rText + pRight->GetValueString( eCoreUnit, ePresUnit, &rIntl ) + cpDelimTmp;
+ rText += pRight->GetValueString( eCoreUnit, ePresUnit, &rIntl ) + cpDelimTmp;
}
}
rText += GetMetricText( static_cast<long>(nTopDist), eCoreUnit, ePresUnit, &rIntl );
if( nTopDist != nBottomDist || nTopDist != nLeftDist ||
nTopDist != nRightDist )
{
- rText = rText +
- cpDelimTmp +
+ rText += cpDelimTmp +
GetMetricText( static_cast<long>(nBottomDist), eCoreUnit,
ePresUnit, &rIntl ) +
cpDelimTmp +
@@ -1767,29 +1759,25 @@ bool SvxBoxItem::GetPresentation
{
if ( pTop )
{
- rText = rText +
- EditResId(RID_SVXITEMS_BORDER_TOP) +
+ rText += EditResId(RID_SVXITEMS_BORDER_TOP) +
pTop->GetValueString( eCoreUnit, ePresUnit, &rIntl, true ) +
cpDelimTmp;
}
if ( pBottom )
{
- rText = rText +
- EditResId(RID_SVXITEMS_BORDER_BOTTOM) +
+ rText += EditResId(RID_SVXITEMS_BORDER_BOTTOM) +
pBottom->GetValueString( eCoreUnit, ePresUnit, &rIntl, true ) +
cpDelimTmp;
}
if ( pLeft )
{
- rText = rText +
- EditResId(RID_SVXITEMS_BORDER_LEFT) +
+ rText += EditResId(RID_SVXITEMS_BORDER_LEFT) +
pLeft->GetValueString( eCoreUnit, ePresUnit, &rIntl, true ) +
cpDelimTmp;
}
if ( pRight )
{
- rText = rText +
- EditResId(RID_SVXITEMS_BORDER_RIGHT) +
+ rText += EditResId(RID_SVXITEMS_BORDER_RIGHT) +
pRight->GetValueString( eCoreUnit, ePresUnit, &rIntl, true ) +
cpDelimTmp;
}
@@ -1800,15 +1788,13 @@ bool SvxBoxItem::GetPresentation
if( nTopDist == nBottomDist && nTopDist == nLeftDist &&
nTopDist == nRightDist )
{
- rText = rText +
- GetMetricText( static_cast<long>(nTopDist), eCoreUnit,
+ rText += GetMetricText( static_cast<long>(nTopDist), eCoreUnit,
ePresUnit, &rIntl ) +
" " + EditResId(GetMetricId(ePresUnit));
}
else
{
- rText = rText +
- EditResId(RID_SVXITEMS_BORDER_TOP) +
+ rText += EditResId(RID_SVXITEMS_BORDER_TOP) +
GetMetricText( static_cast<long>(nTopDist), eCoreUnit,
ePresUnit, &rIntl ) +
" " + EditResId(GetMetricId(ePresUnit)) +
diff --git a/editeng/source/items/optitems.cxx b/editeng/source/items/optitems.cxx
index 1186192c088b..aeeb15314e01 100644
--- a/editeng/source/items/optitems.cxx
+++ b/editeng/source/items/optitems.cxx
@@ -56,8 +56,7 @@ bool SfxHyphenRegionItem::GetPresentation
const IntlWrapper&
) const
{
- rText = rText +
- EditResId(RID_SVXITEMS_HYPHEN_MINLEAD).replaceAll("%1", OUString::number(nMinLead)) +
+ rText += EditResId(RID_SVXITEMS_HYPHEN_MINLEAD).replaceAll("%1", OUString::number(nMinLead)) +
"," +
EditResId(RID_SVXITEMS_HYPHEN_MINTRAIL).replaceAll("%1", OUString::number(nMinTrail));
return true;
diff --git a/editeng/source/items/paraitem.cxx b/editeng/source/items/paraitem.cxx
index ed159e14ed08..983f210dec3a 100644
--- a/editeng/source/items/paraitem.cxx
+++ b/editeng/source/items/paraitem.cxx
@@ -660,7 +660,7 @@ bool SvxHyphenZoneItem::GetPresentation
if ( bPageEnd )
pId = RID_SVXITEMS_PAGE_END_TRUE;
- rText = rText + EditResId(pId) + cpDelimTmp +
+ rText += EditResId(pId) + cpDelimTmp +
OUString::number( nMinLead ) + cpDelimTmp +
OUString::number( nMinTrail ) + cpDelimTmp +
OUString::number( nMaxHyphens );
@@ -677,8 +677,7 @@ bool SvxHyphenZoneItem::GetPresentation
if ( bPageEnd )
pId = RID_SVXITEMS_PAGE_END_TRUE;
- rText = rText +
- EditResId(pId) +
+ rText += EditResId(pId) +
cpDelimTmp +
EditResId(RID_SVXITEMS_HYPHEN_MINLEAD).replaceAll("%1", OUString::number(nMinLead)) +
cpDelimTmp +
diff --git a/editeng/source/items/textitem.cxx b/editeng/source/items/textitem.cxx
index ffb4e63e2e21..82889dc91be7 100644
--- a/editeng/source/items/textitem.cxx
+++ b/editeng/source/items/textitem.cxx
@@ -965,7 +965,7 @@ bool SvxTextLineItem::GetPresentation
{
rText = GetValueTextByPos( GetValue() );
if( !mColor.GetTransparency() )
- rText = rText + OUString(cpDelim) + ::GetColorString( mColor );
+ rText += OUString(cpDelim) + ::GetColorString( mColor );
return true;
}
@@ -1540,8 +1540,7 @@ bool SvxKerningItem::GetPresentation
if (pId)
rText += EditResId(pId);
- rText = rText +
- GetMetricText( static_cast<long>(GetValue()), eCoreUnit, MapUnit::MapPoint, &rIntl ) +
+ rText += GetMetricText( static_cast<long>(GetValue()), eCoreUnit, MapUnit::MapPoint, &rIntl ) +
" " + EditResId(GetMetricId(MapUnit::MapPoint));
return true;
}
@@ -1721,7 +1720,7 @@ bool SvxEscapementItem::GetPresentation
if( DFLT_ESC_AUTO_SUPER == nEsc || DFLT_ESC_AUTO_SUB == nEsc )
rText += EditResId(RID_SVXITEMS_ESCAPEMENT_AUTO);
else
- rText = rText + OUString::number( nEsc ) + "%";
+ rText += OUString::number( nEsc ) + "%";
}
return true;
}
diff --git a/editeng/source/rtf/svxrtf.cxx b/editeng/source/rtf/svxrtf.cxx
index f78fcb891454..7f5fdd34ac6a 100644
--- a/editeng/source/rtf/svxrtf.cxx
+++ b/editeng/source/rtf/svxrtf.cxx
@@ -559,7 +559,7 @@ void SvxRTFParser::ReadFontTable()
{
// All data from the font is available, so off to the table
if (!sAltNm.isEmpty())
- sFntNm = sFntNm + ";" + sAltNm;
+ sFntNm += ";" + sAltNm;
pFont->SetFamilyName( sFntNm );
m_FontTable.insert(std::make_pair(nInsFontNo, std::move(pFont)));
diff --git a/helpcompiler/inc/HelpCompiler.hxx b/helpcompiler/inc/HelpCompiler.hxx
index 2b4a6bad049d..49a22205cae7 100644
--- a/helpcompiler/inc/HelpCompiler.hxx
+++ b/helpcompiler/inc/HelpCompiler.hxx
@@ -108,7 +108,7 @@ namespace fs
{
OString tmp(in);
OUString ustrSystemPath(OStringToOUString(tmp, getThreadTextEncoding()));
- data = data + ustrSystemPath;
+ data += ustrSystemPath;
}
void append(const std::string &in) { append(in.c_str()); }
};
diff --git a/include/vcl/weld.hxx b/include/vcl/weld.hxx
index be9cf23f0779..33dadea0cc03 100644
--- a/include/vcl/weld.hxx
+++ b/include/vcl/weld.hxx
@@ -1857,7 +1857,7 @@ public:
sal_Int32 nIdx = m_sHelpRoot.lastIndexOf('.');
if (nIdx != -1)
m_sHelpRoot = m_sHelpRoot.copy(0, nIdx);
- m_sHelpRoot = m_sHelpRoot + OString('/');
+ m_sHelpRoot += OString('/');
}
virtual std::unique_ptr<MessageDialog> weld_message_dialog(const OString& id,
bool bTakeOwnership = true)
diff --git a/lotuswordpro/source/filter/xfilter/xfstylecont.cxx b/lotuswordpro/source/filter/xfilter/xfstylecont.cxx
index 8e2a675a3281..2d8f9139aaaa 100644
--- a/lotuswordpro/source/filter/xfilter/xfstylecont.cxx
+++ b/lotuswordpro/source/filter/xfilter/xfstylecont.cxx
@@ -113,7 +113,7 @@ IXFStyleRet XFStyleContainer::AddStyle(std::unique_ptr<IXFStyle> pStyle)
//for name conflict
if(FindStyle( name))
{
- name = name + OUString::number(m_aStyles.size()+1);
+ name += OUString::number(m_aStyles.size()+1);
pStyle->SetStyleName(name);
}
}
diff --git a/odk/examples/DevelopersGuide/Database/DriverSkeleton/SConnection.cxx b/odk/examples/DevelopersGuide/Database/DriverSkeleton/SConnection.cxx
index 1734031e11d9..775fe6cef1db 100644
--- a/odk/examples/DevelopersGuide/Database/DriverSkeleton/SConnection.cxx
+++ b/odk/examples/DevelopersGuide/Database/DriverSkeleton/SConnection.cxx
@@ -109,12 +109,12 @@ void OConnection::construct(const ::rtl::OUString& url,const Sequence< PropertyV
else if(pBegin->Name.equalsAscii(pUser))
{
pBegin->Value >>= aUID;
- aDSN = aDSN + ";UID=" + aUID;
+ aDSN += ";UID=" + aUID;
}
else if(pBegin->Name.equalsAscii(pPwd))
{
pBegin->Value >>= aPWD;
- aDSN = aDSN + ";PWD=" + aPWD;
+ aDSN += ";PWD=" + aPWD;
}
else if(pBegin->Name.equalsAscii(pUseCatalog))
{
diff --git a/reportdesign/source/filter/xml/xmlExport.cxx b/reportdesign/source/filter/xml/xmlExport.cxx
index b548f991dcc6..2f00950b265c 100644
--- a/reportdesign/source/filter/xml/xmlExport.cxx
+++ b/reportdesign/source/filter/xml/xmlExport.cxx
@@ -1535,7 +1535,7 @@ void ORptExport::exportGroupsExpressionAsFunction(const Reference< XGroups>& _xG
// so we need to expand the formula of sCountName
sPrefix = " + 1) / " + OUString::number(xGroup->getGroupInterval());
sFunctionName = sFunction + "_" + sExpression;
- sFunction = sFunction + "(";
+ sFunction += "(";
sInitialFormula = "rpt:0";
}
break;
diff --git a/sax/qa/cppunit/xmlimport.cxx b/sax/qa/cppunit/xmlimport.cxx
index 0762c249bdd4..63415c8b0b39 100644
--- a/sax/qa/cppunit/xmlimport.cxx
+++ b/sax/qa/cppunit/xmlimport.cxx
@@ -156,15 +156,15 @@ void SAL_CALL TestDocumentHandler::startElement( const OUString& aName, const Re
OUString sAttrValue = xAttribs->getValueByIndex(i);
OUString sAttrName = canonicalform(xAttribs->getNameByIndex(i), sAttrValue, false);
if (!sAttrName.isEmpty())
- sAttributes = sAttributes + sAttrName + sAttrValue;
+ sAttributes += sAttrName + sAttrValue;
}
- m_aStr = m_aStr + canonicalform(aName, "", true) + sAttributes;
+ m_aStr += canonicalform(aName, "", true) + sAttributes;
}
void SAL_CALL TestDocumentHandler::endElement( const OUString& aName )
{
- m_aStr = m_aStr + canonicalform(aName, "", true);
+ m_aStr += canonicalform(aName, "", true);
sal_uInt16 nPopQty = m_aCountStack.top();
for (sal_uInt16 i=0; i<nPopQty; i++)
m_aNamespaceStack.pop_back();
@@ -174,19 +174,19 @@ void SAL_CALL TestDocumentHandler::endElement( const OUString& aName )
void SAL_CALL TestDocumentHandler::characters( const OUString& aChars )
{
- m_aStr = m_aStr + aChars;
+ m_aStr += aChars;
}
void SAL_CALL TestDocumentHandler::ignorableWhitespace( const OUString& aWhitespaces )
{
- m_aStr = m_aStr + aWhitespaces;
+ m_aStr += aWhitespaces;
}
void SAL_CALL TestDocumentHandler::processingInstruction( const OUString& aTarget, const OUString& aData )
{
- m_aStr = m_aStr + aTarget + aData;
+ m_aStr += aTarget + aData;
}
diff --git a/sc/source/core/data/attrib.cxx b/sc/source/core/data/attrib.cxx
index 5f1517250998..94a8922e334e 100644
--- a/sc/source/core/data/attrib.cxx
+++ b/sc/source/core/data/attrib.cxx
@@ -610,7 +610,7 @@ bool ScPageScaleToItem::GetPresentation(
OUString aName( ScResId( STR_SCATTR_PAGE_SCALETO ) );
OUString aValue( ScResId( STR_SCATTR_PAGE_SCALE_WIDTH ) );
lclAppendScalePageCount( aValue, mnWidth );
- aValue = aValue + ", " + ScResId( STR_SCATTR_PAGE_SCALE_HEIGHT );
+ aValue += ", " + ScResId( STR_SCATTR_PAGE_SCALE_HEIGHT );
lclAppendScalePageCount( aValue, mnHeight );
switch( ePres )
diff --git a/sc/source/core/data/docpool.cxx b/sc/source/core/data/docpool.cxx
index 95597eb2a61b..ce98a6ea45a5 100644
--- a/sc/source/core/data/docpool.cxx
+++ b/sc/source/core/data/docpool.cxx
@@ -429,7 +429,7 @@ static bool lcl_HFPresentation
aText = EditResId(RID_SVXITEMS_LRSPACE_LEFT);
if ( 100 != nPropLeftMargin )
{
- aText = aText + unicode::formatPercent(nPropLeftMargin,
+ aText += unicode::formatPercent(nPropLeftMargin,
Application::GetSettings().GetUILanguageTag());
}
else
@@ -443,7 +443,7 @@ static bool lcl_HFPresentation
EditResId(RID_SVXITEMS_LRSPACE_RIGHT);
if ( 100 != nPropRightMargin )
{
- aText = aText + unicode::formatPercent(nPropLeftMargin,
+ aText += unicode::formatPercent(nPropLeftMargin,
Application::GetSettings().GetUILanguageTag());
}
else
@@ -462,7 +462,7 @@ static bool lcl_HFPresentation
if ( aText.getLength() )
{
- rText = rText + aText + " + ";
+ rText += aText + " + ";
}
}
@@ -559,7 +559,7 @@ bool ScDocumentPool::GetPresentation(
if( nPercent )
{
rText = ScResId(STR_SCATTR_PAGE_SCALE) + aStrSep;
- rText = rText + unicode::formatPercent(nPercent,
+ rText += unicode::formatPercent(nPercent,
Application::GetSettings().GetUILanguageTag());
}
else
diff --git a/sc/source/core/opencl/formulagroupcl.cxx b/sc/source/core/opencl/formulagroupcl.cxx
index 8948160c4695..05deff486faa 100644
--- a/sc/source/core/opencl/formulagroupcl.cxx
+++ b/sc/source/core/opencl/formulagroupcl.cxx
@@ -2583,7 +2583,7 @@ public:
{
std::string t = "_" + mpCodeGen->BinFuncName();
for (const auto & rSubArgument : mvSubArguments)
- t = t + rSubArgument->DumpOpName();
+ t += rSubArgument->DumpOpName();
return t;
}
virtual void DumpInlineFun( std::set<std::string>& decls,
diff --git a/sc/source/filter/excel/xecontent.cxx b/sc/source/filter/excel/xecontent.cxx
index c61425647e59..fc354d0ba69b 100644
--- a/sc/source/filter/excel/xecontent.cxx
+++ b/sc/source/filter/excel/xecontent.cxx
@@ -1255,7 +1255,7 @@ OString createHexStringFromDigit(sal_uInt8 nDigit)
{
OString aString = OString::number( nDigit, 16 );
if(aString.getLength() == 1)
- aString = aString + OString::number(0);
+ aString += OString::number(0);
return aString;
}
diff --git a/sc/source/ui/app/inputhdl.cxx b/sc/source/ui/app/inputhdl.cxx
index e7090c108bbd..111c57d9f105 100644
--- a/sc/source/ui/app/inputhdl.cxx
+++ b/sc/source/ui/app/inputhdl.cxx
@@ -1592,7 +1592,7 @@ static OUString lcl_Calculate( const OUString& rFormula, ScDocument* pDoc, const
ScRange aTestRange;
if ( bColRowName || (aTestRange.Parse(rFormula) & ScRefFlags::VALID) )
- aValue = aValue + " ...";
+ aValue += " ...";
return aValue;
}
diff --git a/sc/source/ui/view/gridwin.cxx b/sc/source/ui/view/gridwin.cxx
index 06f74bab50bd..18c83970a895 100644
--- a/sc/source/ui/view/gridwin.cxx
+++ b/sc/source/ui/view/gridwin.cxx
@@ -2282,7 +2282,7 @@ void ScGridWindow::MouseButtonUp( const MouseEvent& rMEvt )
if ( aScRange.aStart == aScRange.aEnd )
{
// make sure there is a range selection string even for a single cell
- aAddr = aAddr + ":" + aAddr;
+ aAddr += ":" + aAddr;
}
//! SID_MARKAREA does not exist anymore ???
diff --git a/sd/source/ui/accessibility/AccessibleDocumentViewBase.cxx b/sd/source/ui/accessibility/AccessibleDocumentViewBase.cxx
index c3004e7c1d61..d0fc351dee60 100644
--- a/sd/source/ui/accessibility/AccessibleDocumentViewBase.cxx
+++ b/sd/source/ui/accessibility/AccessibleDocumentViewBase.cxx
@@ -691,7 +691,7 @@ uno::Any SAL_CALL AccessibleDocumentViewBase::getExtendedAttributes()
if (!layerAltText.isEmpty())
{
sName = " ";
- sDisplay = sDisplay + sName + layerAltText;
+ sDisplay += sName + layerAltText;
}
}
}
diff --git a/sd/source/ui/annotations/annotationmanager.cxx b/sd/source/ui/annotations/annotationmanager.cxx
index aa56364c8889..512415465307 100644
--- a/sd/source/ui/annotations/annotationmanager.cxx
+++ b/sd/source/ui/annotations/annotationmanager.cxx
@@ -155,7 +155,7 @@ OUString getAnnotationDateTimeString( const Reference< XAnnotation >& xAnnotatio
::tools::Time aTime( aDateTime );
if(aTime.GetTime() != 0)
- sRet = sRet + " " + rLocalData.getTime( aTime,false );
+ sRet += " " + rLocalData.getTime( aTime,false );
}
return sRet;
}
diff --git a/sd/source/ui/dlg/custsdlg.cxx b/sd/source/ui/dlg/custsdlg.cxx
index b18a6e05e5d2..dfb07394b904 100644
--- a/sd/source/ui/dlg/custsdlg.cxx
+++ b/sd/source/ui/dlg/custsdlg.cxx
@@ -182,14 +182,14 @@ void SdCustomShowDlg::SelectHdl(void const *p)
sal_Int32 nNum = 1;
if( nStrPos < 0 )
{
- aStr = aStr + " (" + aStrCopy + OUString::number( nNum ) + ")";
+ aStr += " (" + aStrCopy + OUString::number( nNum ) + ")";
nStrPos = aStr.indexOf( aStrCopy );
}
nStrPos = nStrPos + aStrCopy.getLength();
// that we do not access into the nirvana (--> endless loop)
if( nStrPos >= aStr.getLength() )
{
- aStr = aStr + " " + OUString::number( nNum );
+ aStr += " " + OUString::number( nNum );
}
// check name...
diff --git a/setup_native/source/win32/customactions/shellextensions/completeinstallpath.cxx b/setup_native/source/win32/customactions/shellextensions/completeinstallpath.cxx
index ee8c71369720..787041ed8f7e 100644
--- a/setup_native/source/win32/customactions/shellextensions/completeinstallpath.cxx
+++ b/setup_native/source/win32/customactions/shellextensions/completeinstallpath.cxx
@@ -104,7 +104,7 @@ extern "C" __declspec(dllexport) UINT __stdcall CompleteInstallPath( MSIHANDLE h
if ( oldVersionExists )
{
// Adding the new path content sOfficeDirHostname
- sInstallLocation = sInstallLocation + sOfficeDirHostname;
+ sInstallLocation += sOfficeDirHostname;
// Setting the new property value
MsiSetPropertyW(handle, L"INSTALLLOCATION", sInstallLocation.c_str());
}
diff --git a/sfx2/source/dialog/mailmodel.cxx b/sfx2/source/dialog/mailmodel.cxx
index 901c435f6d94..4957f7168c50 100644
--- a/sfx2/source/dialog/mailmodel.cxx
+++ b/sfx2/source/dialog/mailmodel.cxx
@@ -849,7 +849,7 @@ bool CreateFromAddress_Impl( OUString& rFrom )
{
if ( !rFrom.isEmpty() )
rFrom += " ";
- rFrom = rFrom + "<" + comphelper::string::strip(aEmailName, ' ') + ">";
+ rFrom += "<" + comphelper::string::strip(aEmailName, ' ') + ">";
}
else
rFrom.clear();
diff --git a/sfx2/source/dialog/versdlg.cxx b/sfx2/source/dialog/versdlg.cxx
index 2aece774b2cd..1901ce9b2c29 100644
--- a/sfx2/source/dialog/versdlg.cxx
+++ b/sfx2/source/dialog/versdlg.cxx
@@ -181,8 +181,8 @@ SfxVersionDialog::SfxVersionDialog(weld::Window* pParent, SfxViewFrame* pVwFrame
m_xVersionBox->grab_focus();
// set dialog title (filename or docinfo title)
- OUString sText = m_xDialog->get_title();
- sText = sText + " " + m_pViewFrame->GetObjectShell()->GetTitle();
+ OUString sText = m_xDialog->get_title() +
+ " " + m_pViewFrame->GetObjectShell()->GetTitle();
m_xDialog->set_title(sText);
Init_Impl();
@@ -441,8 +441,8 @@ SfxCmisVersionsDialog::SfxCmisVersionsDialog(weld::Window* pParent, SfxViewFrame
m_xVersionBox->grab_focus();
- OUString sText = m_xDialog->get_title();
- sText = sText + " " + m_pViewFrame->GetObjectShell()->GetTitle();
+ OUString sText = m_xDialog->get_title() +
+ " " + m_pViewFrame->GetObjectShell()->GetTitle();
m_xDialog->set_title(sText);
LoadVersions();
diff --git a/sfx2/source/doc/DocumentMetadataAccess.cxx b/sfx2/source/doc/DocumentMetadataAccess.cxx
index 1e5dca13b1e7..5574bcada016 100644
--- a/sfx2/source/doc/DocumentMetadataAccess.cxx
+++ b/sfx2/source/doc/DocumentMetadataAccess.cxx
@@ -150,7 +150,7 @@ uno::Reference<rdf::XURI> createBaseURI(
assert(!pkgURI.isEmpty());
if (!pkgURI.isEmpty() && !pkgURI.endsWith("/"))
{
- pkgURI = pkgURI + "/";
+ pkgURI += "/";
}
}
diff --git a/sfx2/source/doc/sfxbasemodel.cxx b/sfx2/source/doc/sfxbasemodel.cxx
index 46d6be1a7840..cc2aae255542 100644
--- a/sfx2/source/doc/sfxbasemodel.cxx
+++ b/sfx2/source/doc/sfxbasemodel.cxx
@@ -295,7 +295,7 @@ struct IMPL_SfxBaseModel_DataContainer : public ::sfx2::IModifiableDocument
OSL_ENSURE(!uri.isEmpty(), "GetDMA: empty uri?");
if (!uri.isEmpty() && !uri.endsWith("/"))
{
- uri = uri + "/";
+ uri += "/";
}
m_xDocumentMetadata = new ::sfx2::DocumentMetadataAccess(
diff --git a/sfx2/source/doc/templatedlg.cxx b/sfx2/source/doc/templatedlg.cxx
index db83da0140bd..90c8c4dda017 100644
--- a/sfx2/source/doc/templatedlg.cxx
+++ b/sfx2/source/doc/templatedlg.cxx
@@ -914,7 +914,7 @@ void SfxTemplateManagerDlg::OnTemplateImportCategory(const OUString& sCategory)
if (aTemplateList.isEmpty())
aTemplateList = rFile;
else
- aTemplateList = aTemplateList + "\n" + rFile;
+ aTemplateList += "\n" + rFile;
}
}
@@ -968,7 +968,7 @@ void SfxTemplateManagerDlg::OnTemplateExport()
if (aTemplateList.isEmpty())
aTemplateList = pItem->maTitle;
else
- aTemplateList = aTemplateList + "\n" + pItem->maTitle;
+ aTemplateList += "\n" + pItem->maTitle;
}
++i;
}
@@ -1001,7 +1001,7 @@ void SfxTemplateManagerDlg::OnTemplateExport()
if (aTemplateList.isEmpty())
aTemplateList = pItem->maTitle;
else
- aTemplateList = aTemplateList + "\n" + pItem->maTitle;
+ aTemplateList += "\n" + pItem->maTitle;
}
++i;
}
diff --git a/stoc/source/defaultregistry/defaultregistry.cxx b/stoc/source/defaultregistry/defaultregistry.cxx
index 71a637309c5a..f0a4d0ff18a3 100644
--- a/stoc/source/defaultregistry/defaultregistry.cxx
+++ b/stoc/source/defaultregistry/defaultregistry.cxx
@@ -827,7 +827,7 @@ sal_Bool SAL_CALL NestedKeyImpl::createLink( const OUString& aLinkName, const OU
throw InvalidRegistryException();
}
- resolvedName = resolvedName + aLinkName.copy(lastIndex);
+ resolvedName += aLinkName.copy(lastIndex);
}
else
{
@@ -882,7 +882,7 @@ void SAL_CALL NestedKeyImpl::deleteLink( const OUString& rLinkName )
throw InvalidRegistryException();
}
- resolvedName = resolvedName + rLinkName.copy(lastIndex);
+ resolvedName += rLinkName.copy(lastIndex);
}
else
{
@@ -925,7 +925,7 @@ OUString SAL_CALL NestedKeyImpl::getLinkTarget( const OUString& rLinkName )
throw InvalidRegistryException();
}
- resolvedName = resolvedName + rLinkName.copy(lastIndex);
+ resolvedName += rLinkName.copy(lastIndex);
}
else
{
diff --git a/stoc/source/implementationregistration/implreg.cxx b/stoc/source/implementationregistration/implreg.cxx
index c3d0f1e38820..730dd7febdd8 100644
--- a/stoc/source/implementationregistration/implreg.cxx
+++ b/stoc/source/implementationregistration/implreg.cxx
@@ -194,7 +194,7 @@ void prepareLink( const Reference < XSimpleRegistry > & xDest,
if (pShortName)
{
- linkRefName = linkRefName + link.copy(pShortName - pTmpName + 1);
+ linkRefName += link.copy(pShortName - pTmpName + 1);
linkName = link.copy(0, pShortName - pTmpName);
}
@@ -515,7 +515,7 @@ void prepareUserKeys(const Reference < XSimpleRegistry >& xDest,
OUString linkTarget = xKey->getLinkTarget(relativKey);
OUString linkName(xKey->getKeyName().copy(xUnoKey->getKeyName().getLength()));
- linkName = linkName + "/" + relativKey;
+ linkName += "/" + relativKey;
if (bRegister)
{
diff --git a/stoc/source/javavm/javavm.cxx b/stoc/source/javavm/javavm.cxx
index 71ffbb3fffb1..c6b3bd19dbf8 100644
--- a/stoc/source/javavm/javavm.cxx
+++ b/stoc/source/javavm/javavm.cxx
@@ -473,8 +473,7 @@ void getJavaPropsFromSafetySettings(
case 3: sVal = "none";
break;
}
- OUString sProperty("appletviewer.security.mode=");
- sProperty= sProperty + sVal;
+ OUString sProperty = "appletviewer.security.mode=" + sVal;
pjvm->pushProp(sProperty);
}
css::uno::Reference<css::registry::XRegistryKey> key_CheckSecurity= xRegistryRootKey->openKey(
@@ -484,9 +483,9 @@ void getJavaPropsFromSafetySettings(
bool val = static_cast<bool>(key_CheckSecurity->getLongValue());
OUString sProperty("stardiv.security.disableSecurity=");
if( val)
- sProperty= sProperty + "false";
+ sProperty += "false";
else
- sProperty= sProperty + "true";
+ sProperty += "true";
pjvm->pushProp( sProperty);
}
}
diff --git a/svx/qa/unit/customshapes.cxx b/svx/qa/unit/customshapes.cxx
index 4a1f3cb39c46..ad0e305bf6ca 100644
--- a/svx/qa/unit/customshapes.cxx
+++ b/svx/qa/unit/customshapes.cxx
@@ -380,14 +380,12 @@ CPPUNIT_TEST_FIXTURE(CustomshapesTest, testTdf115813_OOXML_XY_handle)
|| (abs(nDesiredX - nObservedX) == 100 && abs(nDesiredY - nObservedY) <= 1)
|| (abs(nDesiredX - nObservedX) <= 1 && abs(nDesiredY - nObservedY) <= 1)))
{
- sErrors += "\n";
- //sErrors += OUString(sal_Unicode(10));
- sErrors
- = sErrors + OUString::number(i) + " " + sShapeType + ": " + OUString::number(j);
- sErrors = sErrors + " X " + OUString::number(nDesiredX) + "|"
- + OUString::number(nObservedX);
- sErrors = sErrors + " Y " + OUString::number(nDesiredY) + "|"
- + OUString::number(nObservedY);
+ sErrors += "\n" +
+ //sErrors += OUString(sal_Unicode(10));
+ OUString::number(i) + " " + sShapeType + ": " + OUString::number(j)
+ + " X " + OUString::number(nDesiredX) + "|"
+ + OUString::number(nObservedX) + " Y " + OUString::number(nDesiredY)
+ + "|" + OUString::number(nObservedY);
}
}
}
diff --git a/svx/source/form/fmshimp.cxx b/svx/source/form/fmshimp.cxx
index 46547f22ad38..b17a794c79ff 100644
--- a/svx/source/form/fmshimp.cxx
+++ b/svx/source/form/fmshimp.cxx
@@ -2401,9 +2401,9 @@ IMPL_LINK(FmXFormShell, OnSearchContextRequest_Lock, FmSearchContext&, rfmscCont
// the cursor has a field matching the control source ?
if (xValidFormFields->hasByName(aName))
{
- strFieldList = strFieldList + aName + ";";
+ strFieldList += aName + ";";
- sFieldDisplayNames = sFieldDisplayNames +
+ sFieldDisplayNames +=
::comphelper::getString(xCurrentColModel->getPropertyValue(FM_PROP_LABEL)) +
";";
@@ -2431,10 +2431,10 @@ IMPL_LINK(FmXFormShell, OnSearchContextRequest_Lock, FmSearchContext&, rfmscCont
if (IsSearchableControl(xControl))
{
// all tests passed -> take along in the list
- strFieldList = strFieldList + sControlSource + ";";
+ strFieldList += sControlSource + ";";
// the label which should appear for the control :
- sFieldDisplayNames = sFieldDisplayNames +
+ sFieldDisplayNames +=
getLabelName(Reference< XPropertySet>(xControlModel, UNO_QUERY)) +
";";
diff --git a/svx/source/form/fmsrcimp.cxx b/svx/source/form/fmsrcimp.cxx
index 6634c981616a..fb01b66e9524 100644
--- a/svx/source/form/fmsrcimp.cxx
+++ b/svx/source/form/fmsrcimp.cxx
@@ -805,7 +805,7 @@ void FmSearchEngine::SearchNextImpl()
strSearchExpression = "*" + strSearchExpression + "*";
break;
case MATCHING_BEGINNING :
- strSearchExpression = strSearchExpression + "*";
+ strSearchExpression += "*";
break;
case MATCHING_END :
strSearchExpression = "*" + strSearchExpression;
diff --git a/svx/source/svdraw/presetooxhandleadjustmentrelations.cxx b/svx/source/svdraw/presetooxhandleadjustmentrelations.cxx
index e334bb06bdc2..0bf47ad64321 100644
--- a/svx/source/svdraw/presetooxhandleadjustmentrelations.cxx
+++ b/svx/source/svdraw/presetooxhandleadjustmentrelations.cxx
@@ -316,7 +316,7 @@ void PresetOOXHandleAdj::GetOOXHandleAdjRelation(
rSecondAdjValueIndex = -1;
if (sFullOOXShapeName.startsWith("ooxml-", &sKey))
{
- sKey = sKey + "_" + OUString::number(nHandleIndex);
+ sKey += "_" + OUString::number(nHandleIndex);
HandleAdjRelHashMap::const_iterator aHashIter(s_HashMap.find(sKey));
if (aHashIter != s_HashMap.end())
sValue = (*aHashIter).second;
diff --git a/svx/source/svdraw/svdattr.cxx b/svx/source/svdraw/svdattr.cxx
index c9977fd44ff1..0a757b343c86 100644
--- a/svx/source/svdraw/svdattr.cxx
+++ b/svx/source/svdraw/svdattr.cxx
@@ -633,7 +633,7 @@ bool SdrFractionItem::GetPresentation(
if(nDiv != 1)
{
- rText = rText + "/" + OUString::number(nDiv);
+ rText += "/" + OUString::number(nDiv);
}
}
else
diff --git a/sw/qa/extras/globalfilter/globalfilter.cxx b/sw/qa/extras/globalfilter/globalfilter.cxx
index 0d60df55210b..b6af6e5ea1dc 100644
--- a/sw/qa/extras/globalfilter/globalfilter.cxx
+++ b/sw/qa/extras/globalfilter/globalfilter.cxx
@@ -780,7 +780,7 @@ void Test::testSkipImages()
for (auto const & rFilterNamePair : aFilterNames)
{
bool bSkipImages = !rFilterNamePair.second.isEmpty();
- OString sFailedMessage = OString("Failed on filter: ") + rFilterNamePair.first.toUtf8();
+ OString sFailedMessage = OStringLiteral("Failed on filter: ") + rFilterNamePair.first.toUtf8();
if (mxComponent.is())
mxComponent->dispose();
@@ -792,7 +792,7 @@ void Test::testSkipImages()
{ "FilterOptions", uno::Any(rFilterNamePair.second) }
}));
mxComponent = loadFromDesktop(m_directories.getURLFromSrc(rFilterNamePair.first), "com.sun.star.text.TextDocument", args);
- sFailedMessage = sFailedMessage + " - " + rFilterNamePair.second.toUtf8();
+ sFailedMessage += " - " + rFilterNamePair.second.toUtf8();
}
else
{
diff --git a/sw/source/core/access/acctable.cxx b/sw/source/core/access/acctable.cxx
index 7094aeecbb01..6bb085d6a3bc 100644
--- a/sw/source/core/access/acctable.cxx
+++ b/sw/source/core/access/acctable.cxx
@@ -845,7 +845,7 @@ OUString SAL_CALL SwAccessibleTable::getAccessibleRowDescription(
uno::Reference< XAccessibleText > xChildText( xChild, uno::UNO_QUERY );
if ( xChildText.is() )
{
- sRowDesc = sRowDesc + xChildText->getText();
+ sRowDesc += xChildText->getText();
}
}
}
@@ -878,7 +878,7 @@ OUString SAL_CALL SwAccessibleTable::getAccessibleColumnDescription(
uno::Reference< XAccessibleText > xChildText( xChild, uno::UNO_QUERY );
if ( xChildText.is() )
{
- sColumnDesc = sColumnDesc + xChildText->getText();
+ sColumnDesc += xChildText->getText();
}
}
}
diff --git a/sw/source/filter/ww8/ww8par5.cxx b/sw/source/filter/ww8/ww8par5.cxx
index e39d0416c431..89335622da0f 100644
--- a/sw/source/filter/ww8/ww8par5.cxx
+++ b/sw/source/filter/ww8/ww8par5.cxx
@@ -3521,7 +3521,7 @@ eF_ResT SwWW8ImplReader::Read_F_Hyperlink( WW8FieldDesc* /*pF*/, OUString& rStr
OSL_ENSURE(!sURL.isEmpty() || !sMark.isEmpty(), "WW8: Empty URL");
if( !sMark.isEmpty() )
- sURL = sURL + "#" + sMark;
+ sURL += "#" + sMark;
SwFormatINetFormat aURL(sURL, sTarget);
// If on loading TOC field, change the default style into the "index link"
diff --git a/sw/source/uibase/utlui/attrdesc.cxx b/sw/source/uibase/utlui/attrdesc.cxx
index f6f88080bab2..7c5d9bee918e 100644
--- a/sw/source/uibase/utlui/attrdesc.cxx
+++ b/sw/source/uibase/utlui/attrdesc.cxx
@@ -99,9 +99,8 @@ bool SwFormatCharFormat::GetPresentation
if ( pCharFormat )
{
OUString aStr;
- rText = SwResId( STR_CHARFMT );
pCharFormat->GetPresentation( ePres, eCoreUnit, ePresUnit, aStr );
- rText = rText + "(" + aStr + ")";
+ rText = SwResId( STR_CHARFMT ) + "(" + aStr + ")";
}
else
rText = SwResId( STR_NO_CHARFMT );
@@ -158,8 +157,7 @@ bool SwFormatDrop::GetPresentation
{
rText = OUString::number( GetChars() ) + " ";
}
- rText = rText +
- SwResId( STR_DROP_OVER ) +
+ rText += SwResId( STR_DROP_OVER ) +
" " +
OUString::number( GetLines() ) +
" " +
@@ -229,22 +227,22 @@ bool SwFormatFrameSize::GetPresentation
rText = SwResId( STR_FRM_WIDTH ) + " ";
if ( GetWidthPercent() )
{
- rText = rText + unicode::formatPercent(GetWidthPercent(),
+ rText += unicode::formatPercent(GetWidthPercent(),
Application::GetSettings().GetUILanguageTag());
}
else
{
- rText = rText + ::GetMetricText( GetWidth(), eCoreUnit, ePresUnit, &rIntl ) +
+ rText += ::GetMetricText( GetWidth(), eCoreUnit, ePresUnit, &rIntl ) +
" " + ::EditResId( ::GetMetricId( ePresUnit ) );
}
if ( ATT_VAR_SIZE != GetHeightSizeType() )
{
const char* pId = ATT_FIX_SIZE == m_eFrameHeightType ?
STR_FRM_FIXEDHEIGHT : STR_FRM_MINHEIGHT;
- rText = rText + ", " + SwResId(pId) + " ";
+ rText += ", " + SwResId(pId) + " ";
if ( GetHeightPercent() )
{
- rText = rText + unicode::formatPercent(GetHeightPercent(),
+ rText += unicode::formatPercent(GetHeightPercent(),
Application::GetSettings().GetUILanguageTag());
}
else
@@ -327,7 +325,7 @@ bool SwFormatSurround::GetPresentation
if ( IsAnchorOnly() )
{
- rText = rText + " " + SwResId( STR_SURROUND_ANCHORONLY );
+ rText += " " + SwResId( STR_SURROUND_ANCHORONLY );
}
return true;
}
@@ -348,7 +346,7 @@ bool SwFormatVertOrient::GetPresentation
{
case text::VertOrientation::NONE:
{
- rText = rText + SwResId( STR_POS_Y ) + " " +
+ rText += SwResId( STR_POS_Y ) + " " +
::GetMetricText( GetPos(), eCoreUnit, ePresUnit, &rIntl ) +
" " + EditResId( ::GetMetricId( ePresUnit ) );
}
@@ -394,7 +392,7 @@ bool SwFormatHoriOrient::GetPresentation
{
case text::HoriOrientation::NONE:
{
- rText = rText + SwResId( STR_POS_X ) + " " +
+ rText += SwResId( STR_POS_X ) + " " +
::GetMetricText( GetPos(), eCoreUnit, ePresUnit, &rIntl ) +
" " + EditResId( ::GetMetricId( ePresUnit ) );
}
@@ -489,7 +487,7 @@ bool SwFormatCol::GetPresentation
if ( COLADJ_NONE != GetLineAdj() )
{
const long nWdth = static_cast<long>(GetLineWidth());
- rText = rText + " " + SwResId( STR_LINE_WIDTH ) + " " +
+ rText += " " + SwResId( STR_LINE_WIDTH ) + " " +
::GetMetricText( nWdth, eCoreUnit,
MapUnit::MapPoint, &rIntl );
}
@@ -517,13 +515,13 @@ bool SwFormatURL::GetPresentation
{
if ( m_pMap )
rText += " - ";
- rText = rText + "URL: " + m_sURL;
+ rText += "URL: " + m_sURL;
if ( m_bIsServerMap )
rText += " (Server-Map)";
}
if ( !m_sTargetFrameName.isEmpty() )
{
- rText = rText + ", Target: " + m_sTargetFrameName;
+ rText += ", Target: " + m_sTargetFrameName;
}
return true;
}
@@ -628,7 +626,7 @@ bool SwFormatLineNumber::GetPresentation
rText += SwResId(STR_DONTLINECOUNT);
if ( GetStartValue() )
{
- rText = rText + " " + SwResId(STR_LINCOUNT_START) +
+ rText += " " + SwResId(STR_LINCOUNT_START) +
OUString::number( GetStartValue() );
}
return true;
@@ -706,7 +704,7 @@ bool SwRotationGrf::GetPresentation(
rText = SwResId( STR_ROTATION );
else if( rText.getLength() )
rText.clear();
- rText = rText + OUString::number( GetValue() ) + "\xB0";
+ rText += OUString::number( GetValue() ) + "\xB0";
return true;
}
@@ -718,8 +716,8 @@ bool SwLuminanceGrf::GetPresentation(
rText = SwResId( STR_LUMINANCE );
else if( rText.getLength() )
rText.clear();
- rText = rText + unicode::formatPercent(GetValue(),
- Application::GetSettings().GetUILanguageTag());
+ rText += unicode::formatPercent(GetValue(),
+ Application::GetSettings().GetUILanguageTag());
return true;
}
@@ -731,8 +729,8 @@ bool SwContrastGrf::GetPresentation(
rText = SwResId( STR_CONTRAST );
else if( rText.getLength() )
rText.clear();
- rText = rText + unicode::formatPercent(GetValue(),
- Application::GetSettings().GetUILanguageTag());
+ rText += unicode::formatPercent(GetValue(),
+ Application::GetSettings().GetUILanguageTag());
return true;
}
@@ -757,8 +755,8 @@ bool SwChannelGrf::GetPresentation(
}
else if( rText.getLength() )
rText.clear();
- rText = rText + unicode::formatPercent(GetValue(),
- Application::GetSettings().GetUILanguageTag());
+ rText += unicode::formatPercent(GetValue(),
+ Application::GetSettings().GetUILanguageTag());
return true;
}
@@ -796,7 +794,7 @@ bool SwTransparencyGrf::GetPresentation(
rText = SwResId( STR_TRANSPARENCY );
else if( rText.getLength() )
rText.clear();
- rText = rText + unicode::formatPercent(GetValue(),
+ rText += unicode::formatPercent(GetValue(),
Application::GetSettings().GetUILanguageTag());
return true;
}
diff --git a/test/source/bootstrapfixture.cxx b/test/source/bootstrapfixture.cxx
index 6e06d352dec1..6279ec8d8633 100644
--- a/test/source/bootstrapfixture.cxx
+++ b/test/source/bootstrapfixture.cxx
@@ -211,8 +211,7 @@ void test::BootstrapFixture::validate(const OUString& rPath, test::ValidationFor
sal_Int32 nStartOfNumber = nIndex + std::strlen("Grand total of errors in submitted package: ");
OUString aNumber = aContentOUString.copy(nStartOfNumber);
sal_Int32 nErrors = aNumber.toInt32();
- OString aMsg("validation error in OOXML export: Errors: ");
- aMsg = aMsg + OString::number(nErrors);
+ OString aMsg = "validation error in OOXML export: Errors: " + OString::number(nErrors);
if(nErrors)
{
SAL_WARN("test", aContentOUString);
diff --git a/vcl/source/gdi/mtfxmldump.cxx b/vcl/source/gdi/mtfxmldump.cxx
index c5d2dfaa4c6c..efc8dfc5b999 100644
--- a/vcl/source/gdi/mtfxmldump.cxx
+++ b/vcl/source/gdi/mtfxmldump.cxx
@@ -68,7 +68,7 @@ OUString collectPushFlags(PushFlags nFlags)
aString = aStrings[0];
for (size_t i = 1; i < aStrings.size(); ++i)
{
- aString = aString + ", " + aStrings[i];
+ aString += ", " + aStrings[i];
}
return aString;
}
@@ -121,7 +121,7 @@ OUString convertDrawTextFlagsToString(DrawTextFlags eDrawTextFlags)
aString = aStrings[0];
for (size_t i = 1; i < aStrings.size(); ++i)
{
- aString = aString + " " + aStrings[i];
+ aString += " " + aStrings[i];
}
return aString;
};
diff --git a/vcl/source/uitest/logger.cxx b/vcl/source/uitest/logger.cxx
index 861cc4f4fb8e..2ab6026de4af 100644
--- a/vcl/source/uitest/logger.cxx
+++ b/vcl/source/uitest/logger.cxx
@@ -256,11 +256,11 @@ void UITestLogger::logKeyInput(VclPtr<vcl::Window> const& xUIElement, const KeyE
}
if (aParentID.isEmpty())
{
- aContent = aContent + "Type on '" + rID + "' " + aKeyCode;
+ aContent += "Type on '" + rID + "' " + aKeyCode;
}
else
{
- aContent = aContent + "Type on '" + rID + "' " + aKeyCode + " from " + aParentID;
+ aContent += "Type on '" + rID + "' " + aKeyCode + " from " + aParentID;
}
}
else if (pUIObject->get_type() == "SwEditWinUIObject" && rID == "writer_edit")
diff --git a/vcl/source/window/builder.cxx b/vcl/source/window/builder.cxx
index 87b6a740502f..70641bf724ee 100644
--- a/vcl/source/window/builder.cxx
+++ b/vcl/source/window/builder.cxx
@@ -429,7 +429,7 @@ VclBuilder::VclBuilder(vcl::Window* pParent, const OUString& sUIDir, const OUStr
sal_Int32 nIdx = m_sHelpRoot.lastIndexOf('.');
if (nIdx != -1)
m_sHelpRoot = m_sHelpRoot.copy(0, nIdx);
- m_sHelpRoot = m_sHelpRoot + OString('/');
+ m_sHelpRoot += OString('/');
OUString sUri = sUIDir + sUIFile;
diff --git a/vcl/source/window/tabdlg.cxx b/vcl/source/window/tabdlg.cxx
index 1a308f12c7f7..47212cb7fc8a 100644
--- a/vcl/source/window/tabdlg.cxx
+++ b/vcl/source/window/tabdlg.cxx
@@ -252,7 +252,7 @@ std::vector<OString> TabDialog::getAllPageUIXMLDescriptions() const
{
// add the PageId to be able to detect the correct tabPage in
// selectPageByUIXMLDescription below
- aNewName = aNewName + "|" + OString::number(nPageId);
+ aNewName += "|" + OString::number(nPageId);
}
aRetval.push_back(aNewName);
diff --git a/vcl/unx/generic/printer/cupsmgr.cxx b/vcl/unx/generic/printer/cupsmgr.cxx
index ea4dd308d58e..a6d1ccbad7bb 100644
--- a/vcl/unx/generic/printer/cupsmgr.cxx
+++ b/vcl/unx/generic/printer/cupsmgr.cxx
@@ -872,8 +872,8 @@ bool CUPSManager::endSpool( const OUString& rPrintername, const OUString& rJobTi
" option " << pOptions[n].name << "=" << pOptions[n].value);
#if OSL_DEBUG_LEVEL > 1
OString aCmd( "cp " );
- aCmd = aCmd + it->second.getStr();
- aCmd = aCmd + OString( " $HOME/cupsprint.ps" );
+ aCmd += it->second.getStr();
+ aCmd += OString( " $HOME/cupsprint.ps" );
system( aCmd.getStr() );
#endif
diff --git a/vcl/unx/gtk3/gtk3gtkinst.cxx b/vcl/unx/gtk3/gtk3gtkinst.cxx
index 9dd62384c2da..59066bde6502 100644
--- a/vcl/unx/gtk3/gtk3gtkinst.cxx
+++ b/vcl/unx/gtk3/gtk3gtkinst.cxx
@@ -11675,7 +11675,7 @@ public:
sal_Int32 nIdx = m_sHelpRoot.lastIndexOf('.');
if (nIdx != -1)
m_sHelpRoot = m_sHelpRoot.copy(0, nIdx);
- m_sHelpRoot = m_sHelpRoot + OUString('/');
+ m_sHelpRoot += OUString('/');
m_aUtf8HelpRoot = OUStringToOString(m_sHelpRoot, RTL_TEXTENCODING_UTF8);
m_aIconTheme = Application::GetSettings().GetStyleSettings().DetermineIconTheme();
m_aUILang = Application::GetSettings().GetUILanguageTag().getBcp47();
diff --git a/vcl/unx/gtk3/gtk3gtksalmenu.cxx b/vcl/unx/gtk3/gtk3gtksalmenu.cxx
index 5431ea4010ac..6fb544de2642 100644
--- a/vcl/unx/gtk3/gtk3gtksalmenu.cxx
+++ b/vcl/unx/gtk3/gtk3gtksalmenu.cxx
@@ -31,9 +31,9 @@ static bool bUnityMode = false;
*/
static gchar* GetCommandForItem(GtkSalMenu* pParentMenu, sal_uInt16 nItemId)
{
- OString aCommand("window-");
- aCommand = aCommand + OString::number(reinterpret_cast<unsigned long>(pParentMenu));
- aCommand = aCommand + "-" + OString::number(nItemId);
+ OString aCommand = "window-" +
+ OString::number(reinterpret_cast<unsigned long>(pParentMenu)) +
+ "-" + OString::number(nItemId);
return g_strdup(aCommand.getStr());
}
diff --git a/writerfilter/source/dmapper/DomainMapper_Impl.cxx b/writerfilter/source/dmapper/DomainMapper_Impl.cxx
index 6f6f74457486..f69b53fbf2fe 100644
--- a/writerfilter/source/dmapper/DomainMapper_Impl.cxx
+++ b/writerfilter/source/dmapper/DomainMapper_Impl.cxx
@@ -5170,7 +5170,7 @@ void DomainMapper_Impl::SetFieldResult(OUString const& rResult)
OUString titleStr;
uno::Any aValue(propertyVal.Value);
aValue >>= titleStr;
- titleStr = titleStr + rResult;
+ titleStr += rResult;
propertyVal.Value <<= titleStr;
aValues[nTitleFoundIndex] = propertyVal;
}
diff --git a/xmlhelp/source/treeview/tvread.cxx b/xmlhelp/source/treeview/tvread.cxx
index 6a6ea52552b9..f0331dea7a29 100644
--- a/xmlhelp/source/treeview/tvread.cxx
+++ b/xmlhelp/source/treeview/tvread.cxx
@@ -665,7 +665,7 @@ ConfigData TVChildTarget::init( const Reference< XComponentContext >& xContext )
locale = "en-US";
ret = "en";
}
- url = url + ret;
+ url += ret;
// first of all, try do determine whether there are any *.tree files present
diff --git a/xmloff/source/text/txtprhdl.cxx b/xmloff/source/text/txtprhdl.cxx
index 0bcc3d3ea8ba..32644e5f7f1c 100644
--- a/xmloff/source/text/txtprhdl.cxx
+++ b/xmloff/source/text/txtprhdl.cxx
@@ -546,7 +546,7 @@ bool XMLFrameProtectPropHdl_Impl::exportXML(
}
else
{
- rStrExpValue = rStrExpValue + " " + sVal;
+ rStrExpValue += " " + sVal;
}
}
else if( rStrExpValue.isEmpty() )
@@ -775,7 +775,7 @@ bool XMLGrfMirrorPropHdl_Impl::exportXML(
}
else
{
- rStrExpValue = rStrExpValue + " " + sVal;
+ rStrExpValue += " " + sVal;
}
}
else if( rStrExpValue.isEmpty() )
More information about the Libreoffice-commits
mailing list