[Libreoffice-commits] core.git: sc/source

Mike Kaganski (via logerrit) logerrit at kemper.freedesktop.org
Sun Oct 20 12:27:09 UTC 2019


 sc/source/core/tool/rangeutl.cxx |    5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

New commits:
commit f99982a40fda08a037083f4b942ffb349d76e939
Author:     Mike Kaganski <mike.kaganski at collabora.com>
AuthorDate: Sun Oct 20 12:51:12 2019 +0200
Commit:     Mike Kaganski <mike.kaganski at collabora.com>
CommitDate: Sun Oct 20 14:26:10 2019 +0200

    Avoid converting same string to uppercase twice
    
    Change-Id: I8f6be79525f1383bc27eb799efbaa1009cbaed50
    Reviewed-on: https://gerrit.libreoffice.org/81153
    Tested-by: Jenkins
    Reviewed-by: Mike Kaganski <mike.kaganski at collabora.com>

diff --git a/sc/source/core/tool/rangeutl.cxx b/sc/source/core/tool/rangeutl.cxx
index aa2c519cd4aa..aed6ed61e5b1 100644
--- a/sc/source/core/tool/rangeutl.cxx
+++ b/sc/source/core/tool/rangeutl.cxx
@@ -268,10 +268,11 @@ bool ScRangeUtil::MakeRangeFromName (
         //then check for local range names
         ScRangeName* pRangeNames = pDoc->GetRangeName( nTable );
         ScRangeData* pData = nullptr;
+        aName = ScGlobal::pCharClass->uppercase(aName);
         if ( pRangeNames )
-            pData = pRangeNames->findByUpperName(ScGlobal::pCharClass->uppercase(aName));
+            pData = pRangeNames->findByUpperName(aName);
         if (!pData)
-            pData = pDoc->GetRangeName()->findByUpperName(ScGlobal::pCharClass->uppercase(aName));
+            pData = pDoc->GetRangeName()->findByUpperName(aName);
         if (pData)
         {
             OUString         aStrArea;


More information about the Libreoffice-commits mailing list