From 8ce81efa73f93afdf46f9586b7b6cecf36cf6914 Mon Sep 17 00:00:00 2001 From: Noel Grandin Date: Fri, 29 Oct 2021 12:25:39 +0200 Subject: simplify framework::TitleHelper Every callsite (bar one) calls setOwner and connectWithUntitledNumbers after construction, so just pass them in the constructor. And for that lone callsite we can just nullptr as an argument. Change-Id: If838e068bb59a407225d853a0f67983be400f2f5 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/124426 Tested-by: Jenkins Reviewed-by: Noel Grandin --- reportdesign/source/core/api/ReportDefinition.cxx | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) (limited to 'reportdesign') diff --git a/reportdesign/source/core/api/ReportDefinition.cxx b/reportdesign/source/core/api/ReportDefinition.cxx index 82776672d8ee..6e8c0950b72e 100644 --- a/reportdesign/source/core/api/ReportDefinition.cxx +++ b/reportdesign/source/core/api/ReportDefinition.cxx @@ -2490,10 +2490,8 @@ uno::Reference< frame::XTitle > OReportDefinition::impl_getTitleHelper_throw() uno::Reference< frame::XDesktop2 > xDesktop = frame::Desktop::create(m_aProps->m_xContext); uno::Reference< frame::XModel > xThis(static_cast< frame::XModel* >(this), uno::UNO_QUERY_THROW); - rtl::Reference<::framework::TitleHelper> pHelper = new ::framework::TitleHelper( m_aProps->m_xContext ); - m_pImpl->m_xTitleHelper = pHelper; - pHelper->setOwner (xThis ); - pHelper->connectWithUntitledNumbers (uno::Reference(xDesktop, uno::UNO_QUERY_THROW)); + m_pImpl->m_xTitleHelper = new ::framework::TitleHelper( m_aProps->m_xContext, xThis, + uno::Reference(xDesktop, uno::UNO_QUERY_THROW) ); } return m_pImpl->m_xTitleHelper; -- cgit