desktop/source/deployment/inc/dp_ucb.h | 7 +------ desktop/source/deployment/misc/dp_ucb.cxx | 4 ++-- 2 files changed, 3 insertions(+), 8 deletions(-)
New commits: commit 8d9ac64eea1c87169745f92c2703594e015971f6 Author: Mike Kaganski <mike.kagan...@collabora.com> AuthorDate: Wed Jul 23 11:36:22 2025 +0200 Commit: Mike Kaganski <mike.kagan...@collabora.com> CommitDate: Wed Jul 23 14:39:17 2025 +0200 Drop StrTitle::getTitleSequence It is unhelpful; its use requires to know what it does anyway. Having both the name sequence and value sequence be explicit improves readability. Change-Id: I9ee73e4a6c4f19b979307dfe93b63dbf618cc105 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/188216 Tested-by: Jenkins Reviewed-by: Mike Kaganski <mike.kagan...@collabora.com> diff --git a/desktop/source/deployment/inc/dp_ucb.h b/desktop/source/deployment/inc/dp_ucb.h index dddd8e38d9f6..c44d58340b75 100644 --- a/desktop/source/deployment/inc/dp_ucb.h +++ b/desktop/source/deployment/inc/dp_ucb.h @@ -29,11 +29,6 @@ namespace dp_misc { struct DESKTOP_DEPLOYMENTMISC_DLLPUBLIC StrTitle { - static css::uno::Sequence< OUString > getTitleSequence() - { - css::uno::Sequence<OUString> aSeq { u"Title"_ustr }; - return aSeq; - } static OUString getTitle( ::ucbhelper::Content &rContent ) { return rContent.getPropertyValue(u"Title"_ustr).get<OUString>(); @@ -43,7 +38,7 @@ struct DESKTOP_DEPLOYMENTMISC_DLLPUBLIC StrTitle createCursor( ::ucbhelper::Content &rContent, ucbhelper::ResultSetInclude eInclude ) { - return rContent.createCursor( StrTitle::getTitleSequence(), eInclude ); + return rContent.createCursor({ u"Title"_ustr }, eInclude); } }; diff --git a/desktop/source/deployment/misc/dp_ucb.cxx b/desktop/source/deployment/misc/dp_ucb.cxx index cb92c8bd76a5..ea895c9eaa7e 100644 --- a/desktop/source/deployment/misc/dp_ucb.cxx +++ b/desktop/source/deployment/misc/dp_ucb.cxx @@ -130,8 +130,8 @@ bool create_folder( try { if (parentContent.insertNewContent( info.Type, - StrTitle::getTitleSequence(), - Sequence<Any>( &title, 1 ), + { u"Title"_ustr }, + { title }, ucb_content )) { if (ret_ucb_content != nullptr) *ret_ucb_content = ucb_content;