diff OrthancStone/Sources/Scene2DViewport/MeasureTrackers.h @ 1606:874e178f34e9

- ViewportController now has weak ptr to Viewport - Measuring tools + related commands and all trackers now store only a weak_ptr to the Viewport and lock() on demand for usage - LayerHolder and FixedPoint aligner store only a weak_ptr to the Viewport, too - Fixed float/double warning in GrayscaleWindowingSceneTracker
author Benjamin Golinvaux <bgo@osimis.io>
date Wed, 28 Oct 2020 20:14:34 +0100
parents 8563ea5d8ae4
children 9ac2a65d4172
line wrap: on
line diff
--- a/OrthancStone/Sources/Scene2DViewport/MeasureTrackers.h	Wed Oct 28 20:06:55 2020 +0100
+++ b/OrthancStone/Sources/Scene2DViewport/MeasureTrackers.h	Wed Oct 28 20:14:34 2020 +0100
@@ -42,10 +42,16 @@
     
   protected:
     boost::shared_ptr<CreateMeasureCommand>  command_;
-    boost::shared_ptr<IViewport>             viewport_;
+    boost::weak_ptr<IViewport>               viewport_;
     bool                                     alive_;
 
-    explicit CreateMeasureTracker(boost::shared_ptr<IViewport> viewport);
+    /**
+    This will return a scoped lock to the viewport.
+    If the viewport does not exist anymore, then nullptr is returned.
+    */
+    IViewport::ILock* GetViewportLock();
+
+    explicit CreateMeasureTracker(boost::weak_ptr<IViewport> viewport);
 
     virtual ~CreateMeasureTracker();
     
@@ -64,10 +70,16 @@
 
   protected:
     boost::shared_ptr<EditMeasureCommand>  command_;
-    boost::shared_ptr<IViewport>           viewport_;
+    boost::weak_ptr<IViewport>             viewport_;
     bool                                   alive_;
     
-    EditMeasureTracker(boost::shared_ptr<IViewport> viewport,
+    /**
+    This will return a scoped lock to the viewport.
+    If the viewport does not exist anymore, then nullptr is returned.
+    */
+    IViewport::ILock* GetViewportLock();
+
+    EditMeasureTracker(boost::weak_ptr<IViewport> viewport,
                        const PointerEvent& e);
 
     ~EditMeasureTracker();