[Libreoffice-commits] core.git: basegfx/source

Stephan Bergmann sbergman at redhat.com
Sun Feb 26 20:57:07 UTC 2017


 basegfx/source/polygon/b2dpolypolygoncutter.cxx |    7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

New commits:
commit 8b7036450f46c87f0b6f9ac40ccd69d5a57cac55
Author: Stephan Bergmann <sbergman at redhat.com>
Date:   Sun Feb 26 21:56:36 2017 +0100

    loplugin:loopvartoosmall
    
    Change-Id: I085d409f3271c5a4d8237fa3e3c4a74bed296acb

diff --git a/basegfx/source/polygon/b2dpolypolygoncutter.cxx b/basegfx/source/polygon/b2dpolypolygoncutter.cxx
index 88cafde..f2cbf21 100644
--- a/basegfx/source/polygon/b2dpolypolygoncutter.cxx
+++ b/basegfx/source/polygon/b2dpolypolygoncutter.cxx
@@ -1047,16 +1047,17 @@ namespace basegfx
                         const B2DRange aCandidateRange(aCandidate.getB2DRange());
                         bool bCouldMergeSimple(false);
 
-                        for(sal_uInt32 b(0); !bCouldMergeSimple && b < aResult.size(); b++)
+                        for(auto & b: aResult)
                         {
-                            basegfx::B2DPolyPolygon aTarget(aResult[b]);
+                            basegfx::B2DPolyPolygon aTarget(b);
                             const B2DRange aTargetRange(aTarget.getB2DRange());
 
                             if(!aCandidateRange.overlaps(aTargetRange))
                             {
                                 aTarget.append(aCandidate);
-                                aResult[b] = aTarget;
+                                b = aTarget;
                                 bCouldMergeSimple = true;
+                                break;
                             }
                         }
 


More information about the Libreoffice-commits mailing list