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

[Libreoffice Gerrit user]

 sw/source/core/layout/wsfrm.cxx |    3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

New commits:
commit 360f389e68a44a8dae896a9ccbdb20b9636a0041
Author:     Xisco Fauli <xiscofauli at libreoffice.org>
AuthorDate: Sat Oct 20 14:16:04 2018 +0200
Commit:     Caolán McNamara <caolanm at redhat.com>
CommitDate: Sun Oct 21 18:20:51 2018 +0200

    tdf#120703 (PVS): Recurring check.
    
    V571 The 'if (IsTabFrame())' condition was already verified in line 2879.
    
    Change-Id: I738d818207d7303bc914b140c73feeb182194b27
    Reviewed-on: https://gerrit.libreoffice.org/62047
    Reviewed-by: Caolán McNamara <caolanm at redhat.com>
    Tested-by: Caolán McNamara <caolanm at redhat.com>

diff --git a/sw/source/core/layout/wsfrm.cxx b/sw/source/core/layout/wsfrm.cxx
index c8648be49808..fa8c6be8254c 100644
--- a/sw/source/core/layout/wsfrm.cxx
+++ b/sw/source/core/layout/wsfrm.cxx
@@ -2881,8 +2881,7 @@ SwTwips SwLayoutFrame::ShrinkFrame( SwTwips nDist, bool bTst, bool bInfo )
                 SetRetouche();
             if ( IsTabFrame() )
             {
-                if( IsTabFrame() )
-                    static_cast<SwTabFrame*>(this)->SetComplete();
+                static_cast<SwTabFrame*>(this)->SetComplete();
                 if ( Lower() )  // Can also be in the Join and be empty!
                     InvalidateNextPos();
             }