diff options
author | Caolán McNamara <caolanm@redhat.com> | 2023-02-16 10:42:06 +0000 |
---|---|---|
committer | Caolán McNamara <caolanm@redhat.com> | 2023-02-16 15:13:21 +0000 |
commit | 0908c7c1184ab7acb7b6f6740a82c2f12c296fae (patch) | |
tree | 7bfe09dce871f6f2bc59a81b689d7ab5a3a6f3ea /sc/source/ui/view/tabvwshd.cxx | |
parent | e71080e19365aa074c56d1136dad2b09783949a2 (diff) |
SfxViewShell::GetViewFrame never returns null, change to a reference
various null checks can be seen to be redundant and removed
Change-Id: Icf49c1de4b0302795d2769a370af3abceaad0221
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/147147
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Diffstat (limited to 'sc/source/ui/view/tabvwshd.cxx')
-rw-r--r-- | sc/source/ui/view/tabvwshd.cxx | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/sc/source/ui/view/tabvwshd.cxx b/sc/source/ui/view/tabvwshd.cxx index 86a3c0e623ba..52d97d1a25a9 100644 --- a/sc/source/ui/view/tabvwshd.cxx +++ b/sc/source/ui/view/tabvwshd.cxx @@ -34,10 +34,10 @@ weld::Window* ScTabViewShell::GetDialogParent() // (necessary when a slot is executed from the dialog's OK handler) if (nCurRefDlgId && nCurRefDlgId == SC_MOD()->GetCurRefDlgId()) { - SfxViewFrame* pViewFrm = GetViewFrame(); - if (pViewFrm->HasChildWindow(nCurRefDlgId)) + SfxViewFrame& rViewFrm = GetViewFrame(); + if (rViewFrm.HasChildWindow(nCurRefDlgId)) { - SfxChildWindow* pChild = pViewFrm->GetChildWindow(nCurRefDlgId); + SfxChildWindow* pChild = rViewFrm.GetChildWindow(nCurRefDlgId); if (pChild) { auto xController = pChild->GetController(); |