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

[Noel Grandin (via logerrit)]

 sc/source/core/inc/interpre.hxx  |    2 +-
 sc/source/core/tool/interpr4.cxx |    2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

New commits:
commit 8db47ac6d52da86451120ce3c095958939e2da60
Author:     Noel Grandin <noel.gran...@collabora.co.uk>
AuthorDate: Mon Sep 19 10:40:25 2022 +0200
Commit:     Noel Grandin <noel.gran...@collabora.co.uk>
CommitDate: Mon Sep 19 16:42:45 2022 +0200

    can return by const& in PopString
    
    and avoid some ref-counting
    
    Change-Id: Id1e6fc09d1c517ce168cebb7a225aa6df48e72c9
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/140138
    Tested-by: Jenkins
    Reviewed-by: Noel Grandin <noel.gran...@collabora.co.uk>

diff --git a/sc/source/core/inc/interpre.hxx b/sc/source/core/inc/interpre.hxx
index 17dd79534260..85d2955003af 100644
--- a/sc/source/core/inc/interpre.hxx
+++ b/sc/source/core/inc/interpre.hxx
@@ -321,7 +321,7 @@ private:
     void Pop();
     void PopError();
     double PopDouble();
-    svl::SharedString PopString();
+    const svl::SharedString & PopString();
     void ValidateRef( const ScSingleRefData & rRef );
     void ValidateRef( const ScComplexRefData & rRef );
     void ValidateRef( const ScRefList & rRefList );
diff --git a/sc/source/core/tool/interpr4.cxx b/sc/source/core/tool/interpr4.cxx
index bc16f32badb1..90b02b1e7f47 100644
--- a/sc/source/core/tool/interpr4.cxx
+++ b/sc/source/core/tool/interpr4.cxx
@@ -809,7 +809,7 @@ double ScInterpreter::PopDouble()
     return 0.0;
 }
 
-svl::SharedString ScInterpreter::PopString()
+const svl::SharedString & ScInterpreter::PopString()
 {
     nCurFmtType = SvNumFormatType::TEXT;
     nCurFmtIndex = 0;