[Libreoffice-commits] .: sd/source

Thorsten Behrens thorsten at kemper.freedesktop.org
Wed Nov 16 03:39:57 PST 2011


 sd/source/ui/slideshow/slideshowimpl.cxx |    5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

New commits:
commit 84da4113b4ec80cccd5916bf1289564837d6d5f9
Author: Thorsten Behrens <tbehrens at suse.com>
Date:   Wed Nov 16 12:37:04 2011 +0100

    Fix slideshow crasher
    
    Recursive event loop killed SlideshowImpl object under my hands -
    make it stay until we pop out of callstack again.

diff --git a/sd/source/ui/slideshow/slideshowimpl.cxx b/sd/source/ui/slideshow/slideshowimpl.cxx
index 867e3bd..7de8329 100644
--- a/sd/source/ui/slideshow/slideshowimpl.cxx
+++ b/sd/source/ui/slideshow/slideshowimpl.cxx
@@ -1873,6 +1873,9 @@ IMPL_LINK( SlideshowImpl, updateHdl, Timer*, EMPTYARG )
 
 IMPL_LINK( SlideshowImpl, PostYieldListener, void*, EMPTYARG )
 {
+    // prevent me from deletion when recursing (App::Reschedule does)
+    const rtl::Reference<SlideshowImpl> this_(this);
+
     Application::EnableNoYieldMode(false);
     Application::RemovePostYieldListener(LINK(this, SlideshowImpl, PostYieldListener));
     Application::Reschedule(true); // fix for fdo#32861 - process
@@ -1888,7 +1891,7 @@ IMPL_LINK( SlideshowImpl, PostYieldListener, void*, EMPTYARG )
 
 sal_Int32 SlideshowImpl::updateSlideShow (void)
 {
-    // doing some nMagic
+    // prevent me from deletion when recursing (App::EnableYieldMode does)
     const rtl::Reference<SlideshowImpl> this_(this);
 
     Reference< XSlideShow > xShow( mxShow );


More information about the Libreoffice-commits mailing list