From 730df730a9425dbf4ed778dce7a95c1c0be274f6 Mon Sep 17 00:00:00 2001
From: Matteo Casalin <matteo.casalin@yahoo.com>
Date: Sun, 17 Feb 2019 23:32:22 +0100
Subject: Avoid unneeded OUString copy

Change-Id: I43d66f1bf4fc4a17f7dbea62e3fb13675dbbfb8a
Reviewed-on: https://gerrit.libreoffice.org/68119
Tested-by: Jenkins
Reviewed-by: Matteo Casalin <matteo.casalin@yahoo.com>
---
 l10ntools/source/xrmmerge.cxx | 11 ++++-------
 1 file changed, 4 insertions(+), 7 deletions(-)

(limited to 'l10ntools')

diff --git a/l10ntools/source/xrmmerge.cxx b/l10ntools/source/xrmmerge.cxx
index 4670dc7c3ab2..b6f7d388c31f 100644
--- a/l10ntools/source/xrmmerge.cxx
+++ b/l10ntools/source/xrmmerge.cxx
@@ -269,13 +269,10 @@ OString XRMResParser::GetAttribute( const OString &rToken, const OString &rAttri
     sSearch += "=";
     sal_Int32 nPos = sTmp.indexOf( sSearch );
 
-    if ( nPos != -1 )
-    {
-        sTmp = sTmp.copy( nPos );
-        OString sId = sTmp.getToken(1, '"');
-        return sId;
-    }
-    return OString();
+    if ( nPos<0 )
+        return OString();
+
+    return sTmp.getToken(1, '"', nPos);
 }
 
 
-- 
cgit