diff options
author | Noel Grandin <noel.grandin@collabora.co.uk> | 2025-03-20 08:14:33 +0200 |
---|---|---|
committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2025-03-20 16:43:56 +0100 |
commit | dfe21d378722e90218f2b97cea5c7a449d00b48c (patch) | |
tree | 65cd6e8ce78ef12b48d84926c0d20239599f6c03 /oox/source/export/drawingml.cxx | |
parent | 76c2a523c6ab19bdaed2b4ad068348b9376d9c59 (diff) |
loplugin:constparam in oox
Change-Id: Iadec18cc82fd3fa8fb129698be92fed4ec2fbb1f
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/183144
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Tested-by: Jenkins
Diffstat (limited to 'oox/source/export/drawingml.cxx')
-rw-r--r-- | oox/source/export/drawingml.cxx | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/oox/source/export/drawingml.cxx b/oox/source/export/drawingml.cxx index 77baafbe755d..6f9905a4c612 100644 --- a/oox/source/export/drawingml.cxx +++ b/oox/source/export/drawingml.cxx @@ -2427,7 +2427,7 @@ void DrawingML::WriteShapeTransformation( const Reference< XShape >& rXShape, sa bFlipHWrite, bFlipVWrite, ExportRotateClockwisify(nRotation + nCameraRotation), IsGroupShape( rXShape )); } -static OUString lcl_GetTarget(const css::uno::Reference<css::frame::XModel>& xModel, OUString& rURL) +static OUString lcl_GetTarget(const css::uno::Reference<css::frame::XModel>& xModel, std::u16string_view rURL) { Reference<drawing::XDrawPagesSupplier> xDPS(xModel, uno::UNO_QUERY_THROW); Reference<drawing::XDrawPages> xDrawPages(xDPS->getDrawPages(), uno::UNO_SET_THROW); @@ -2450,9 +2450,9 @@ static OUString lcl_GetTarget(const css::uno::Reference<css::frame::XModel>& xMo } if (sTarget.isEmpty()) { - sal_Int32 nSplit = rURL.lastIndexOf(' '); - if (nSplit > -1) - sTarget = OUString::Concat("slide") + rURL.subView(nSplit + 1) + ".xml"; + size_t nSplit = rURL.rfind(' '); + if (nSplit != std::u16string_view::npos) + sTarget = OUString::Concat("slide") + rURL.substr(nSplit + 1) + ".xml"; } return sTarget; |