sw/source/core/doc/docbm.cxx | 38 +++++++++++++++++++------------------- 1 file changed, 19 insertions(+), 19 deletions(-)
New commits: commit 574abb8e496ba36732ee7f51b93c781a766a3233 Author: Mike Kaganski <[email protected]> AuthorDate: Tue Oct 28 11:52:21 2025 +0500 Commit: Mike Kaganski <[email protected]> CommitDate: Tue Oct 28 11:26:22 2025 +0100 Use references instead of pointers Change-Id: I3669f463cf4a1344fba2a58db2e280c4e9813535 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/193068 Reviewed-by: Mike Kaganski <[email protected]> Tested-by: Jenkins diff --git a/sw/source/core/doc/docbm.cxx b/sw/source/core/doc/docbm.cxx index fa864a001a8e..28ac878e2825 100644 --- a/sw/source/core/doc/docbm.cxx +++ b/sw/source/core/doc/docbm.cxx @@ -414,51 +414,51 @@ namespace static bool IsNavigatorReminder(const MarkBase& rBkmk) { - const std::type_info* const pMarkTypeInfo = &typeid(rBkmk); + const std::type_info& rMarkTypeInfo = typeid(rBkmk); // not using dynamic_cast<> here for performance - return (*pMarkTypeInfo == typeid(NavigatorReminder)); + return (rMarkTypeInfo == typeid(NavigatorReminder)); } static bool IsCrossRefBookmark(const sw::mark::MarkBase& rBkmk) { // not using dynamic_cast<> here for performance - const std::type_info* const pMarkTypeInfo = &typeid(rBkmk); - return (*pMarkTypeInfo == typeid(CrossRefHeadingBookmark)) - || (*pMarkTypeInfo == typeid(CrossRefNumItemBookmark)); + const std::type_info& rMarkTypeInfo = typeid(rBkmk); + return (rMarkTypeInfo == typeid(CrossRefHeadingBookmark)) + || (rMarkTypeInfo == typeid(CrossRefNumItemBookmark)); } static bool IsAnnotationMark(const sw::mark::MarkBase& rBkmk) { // not using dynamic_cast<> here for performance - const std::type_info* const pMarkTypeInfo = &typeid(rBkmk); - return (*pMarkTypeInfo == typeid(AnnotationMark)); + const std::type_info& rMarkTypeInfo = typeid(rBkmk); + return (rMarkTypeInfo == typeid(AnnotationMark)); } IDocumentMarkAccess::MarkType IDocumentMarkAccess::GetType(const MarkBase& rBkmk) { - const std::type_info* const pMarkTypeInfo = &typeid(rBkmk); + const std::type_info& rMarkTypeInfo = typeid(rBkmk); // not using dynamic_cast<> here for performance - if(*pMarkTypeInfo == typeid(UnoMark)) + if(rMarkTypeInfo == typeid(UnoMark)) return MarkType::UNO_BOOKMARK; - else if(*pMarkTypeInfo == typeid(DdeBookmark)) + else if(rMarkTypeInfo == typeid(DdeBookmark)) return MarkType::DDE_BOOKMARK; - else if(*pMarkTypeInfo == typeid(Bookmark)) + else if(rMarkTypeInfo == typeid(Bookmark)) return MarkType::BOOKMARK; - else if(*pMarkTypeInfo == typeid(CrossRefHeadingBookmark)) + else if(rMarkTypeInfo == typeid(CrossRefHeadingBookmark)) return MarkType::CROSSREF_HEADING_BOOKMARK; - else if(*pMarkTypeInfo == typeid(CrossRefNumItemBookmark)) + else if(rMarkTypeInfo == typeid(CrossRefNumItemBookmark)) return MarkType::CROSSREF_NUMITEM_BOOKMARK; - else if(*pMarkTypeInfo == typeid(AnnotationMark)) + else if(rMarkTypeInfo == typeid(AnnotationMark)) return MarkType::ANNOTATIONMARK; - else if(*pMarkTypeInfo == typeid(TextFieldmark)) + else if(rMarkTypeInfo == typeid(TextFieldmark)) return MarkType::TEXT_FIELDMARK; - else if(*pMarkTypeInfo == typeid(CheckboxFieldmark)) + else if(rMarkTypeInfo == typeid(CheckboxFieldmark)) return MarkType::CHECKBOX_FIELDMARK; - else if(*pMarkTypeInfo == typeid(DropDownFieldmark)) + else if(rMarkTypeInfo == typeid(DropDownFieldmark)) return MarkType::DROPDOWN_FIELDMARK; - else if(*pMarkTypeInfo == typeid(DateFieldmark)) + else if(rMarkTypeInfo == typeid(DateFieldmark)) return MarkType::DATE_FIELDMARK; - else if(*pMarkTypeInfo == typeid(NavigatorReminder)) + else if(rMarkTypeInfo == typeid(NavigatorReminder)) return MarkType::NAVIGATOR_REMINDER; else {
