basctl/source/basicide/macrodlg.cxx | 1 - 1 file changed, 1 deletion(-)
New commits: commit 4035f51534610160423692dc8e10bef797b5cddf Author: Michael Weghorn <[email protected]> AuthorDate: Sun Dec 21 23:25:00 2025 +0100 Commit: Michael Weghorn <[email protected]> CommitDate: Mon Dec 22 09:18:20 2025 +0100 basctl: Drop unnecessary call to weld::ItemView::get_selected This call sets m_xBasicBoxIter to the currently selected entry, but it's not used here after that, and m_xBasicBoxIter is reassigned a different value first before it's used elsewhere. Therefore, drop this unnecessary call altogether. Change-Id: I301b259a3edfce4af7deccdc8db9c4728d502801 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/196055 Tested-by: Jenkins Reviewed-by: Michael Weghorn <[email protected]> diff --git a/basctl/source/basicide/macrodlg.cxx b/basctl/source/basicide/macrodlg.cxx index b880b36fafc3..b5c2bd18b399 100644 --- a/basctl/source/basicide/macrodlg.cxx +++ b/basctl/source/basicide/macrodlg.cxx @@ -764,7 +764,6 @@ IMPL_LINK(MacroChooser, ButtonHdl, weld::Button&, rButton, void) { StoreMacroDescription(); - m_xBasicBox->get_selected(m_xBasicBoxIter.get()); auto xDlg(std::make_shared<OrganizeDialog>(m_xDialog.get(), nullptr, 0)); weld::DialogController::runAsync(xDlg, [this](sal_Int32 nRet) { if (nRet == RET_OK) // not only closed
