sc/source/core/data/PivotTableFormatOutput.cxx | 4 ++-- sd/source/filter/eppt/pptexanimations.cxx | 4 ++-- sd/source/ui/tools/SlideshowLayerRenderer.cxx | 8 ++++---- 3 files changed, 8 insertions(+), 8 deletions(-)
New commits: commit 968b337faf08a8c9177df5b6a3d1e2f4b1861109 Author: toadkarter <[email protected]> AuthorDate: Wed Sep 10 10:19:57 2025 +0100 Commit: Hossein <[email protected]> CommitDate: Fri Sep 12 11:56:29 2025 +0200 tdf#145538 Use range based for loops Adds range based for loops to sc and sd modules where appropriate. Change-Id: I5261466ac2de0397dcb506818d92775d63734a40 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/190776 Tested-by: Jenkins Reviewed-by: Hossein <[email protected]> diff --git a/sc/source/core/data/PivotTableFormatOutput.cxx b/sc/source/core/data/PivotTableFormatOutput.cxx index 2eef2b311e62..45308a2129a0 100644 --- a/sc/source/core/data/PivotTableFormatOutput.cxx +++ b/sc/source/core/data/PivotTableFormatOutput.cxx @@ -70,9 +70,9 @@ public: void initLines(std::vector<LineData>& rLines, std::vector<ScDPOutLevelData> const& rFields) { - for (size_t i = 0; i < rFields.size(); i++) + for (const ScDPOutLevelData& rField : rFields) { - size_t nFieldLength(rFields[i].maResult.getLength()); + size_t nFieldLength(rField.maResult.getLength()); if (rLines.size() < nFieldLength) rLines.resize(nFieldLength); diff --git a/sd/source/filter/eppt/pptexanimations.cxx b/sd/source/filter/eppt/pptexanimations.cxx index 3c19f018a588..a3b95012c847 100644 --- a/sd/source/filter/eppt/pptexanimations.cxx +++ b/sd/source/filter/eppt/pptexanimations.cxx @@ -1189,8 +1189,8 @@ void AnimationExporter::exportAnimPropertyByte( SvStream& rStrm, const sal_uInt1 void AnimationExporter::writeZString( SvStream& rStrm, std::u16string_view aVal ) { - for ( size_t i = 0; i < aVal.size(); i++ ) - rStrm.WriteUInt16( aVal[ i ] ); + for ( const sal_uInt16 codeUnit : aVal ) + rStrm.WriteUInt16( codeUnit ); rStrm.WriteUInt16( 0 ); } diff --git a/sd/source/ui/tools/SlideshowLayerRenderer.cxx b/sd/source/ui/tools/SlideshowLayerRenderer.cxx index 4bd8405be04d..53cba4849b64 100644 --- a/sd/source/ui/tools/SlideshowLayerRenderer.cxx +++ b/sd/source/ui/tools/SlideshowLayerRenderer.cxx @@ -237,9 +237,9 @@ void changeBackground( drawinglayer::primitive2d::Primitive2DContainer const& rContainer, bool bRenderObject, std::vector<drawinglayer::primitive2d::Primitive2DReference>& rPrimitivesToUnhide) { - for (size_t i = 0; i < rContainer.size(); i++) + for (const drawinglayer::primitive2d::Primitive2DReference& rBasePrimitive : rContainer) { - drawinglayer::primitive2d::BasePrimitive2D* pBasePrimitive = rContainer[i].get(); + drawinglayer::primitive2d::BasePrimitive2D* pBasePrimitive = rBasePrimitive.get(); if (pBasePrimitive->getPrimitive2DID() == PRIMITIVE2D_ID_SDRRECTANGLEPRIMITIVE2D) { drawinglayer::primitive2d::Primitive2DContainer aPrimitiveContainer; @@ -255,9 +255,9 @@ drawinglayer::primitive2d::TextHierarchyBlockPrimitive2D* findTextBlock(drawinglayer::primitive2d::Primitive2DContainer const& rContainer, drawinglayer::geometry::ViewInformation2D const& rViewInformation2D) { - for (size_t i = 0; i < rContainer.size(); i++) + for (const drawinglayer::primitive2d::Primitive2DReference& rBasePrimitive : rContainer) { - drawinglayer::primitive2d::BasePrimitive2D* pBasePrimitive = rContainer[i].get(); + drawinglayer::primitive2d::BasePrimitive2D* pBasePrimitive = rBasePrimitive.get(); if (pBasePrimitive->getPrimitive2DID() == PRIMITIVE2D_ID_TEXTHIERARCHYBLOCKPRIMITIVE2D) { auto* pPrimitive
