From 070ea7faa821a01b6a077e24724e37962173cb23 Mon Sep 17 00:00:00 2001 From: Matteo Casalin Date: Thu, 21 Feb 2019 18:25:02 +0100 Subject: Use optimized OString concatenation Change-Id: Iaefacf4a57398d0e88b4de7552af11832db3e881 Reviewed-on: https://gerrit.libreoffice.org/68178 Tested-by: Jenkins Reviewed-by: Matteo Casalin --- l10ntools/source/lngmerge.cxx | 28 +++++++++++----------------- 1 file changed, 11 insertions(+), 17 deletions(-) (limited to 'l10ntools') diff --git a/l10ntools/source/lngmerge.cxx b/l10ntools/source/lngmerge.cxx index 39f98747e99c..f6939458c588 100644 --- a/l10ntools/source/lngmerge.cxx +++ b/l10ntools/source/lngmerge.cxx @@ -209,9 +209,7 @@ void LngParser::Merge( { sLang = sLang.trim(); - OString sSearch( ";" ); - sSearch += sLang; - sSearch += ";"; + OString sSearch{ ";" + sLang + ";" }; if ( sLanguagesDone.indexOf( sSearch ) != -1 ) { mvLines.erase( mvLines.begin() + nPos ); @@ -226,14 +224,11 @@ void LngParser::Merge( continue; if ( !sNewText.isEmpty()) { - OString & rLine = mvLines[ nPos ]; - - OString sText1( sLang ); - sText1 += " = \""; - // escape quotes, unescape double escaped quotes fdo#56648 - sText1 += sNewText.replaceAll("\"","\\\"").replaceAll("\\\\\"","\\\""); - sText1 += "\""; - rLine = sText1; + mvLines[ nPos ] = sLang + + " = \"" + // escape quotes, unescape double escaped quotes fdo#56648 + + sNewText.replaceAll("\"","\\\"").replaceAll("\\\\\"","\\\"") + + "\""; Text[ sLang ] = sNewText; } } @@ -264,12 +259,11 @@ void LngParser::Merge( continue; if ( !sNewText.isEmpty() && sCur != "x-comment") { - OString sLine; - sLine += sCur; - sLine += " = \""; - // escape quotes, unescape double escaped quotes fdo#56648 - sLine += sNewText.replaceAll("\"","\\\"").replaceAll("\\\\\"","\\\""); - sLine += "\""; + const OString sLine { sCur + + " = \"" + // escape quotes, unescape double escaped quotes fdo#56648 + + sNewText.replaceAll("\"","\\\"").replaceAll("\\\\\"","\\\"") + + "\"" }; nLastLangPos++; nPos++; -- cgit