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

[Noel Grandin (via logerrit)]

 sc/source/core/data/mtvelements.cxx |    6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

New commits:
commit d5e1a11b028fb9c03ff80f986f33927d6667627d
Author:     Noel Grandin <noel.gran...@collabora.co.uk>
AuthorDate: Mon Sep 19 10:41:30 2022 +0200
Commit:     Noel Grandin <noel.gran...@collabora.co.uk>
CommitDate: Mon Sep 19 13:02:16 2022 +0200

    get_value is cheaper than at() in toRefCell()
    
    and we expect the position to be valid here, so throwing an exception is
    no better than a segment violation
    
    Change-Id: I20e4849c95da57e6a5fc0937c4f61fbe004d11d9
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/140139
    Tested-by: Jenkins
    Reviewed-by: Noel Grandin <noel.gran...@collabora.co.uk>

diff --git a/sc/source/core/data/mtvelements.cxx 
b/sc/source/core/data/mtvelements.cxx
index 2c2c8daa64fb..9bc28056bd79 100644
--- a/sc/source/core/data/mtvelements.cxx
+++ b/sc/source/core/data/mtvelements.cxx
@@ -176,16 +176,16 @@ ScRefCellValue toRefCell( const 
sc::CellStoreType::const_iterator& itPos, size_t
     {
         case sc::element_type_numeric:
             // Numeric cell
-            return ScRefCellValue(sc::numeric_block::at(*itPos->data, 
nOffset));
+            return ScRefCellValue(sc::numeric_block::get_value(*itPos->data, 
nOffset));
         case sc::element_type_string:
             // String cell
             return ScRefCellValue(&sc::string_block::at(*itPos->data, 
nOffset));
         case sc::element_type_edittext:
             // Edit cell
-            return ScRefCellValue(sc::edittext_block::at(*itPos->data, 
nOffset));
+            return ScRefCellValue(sc::edittext_block::get_value(*itPos->data, 
nOffset));
         case sc::element_type_formula:
             // Formula cell
-            return ScRefCellValue(sc::formula_block::at(*itPos->data, 
nOffset));
+            return ScRefCellValue(sc::formula_block::get_value(*itPos->data, 
nOffset));
         default:
             ;
     }