sfx2/source/control/templatelocalview.cxx | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-)
New commits: commit c5dd771cc95f21a397f12ef1b76b18fd7b0c5745 Author: Michael Weghorn <[email protected]> AuthorDate: Wed Dec 31 00:37:20 2025 +0100 Commit: Michael Weghorn <[email protected]> CommitDate: Wed Dec 31 20:29:13 2025 +0100 sfx2: Cast once in TemplateLocalView::Command More consistently use pViewItem after casting pItem and don't do another dynamic_cast again later, but directly use pViewItem. Change-Id: I143308e3f72468c2919b1f871867ba10c75b20fa Reviewed-on: https://gerrit.libreoffice.org/c/core/+/196342 Tested-by: Jenkins Reviewed-by: Michael Weghorn <[email protected]> diff --git a/sfx2/source/control/templatelocalview.cxx b/sfx2/source/control/templatelocalview.cxx index 3284cf4a1978..169c0da2464e 100644 --- a/sfx2/source/control/templatelocalview.cxx +++ b/sfx2/source/control/templatelocalview.cxx @@ -706,18 +706,18 @@ bool TemplateLocalView::Command(const CommandEvent& rCEvt) Point aPosition(rCEvt.GetMousePosPixel()); maPosition = aPosition; ThumbnailViewItem* pItem = ImplGetItem(nPos); - const TemplateViewItem *pViewItem = dynamic_cast<const TemplateViewItem*>(pItem); + TemplateViewItem* pViewItem = dynamic_cast<TemplateViewItem*>(pItem); if(pViewItem) { - if(!pItem->isSelected()) + if (!pViewItem->isSelected()) { deselectItems(); - pItem->setSelection(true); - maItemStateHdl.Call(pItem); + pViewItem->setSelection(true); + maItemStateHdl.Call(pViewItem); } - maSelectedItem = dynamic_cast<TemplateViewItem*>(pItem); - maCreateContextMenuHdl.Call(pItem); + maSelectedItem = pViewItem; + maCreateContextMenuHdl.Call(pViewItem); } } else
