[PATCH] Change in core[libreoffice-3-6]: rhbz#818557: fix it differently: SwDocShell::_LoadStyles():

Michael Stahl (via Code Review) gerrit at gerrit.libreoffice.org
Wed Jan 16 09:24:05 PST 2013


Hi,

I have submitted a patch for review:

    https://gerrit.libreoffice.org/1724

To pull it, you can do:

    git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/24/1724/1

rhbz#818557: fix it differently: SwDocShell::_LoadStyles():

Try to prevent pushing of SfxShells from this function via EndAllAction
and SwView::AttrChangedNotify() by setting a ridiculous pre-exising
global bNoInterrupt variable.

Change-Id: I9c91ca882891c8dfcd9dc08ba197233cfefddefd
(cherry picked from commit beacee6fad46aa2c8fc813bb0150e5c7a5175b26)
---
M sw/source/ui/app/docst.cxx
1 file changed, 9 insertions(+), 0 deletions(-)



diff --git a/sw/source/ui/app/docst.cxx b/sw/source/ui/app/docst.cxx
index 8f90a15..81ed1a7 100644
--- a/sw/source/ui/app/docst.cxx
+++ b/sw/source/ui/app/docst.cxx
@@ -31,6 +31,8 @@
 #include <com/sun/star/beans/XPropertySet.hpp>
 #include <com/sun/star/container/XNameAccess.hpp>
 
+#include <comphelper/flagguard.hxx>
+
 #include <hintids.hxx>
 #include <sfx2/app.hxx>
 #include <svl/whiter.hxx>
@@ -85,6 +87,8 @@
 #include <list.hxx>
 
 #include <paratr.hxx>   //#outline level,add by zhaojianwei
+
+extern bool bNoInterrupt;       // in mainwn.cxx
 
 using namespace ::com::sun::star;
 
@@ -1223,6 +1227,11 @@
             ((SwDocShell&)rSource).pDoc->SetFixFields(false, NULL);
         if( pWrtShell )
         {
+            // rhbz#818557, fdo#58893: EndAllAction will call SelectShell(),
+            // which pushes a bunch of SfxShells that are not cleared
+            // (for unknown reasons) when closing the document, causing crash;
+            // setting bNoInterrupt appears to avoid the problem.
+            ::comphelper::FlagRestorationGuard g(bNoInterrupt, true);
             pWrtShell->StartAllAction();
             pDoc->ReplaceStyles( *((SwDocShell&)rSource).pDoc );
             pWrtShell->EndAllAction();

-- 
To view, visit https://gerrit.libreoffice.org/1724
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I9c91ca882891c8dfcd9dc08ba197233cfefddefd
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: libreoffice-3-6
Gerrit-Owner: Michael Stahl <mstahl at redhat.com>



More information about the LibreOffice mailing list