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

[Mike Kaganski (via logerrit)]

 editeng/source/editeng/impedit3.cxx |    3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

New commits:
commit 825673eac77e2ccef29cfe2d9a1f375c6f5606d7
Author:     Mike Kaganski <mike.kaganski at collabora.com>
AuthorDate: Thu May 20 10:49:14 2021 +0200
Commit:     Mike Kaganski <mike.kaganski at collabora.com>
CommitDate: Thu May 20 11:50:42 2021 +0200

    This check is redundant here
    
    The check immediately above this guarantees that
    !rParaPortion.IsInvalid() may only be true when
    rParaPortion.MustRepaint() is true.
    
    Change-Id: Icedeac87b3b8125997640d330b9612eed2cb5095
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/115834
    Tested-by: Jenkins
    Reviewed-by: Mike Kaganski <mike.kaganski at collabora.com>

diff --git a/editeng/source/editeng/impedit3.cxx b/editeng/source/editeng/impedit3.cxx
index 454f65286416..9576d079574f 100644
--- a/editeng/source/editeng/impedit3.cxx
+++ b/editeng/source/editeng/impedit3.cxx
@@ -383,8 +383,7 @@ void ImpEditEngine::FormatDoc()
         if ( rParaPortion.MustRepaint() || ( rParaPortion.IsInvalid() && rParaPortion.IsVisible() ) )
         {
             // No formatting should be necessary for MustRepaint()!
-            if ( ( rParaPortion.MustRepaint() && !rParaPortion.IsInvalid() )
-                    || CreateLines( nPara, nY ) )
+            if ( !rParaPortion.IsInvalid() || CreateLines( nPara, nY ) )
             {
                 if ( !bGrow && GetTextRanger() )
                 {