sc/source/filter/inc/worksheethelper.hxx | 4 ++++
sc/source/filter/oox/commentsbuffer.cxx | 7 +++++++
sc/source/filter/oox/worksheethelper.cxx | 5 +++++
3 files changed, 16 insertions(+)
New commits:
commit 543fd1b9f8e05988d4c45602f4247566c352ee81
Author: Noel Grandin <noel.gran...@collabora.co.uk>
AuthorDate: Thu Dec 19 15:07:46 2019 +0200
Commit: Noel Grandin <noel.gran...@collabora.co.uk>
CommitDate: Fri Dec 20 15:14:54 2019 +0100
tdf#129228 speedup load xls file with lots of comments
by locking the underlying SdrModel while the comments are added, so we
avoid the reformatting that happends when the model is unlocked
Takes the time from 3m30 to 1m for me.
Change-Id: Ibc3d2b1ded548d5eee7715d9a655a0af61b0eb2b
Reviewed-on: https://gerrit.libreoffice.org/85523
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.gran...@collabora.co.uk>
(cherry picked from commit f2f5d74067705b7c64f8fb510952d7cece417627)
Reviewed-on: https://gerrit.libreoffice.org/85579
diff --git a/sc/source/filter/inc/worksheethelper.hxx
b/sc/source/filter/inc/worksheethelper.hxx
index 8f135e05ea9c..e3a2a955b22a 100644
--- a/sc/source/filter/inc/worksheethelper.hxx
+++ b/sc/source/filter/inc/worksheethelper.hxx
@@ -27,6 +27,7 @@
#include "formulabase.hxx"
struct ScDataBarFormatData;
+class ScDocument;
namespace com { namespace sun { namespace star {
namespace awt { struct Point; }
@@ -294,6 +295,9 @@ public:
void setCellFormulaValue(
const ScAddress& rAddress, const OUString& rValueStr, sal_Int32
nCellType );
+ ScDocument& getScDocument();
+
+
private:
WorksheetGlobals& mrSheetGlob;
};
diff --git a/sc/source/filter/oox/commentsbuffer.cxx
b/sc/source/filter/oox/commentsbuffer.cxx
index c3c60c3f5a6d..43b133aad4fe 100644
--- a/sc/source/filter/oox/commentsbuffer.cxx
+++ b/sc/source/filter/oox/commentsbuffer.cxx
@@ -34,6 +34,8 @@
#include <addressconverter.hxx>
#include <drawingfragment.hxx>
#include <svx/sdtaitm.hxx>
+#include <document.hxx>
+#include <drwlayer.hxx>
namespace oox {
namespace xls {
@@ -221,7 +223,12 @@ CommentRef CommentsBuffer::createComment()
void CommentsBuffer::finalizeImport()
{
+ // keep the model locked to avoid repeated reformatting in the model
+ auto pModel = getScDocument().GetDrawLayer();
+ bool bWasLocked = pModel->isLocked();
+ pModel->setLock(true);
maComments.forEachMem( &Comment::finalizeImport );
+ pModel->setLock(bWasLocked);
}
} // namespace xls
diff --git a/sc/source/filter/oox/worksheethelper.cxx
b/sc/source/filter/oox/worksheethelper.cxx
index a567435fcc28..eb18fb1dd1d5 100644
--- a/sc/source/filter/oox/worksheethelper.cxx
+++ b/sc/source/filter/oox/worksheethelper.cxx
@@ -1363,6 +1363,11 @@ WorksheetHelper::WorksheetHelper( WorksheetGlobals&
rSheetGlob ) :
{
}
+ScDocument& WorksheetHelper::getScDocument()
+{
+ return getDocImport().getDoc();
+}
+
/*static*/ WorksheetGlobalsRef WorksheetHelper::constructGlobals( const
WorkbookHelper& rHelper,
const ISegmentProgressBarRef& rxProgressBar, WorksheetType eSheetType,
SCTAB nSheet )
{
_______________________________________________
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits
