Kaydet (Commit) 07098076 authored tarafından Noel Grandin's avatar Noel Grandin

loplugin:useuniqueptr in ToolBox

Change-Id: Id2cbc7d70de2e0ddf0503448575fe2c2ca0120a0
Reviewed-on: https://gerrit.libreoffice.org/53345Tested-by: 's avatarJenkins <ci@libreoffice.org>
Reviewed-by: 's avatarNoel Grandin <noel.grandin@collabora.co.uk>
üst a11b9c00
...@@ -28,6 +28,7 @@ ...@@ -28,6 +28,7 @@
#include <o3tl/typed_flags_set.hxx> #include <o3tl/typed_flags_set.hxx>
#include <limits> #include <limits>
#include <memory>
#include <vector> #include <vector>
#include <com/sun/star/frame/XFrame.hpp> #include <com/sun/star/frame/XFrame.hpp>
...@@ -98,9 +99,9 @@ private: ...@@ -98,9 +99,9 @@ private:
ImplToolItems::size_type mnLines; ImplToolItems::size_type mnLines;
}; };
ImplToolBoxPrivateData* mpData; std::unique_ptr<ImplToolBoxPrivateData> mpData;
std::vector<ImplToolSize> maFloatSizes; std::vector<ImplToolSize> maFloatSizes;
Idle *mpIdle; std::unique_ptr<Idle> mpIdle;
tools::Rectangle maUpperRect; tools::Rectangle maUpperRect;
tools::Rectangle maLowerRect; tools::Rectangle maLowerRect;
tools::Rectangle maPaintRect; tools::Rectangle maPaintRect;
...@@ -254,7 +255,7 @@ public: ...@@ -254,7 +255,7 @@ public:
SAL_DLLPRIVATE void ImplDrawMenuButton(vcl::RenderContext& rRenderContext, bool bHighlight); SAL_DLLPRIVATE void ImplDrawMenuButton(vcl::RenderContext& rRenderContext, bool bHighlight);
SAL_DLLPRIVATE void ImplDrawButton(vcl::RenderContext& rRenderContext, const tools::Rectangle &rRect, sal_uInt16 highlight, bool bChecked, bool bEnabled, bool bIsWindow); SAL_DLLPRIVATE void ImplDrawButton(vcl::RenderContext& rRenderContext, const tools::Rectangle &rRect, sal_uInt16 highlight, bool bChecked, bool bEnabled, bool bIsWindow);
SAL_DLLPRIVATE ImplToolItems::size_type ImplCountLineBreaks() const; SAL_DLLPRIVATE ImplToolItems::size_type ImplCountLineBreaks() const;
SAL_DLLPRIVATE ImplToolBoxPrivateData* ImplGetToolBoxPrivateData() const { return mpData; } SAL_DLLPRIVATE ImplToolBoxPrivateData* ImplGetToolBoxPrivateData() const { return mpData.get(); }
protected: protected:
virtual void ApplySettings(vcl::RenderContext& rRenderContext) override; virtual void ApplySettings(vcl::RenderContext& rRenderContext) override;
......
...@@ -1093,7 +1093,7 @@ void ToolBox::ImplInitToolBoxData() ...@@ -1093,7 +1093,7 @@ void ToolBox::ImplInitToolBoxData()
{ {
// initialize variables // initialize variables
ImplGetWindowImpl()->mbToolBox = true; ImplGetWindowImpl()->mbToolBox = true;
mpData = new ImplToolBoxPrivateData; mpData.reset(new ImplToolBoxPrivateData);
mpFloatWin = nullptr; mpFloatWin = nullptr;
mnDX = 0; mnDX = 0;
mnDY = 0; mnDY = 0;
...@@ -1147,7 +1147,7 @@ void ToolBox::ImplInitToolBoxData() ...@@ -1147,7 +1147,7 @@ void ToolBox::ImplInitToolBoxData()
mpStatusListener = new VclStatusListener<ToolBox>(this, ".uno:ImageOrientation"); mpStatusListener = new VclStatusListener<ToolBox>(this, ".uno:ImageOrientation");
mpStatusListener->startListening(); mpStatusListener->startListening();
mpIdle = new Idle("vcl::ToolBox maIdle update"); mpIdle.reset(new Idle("vcl::ToolBox maIdle update"));
mpIdle->SetPriority( TaskPriority::RESIZE ); mpIdle->SetPriority( TaskPriority::RESIZE );
mpIdle->SetInvokeHandler( LINK( this, ToolBox, ImplUpdateHdl ) ); mpIdle->SetInvokeHandler( LINK( this, ToolBox, ImplUpdateHdl ) );
...@@ -1355,8 +1355,7 @@ void ToolBox::dispose() ...@@ -1355,8 +1355,7 @@ void ToolBox::dispose()
mpFloatWin = nullptr; mpFloatWin = nullptr;
// delete private data // delete private data
delete mpData; mpData.reset();
mpData = nullptr;
ImplSVData* pSVData = ImplGetSVData(); ImplSVData* pSVData = ImplGetSVData();
delete pSVData->maCtrlData.mpTBDragMgr; delete pSVData->maCtrlData.mpTBDragMgr;
...@@ -1367,8 +1366,7 @@ void ToolBox::dispose() ...@@ -1367,8 +1366,7 @@ void ToolBox::dispose()
mpFloatWin.clear(); mpFloatWin.clear();
delete mpIdle; mpIdle.reset();
mpIdle = nullptr;
DockingWindow::dispose(); DockingWindow::dispose();
} }
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment