Further reduction of OUString copy operations

Change-Id: I79339e7cf8fa6b6a6f19ba598fc66d9e0df558ae
Reviewed-on: https://gerrit.libreoffice.org/65669
Tested-by: Jenkins
Reviewed-by: Matteo Casalin <matteo.casalin@yahoo.com>

1 files changed, 5 insertions, 6 deletions

diff --git a/connectivity/source/commontools/AutoRetrievingBase.cxx b/connectivity/source/commontools/AutoRetrievingBase.cxx
index c99e6b5e82b1..d1170fdd972b 100644
--- a/connectivity/source/commontools/AutoRetrievingBase.cxx
+++ b/connectivity/source/commontools/AutoRetrievingBase.cxx
@@ -27,25 +27,24 @@ namespace connectivity
     {
         OSL_ENSURE( m_bAutoRetrievingEnabled,"Illegal call here. isAutoRetrievingEnabled is false!");
         OUString sStmt = _sInsertStatement.toAsciiUpperCase();
-        OUString sStatement;
         if ( sStmt.startsWith("INSERT") )
         {
-            sStatement = m_sGeneratedValueStatement;
             static const char sTable[] = "$table";
-            const sal_Int32 nColumnIndex {sStatement.indexOf("$column")};
+            const sal_Int32 nColumnIndex {m_sGeneratedValueStatement.indexOf("$column")};
             if ( nColumnIndex>=0 )
             { // we need a column
             }
-            const sal_Int32 nTableIndex {sStatement.indexOf(sTable)};
+            const sal_Int32 nTableIndex {m_sGeneratedValueStatement.indexOf(sTable)};
             if ( nTableIndex>=0 )
             { // we need a table name
                 sal_Int32 nIntoIndex = sStmt.indexOf("INTO ") + 5;
                 while (nIntoIndex<sStmt.getLength() && sStmt[nIntoIndex]==' ') ++nIntoIndex;
                 const OUString sTableName = sStmt.getToken(0, ' ', nIntoIndex);
-                sStatement = sStatement.replaceAt(nTableIndex, strlen(sTable), sTableName);
+                return m_sGeneratedValueStatement.replaceAt(nTableIndex, strlen(sTable), sTableName);
             }
+            return m_sGeneratedValueStatement;
         }
-        return sStatement;
+        return OUString();
     }
 }