summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorCaolán McNamara <caolanm@redhat.com>2018-10-20 19:57:55 +0100
committerCaolán McNamara <caolanm@redhat.com>2018-10-21 18:18:02 +0200
commite8db509ee41369c9f805a8ed1a4447b5c03962be (patch)
treee106650b08cd2508e6445f7b43b4586c1f737964
parent6837fbb1884cd8ad928ac028f9a94fad54dc1827 (diff)
pvs-studio: _pView was already dereferenced
Change-Id: I72511a46524d02007c7b8e8b0e6486d62cbc7fa0 Reviewed-on: https://gerrit.libreoffice.org/62103 Tested-by: Jenkins Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
-rw-r--r--dbaccess/source/ui/querydesign/QueryDesignView.cxx4
1 files changed, 2 insertions, 2 deletions
diff --git a/dbaccess/source/ui/querydesign/QueryDesignView.cxx b/dbaccess/source/ui/querydesign/QueryDesignView.cxx
index fd7599d788fd..ba7fb22d0946 100644
--- a/dbaccess/source/ui/querydesign/QueryDesignView.cxx
+++ b/dbaccess/source/ui/querydesign/QueryDesignView.cxx
@@ -735,7 +735,7 @@ namespace
// only show the messagebox the first time
if (!bCritsOnAsterikWarning)
{
- std::unique_ptr<weld::MessageDialog> xBox(Application::CreateMessageDialog(_pView ? _pView->GetFrameWeld() : nullptr,
+ std::unique_ptr<weld::MessageDialog> xBox(Application::CreateMessageDialog(_pView->GetFrameWeld(),
VclMessageType::Warning, VclButtonsType::Ok,
DBA_RES(STR_QRY_CRITERIA_ON_ASTERISK)));
xBox->run();
@@ -891,7 +891,7 @@ namespace
// only show the MessageBox the first time
if (!bCritsOnAsterikWarning)
{
- std::unique_ptr<weld::MessageDialog> xBox(Application::CreateMessageDialog(_pView ? _pView->GetFrameWeld() : nullptr,
+ std::unique_ptr<weld::MessageDialog> xBox(Application::CreateMessageDialog(_pView->GetFrameWeld(),
VclMessageType::Warning, VclButtonsType::Ok,
DBA_RES(STR_QRY_ORDERBY_ON_ASTERISK)));
xBox->run();