editeng/source/editeng/impedit3.cxx |    9 ++++-----
 1 file changed, 4 insertions(+), 5 deletions(-)

New commits:
commit 707db60df761a98cfdbd771fdc85979084be0b33
Author:     Mike Kaganski <mike.kagan...@collabora.com>
AuthorDate: Tue Apr 27 12:35:20 2021 +0200
Commit:     Mike Kaganski <mike.kagan...@collabora.com>
CommitDate: Tue Apr 27 16:54:16 2021 +0200

    Small refactor
    
    Change-Id: I987cd80bb485d3d811c0932c0822efc5d572dfd0
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/114712
    Tested-by: Jenkins
    Reviewed-by: Mike Kaganski <mike.kagan...@collabora.com>

diff --git a/editeng/source/editeng/impedit3.cxx 
b/editeng/source/editeng/impedit3.cxx
index b91b2c476628..918e9346bbfd 100644
--- a/editeng/source/editeng/impedit3.cxx
+++ b/editeng/source/editeng/impedit3.cxx
@@ -2390,16 +2390,15 @@ void ImpEditEngine::CreateTextPortions( ParaPortion* 
pParaPortion, sal_Int32& rS
     o3tl::sorted_vector< sal_Int32 > aPositions;
     aPositions.insert( 0 );
 
-    sal_uInt16 nAttr = 0;
-    EditCharAttrib* pAttrib = GetAttrib( pNode->GetCharAttribs().GetAttribs(), 
nAttr );
-    while ( pAttrib )
+    for (sal_uInt16 nAttr = 0;; ++nAttr)
     {
         // Insert Start and End into the Array...
         // The Insert method does not allow for duplicate values...
+        EditCharAttrib* pAttrib = 
GetAttrib(pNode->GetCharAttribs().GetAttribs(), nAttr);
+        if (!pAttrib)
+            break;
         aPositions.insert( pAttrib->GetStart() );
         aPositions.insert( pAttrib->GetEnd() );
-        nAttr++;
-        pAttrib = GetAttrib( pNode->GetCharAttribs().GetAttribs(), nAttr );
     }
     aPositions.insert( pNode->Len() );
 
_______________________________________________
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits

Reply via email to