diff options
| author | Noel Grandin <noel.grandin@collabora.co.uk> | 2018-10-16 15:23:12 +0200 |
|---|---|---|
| committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2018-10-17 08:25:47 +0200 |
| commit | ccb2a1f650bc505f8a4f1abebf8ce4f9396562a8 (patch) | |
| tree | 2ee2fd4f300ae95cf23bade1f242e02f9b276c07 /sd/source/ui/tools | |
| parent | da5c3a1ee43dd1a07cbd1b8005488bf05432593d (diff) | |
clang-tidy readability-redundant-smartptr-get
redundant get() call on smart pointer
Change-Id: Icb5a03bbc15e79a30d3d135a507d22914d15c2bd
Reviewed-on: https://gerrit.libreoffice.org/61837
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'sd/source/ui/tools')
| -rw-r--r-- | sd/source/ui/tools/PreviewRenderer.cxx | 4 | ||||
| -rw-r--r-- | sd/source/ui/tools/TimerBasedTaskExecution.cxx | 4 |
2 files changed, 4 insertions, 4 deletions
diff --git a/sd/source/ui/tools/PreviewRenderer.cxx b/sd/source/ui/tools/PreviewRenderer.cxx index 83812ebbe69a..c12930b10ece 100644 --- a/sd/source/ui/tools/PreviewRenderer.cxx +++ b/sd/source/ui/tools/PreviewRenderer.cxx @@ -208,7 +208,7 @@ bool PreviewRenderer::Initialize ( // Create view ProvideView (pDocShell); - if (mpView.get() == nullptr) + if (mpView == nullptr) return false; // Adjust contrast mode. @@ -390,7 +390,7 @@ void PreviewRenderer::ProvideView (DrawDocShell* pDocShell) if (mpDocShellOfView != nullptr) StartListening (*mpDocShellOfView); } - if (mpView.get() == nullptr) + if (mpView == nullptr) { mpView.reset (new DrawView (pDocShell, mpPreviewDevice.get(), nullptr)); } diff --git a/sd/source/ui/tools/TimerBasedTaskExecution.cxx b/sd/source/ui/tools/TimerBasedTaskExecution.cxx index af1b9ff6f065..379665e32f6a 100644 --- a/sd/source/ui/tools/TimerBasedTaskExecution.cxx +++ b/sd/source/ui/tools/TimerBasedTaskExecution.cxx @@ -48,7 +48,7 @@ std::shared_ptr<TimerBasedTaskExecution> TimerBasedTaskExecution::Create ( // Let the new object have a shared_ptr to itself, so that it can // release itself when the AsynchronousTask has been executed // completely. - if (pExecution->mpTask.get() != nullptr) + if (pExecution->mpTask != nullptr) pExecution->mpSelf = pExecution; return pExecution; } @@ -101,7 +101,7 @@ TimerBasedTaskExecution::~TimerBasedTaskExecution() IMPL_LINK_NOARG(TimerBasedTaskExecution, TimerCallback, Timer *, void) { - if (mpTask.get() != nullptr) + if (mpTask != nullptr) { if (mpTask->HasNextStep()) { |