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

Luboš Luňák l.lunak at collabora.com
Thu May 24 15:54:21 UTC 2018


 sc/source/core/data/column2.cxx |    3 +++
 1 file changed, 3 insertions(+)

New commits:
commit 8f7eb6edfa5cc561d52060f44128d4c0e5d439bf
Author: Luboš Luňák <l.lunak at collabora.com>
Date:   Thu May 24 13:54:23 2018 +0200

    avoid CELLTYPE_EDIT for calc's threaded calculations
    
    Since ScEditUtils::GetString() requires EditEngine, which is probably not
    completely thread-safe.
    
    Change-Id: Ie3d64f56b76a5861fa4b0a1f9cb683cd26806b35
    Reviewed-on: https://gerrit.libreoffice.org/54766
    Reviewed-by: Michael Meeks <michael.meeks at collabora.com>
    Tested-by: Michael Meeks <michael.meeks at collabora.com>

diff --git a/sc/source/core/data/column2.cxx b/sc/source/core/data/column2.cxx
index ba3896ddcc35..0a62cb10506d 100644
--- a/sc/source/core/data/column2.cxx
+++ b/sc/source/core/data/column2.cxx
@@ -2823,6 +2823,9 @@ bool ScColumn::HandleRefArrayForParallelism( SCROW nRow1, SCROW nRow2 )
         auto aCell = GetCellValue(i);
         if (aCell.meType == CELLTYPE_FORMULA)
             aCell.mpFormula->MaybeInterpret();
+        // These require EditEngine (in ScEditUtils::GetString()), which is probably too complex for use in threads.
+        if (aCell.meType == CELLTYPE_EDIT)
+            return false;
     }
 
     return true;


More information about the Libreoffice-commits mailing list