diff options
author | Noel Grandin <noel@peralex.com> | 2016-03-02 16:28:45 +0200 |
---|---|---|
committer | Noel Grandin <noel@peralex.com> | 2016-03-02 16:39:13 +0200 |
commit | 037de512546917786c313d23995daaa0521a8e6e (patch) | |
tree | 7bff7e3ad727708b6f44adc0753440c6f7813c6c /slideshow/source/inc | |
parent | cdd7a0fe48aefee420d1f75b9dff04a26362773a (diff) |
tdf#98175 fix Impress crash
Caused by my commit f29c0b2b3e8861909fa2c6c37bf631ab01590541
"boost::shared_ptr->std::shared_ptr in slideshow"
where I foolishly converted
shared_from_this()
to
std::shared_ptr<Activity>((Activity*)this) )
all over the place, which is bad, I should have used
std::dynamic_pointer_cast<ViewEventHandler>(shared_from_this())
Ah, the joys of C++ memory management.
Change-Id: I9b538e78730c9107870f406cb8ba482d491dd568
Diffstat (limited to 'slideshow/source/inc')
-rw-r--r-- | slideshow/source/inc/disposable.hxx | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/slideshow/source/inc/disposable.hxx b/slideshow/source/inc/disposable.hxx index c85e3698e953..f1f693a23e33 100644 --- a/slideshow/source/inc/disposable.hxx +++ b/slideshow/source/inc/disposable.hxx @@ -36,6 +36,8 @@ namespace slideshow */ class SharedPtrAble : public std::enable_shared_from_this<SharedPtrAble> { + public: + virtual ~SharedPtrAble() {} }; /** Disposable interface |