diff options
| author | Caolán McNamara <caolanm@redhat.com> | 2018-10-20 19:57:55 +0100 |
|---|---|---|
| committer | Caolán McNamara <caolanm@redhat.com> | 2018-10-21 18:18:02 +0200 |
| commit | e8db509ee41369c9f805a8ed1a4447b5c03962be (patch) | |
| tree | e106650b08cd2508e6445f7b43b4586c1f737964 /dbaccess/source | |
| parent | 6837fbb1884cd8ad928ac028f9a94fad54dc1827 (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>
Diffstat (limited to 'dbaccess/source')
| -rw-r--r-- | dbaccess/source/ui/querydesign/QueryDesignView.cxx | 4 |
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(); |