sd/source/core/drawdoc2.cxx | 14 ++++++-------- 1 file changed, 6 insertions(+), 8 deletions(-)
New commits: commit dd1870369618b2f1fcc7faa9c8e217e556e56c1a Author: Caolán McNamara <caolan.mcnam...@collabora.com> AuthorDate: Mon Oct 21 17:33:39 2024 +0100 Commit: Caolán McNamara <caolan.mcnam...@collabora.com> CommitDate: Mon Oct 21 22:20:46 2024 +0200 don't need to const_cast away constness for SdPage Change-Id: I44fd36841319eeb3deda7becaf04ec3188586788 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/175368 Tested-by: Jenkins Reviewed-by: Caolán McNamara <caolan.mcnam...@collabora.com> diff --git a/sd/source/core/drawdoc2.cxx b/sd/source/core/drawdoc2.cxx index c790ac1130a9..8a3380cf9475 100644 --- a/sd/source/core/drawdoc2.cxx +++ b/sd/source/core/drawdoc2.cxx @@ -125,7 +125,7 @@ SdrObject* SdDrawDocument::GetObj(std::u16string_view rObjName) const // Find SdPage by name sal_uInt16 SdDrawDocument::GetPageByName(std::u16string_view rPgName, bool& rbIsMasterPage) const { - SdPage* pPage = nullptr; + const SdPage* pPage = nullptr; sal_uInt16 nPage = 0; const sal_uInt16 nMaxPages = GetPageCount(); sal_uInt16 nPageNum = SDRPAGE_NOTFOUND; @@ -136,8 +136,7 @@ sal_uInt16 SdDrawDocument::GetPageByName(std::u16string_view rPgName, bool& rbIs // ignored) while (nPage < nMaxPages && nPageNum == SDRPAGE_NOTFOUND) { - pPage = const_cast<SdPage*>(static_cast<const SdPage*>( - GetPage(nPage))); + pPage = static_cast<const SdPage*>(GetPage(nPage)); if (pPage != nullptr && pPage->GetPageKind() != PageKind::Handout @@ -155,8 +154,7 @@ sal_uInt16 SdDrawDocument::GetPageByName(std::u16string_view rPgName, bool& rbIs while (nPage < nMaxMasterPages && nPageNum == SDRPAGE_NOTFOUND) { - pPage = const_cast<SdPage*>(static_cast<const SdPage*>( - GetMasterPage(nPage))); + pPage = static_cast<const SdPage*>(GetMasterPage(nPage)); if (pPage && pPage->GetName() == rPgName) { @@ -173,14 +171,14 @@ sal_uInt16 SdDrawDocument::GetPageByName(std::u16string_view rPgName, bool& rbIs bool SdDrawDocument::IsPageNameUnique( std::u16string_view rPgName ) const { sal_uInt16 nCount = 0; - SdPage* pPage = nullptr; + const SdPage* pPage = nullptr; // Search all regular pages and all notes pages (handout pages are ignored) sal_uInt16 nPage = 0; sal_uInt16 nMaxPages = GetPageCount(); while (nPage < nMaxPages) { - pPage = const_cast<SdPage*>(static_cast<const SdPage*>(GetPage(nPage))); + pPage = static_cast<const SdPage*>(GetPage(nPage)); if (pPage && pPage->GetName() == rPgName && pPage->GetPageKind() != PageKind::Handout) nCount++; @@ -193,7 +191,7 @@ bool SdDrawDocument::IsPageNameUnique( std::u16string_view rPgName ) const nMaxPages = GetMasterPageCount(); while (nPage < nMaxPages) { - pPage = const_cast<SdPage*>(static_cast<const SdPage*>(GetMasterPage(nPage))); + pPage = static_cast<const SdPage*>(GetMasterPage(nPage)); if (pPage && pPage->GetName() == rPgName) nCount++;