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

[Miklos Vajna (via logerrit)]

 sw/source/core/layout/tabfrm.cxx |   14 +++++++-------
 1 file changed, 7 insertions(+), 7 deletions(-)

New commits:
commit db8db4732cd50342139c1d2c358a4ce260c4b4f3
Author:     Miklos Vajna <vmiklos at collabora.com>
AuthorDate: Thu May 20 21:05:58 2021 +0200
Commit:     Miklos Vajna <vmiklos at collabora.com>
CommitDate: Fri May 21 09:06:56 2021 +0200

    Replace magic numbers in SwTabFrame::UpdateAttr_()
    
    By using the same SwTabFrameInvFlags constants that
    SwTabFrame::SwClientNotify() already does.
    
    Change-Id: I308219262d9d18c41e38290fa2e43bf90ba77b5b
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/115887
    Tested-by: Jenkins
    Reviewed-by: Miklos Vajna <vmiklos at collabora.com>

diff --git a/sw/source/core/layout/tabfrm.cxx b/sw/source/core/layout/tabfrm.cxx
index 34ba690a25b8..fe3016b0037a 100644
--- a/sw/source/core/layout/tabfrm.cxx
+++ b/sw/source/core/layout/tabfrm.cxx
@@ -3356,18 +3356,18 @@ void SwTabFrame::UpdateAttr_( const SfxPoolItem *pOld, const SfxPoolItem *pNew,
                     pHeadline->Paste( this, pLowerRow );
                 }
             }
-            rInvFlags |= static_cast<SwTabFrameInvFlags>(0x02);
+            rInvFlags |= SwTabFrameInvFlags::InvalidatePrt;
             break;
 
         case RES_FRM_SIZE:
         case RES_HORI_ORIENT:
-            rInvFlags |= static_cast<SwTabFrameInvFlags>(0x22);
+            rInvFlags |= SwTabFrameInvFlags::InvalidatePrt | SwTabFrameInvFlags::InvalidateBrowseWidth;
             break;
 
         case RES_PAGEDESC:                      //Attribute changes (on/off)
             if ( IsInDocBody() )
             {
-                rInvFlags |= static_cast<SwTabFrameInvFlags>(0x40);
+                rInvFlags |= SwTabFrameInvFlags::InvalidatePos;
                 SwPageFrame *pPage = FindPageFrame();
                 if (pPage)
                 {
@@ -3382,23 +3382,23 @@ void SwTabFrame::UpdateAttr_( const SfxPoolItem *pOld, const SfxPoolItem *pNew,
             break;
 
         case RES_BREAK:
-            rInvFlags |= static_cast<SwTabFrameInvFlags>(0xC0);
+            rInvFlags |= SwTabFrameInvFlags::InvalidatePos | SwTabFrameInvFlags::InvalidateNextPos;
             break;
 
         case RES_LAYOUT_SPLIT:
             if ( !IsFollow() )
-                rInvFlags |= static_cast<SwTabFrameInvFlags>(0x40);
+                rInvFlags |= SwTabFrameInvFlags::InvalidatePos;
             break;
         case RES_FRAMEDIR :
             SetDerivedR2L( false );
             CheckDirChange();
             break;
         case RES_COLLAPSING_BORDERS :
-            rInvFlags |= static_cast<SwTabFrameInvFlags>(0x02);
+            rInvFlags |= SwTabFrameInvFlags::InvalidatePrt;
             lcl_InvalidateAllLowersPrt( this );
             break;
         case RES_UL_SPACE:
-            rInvFlags |= static_cast<SwTabFrameInvFlags>(0x1C);
+            rInvFlags |= SwTabFrameInvFlags::InvalidateIndNextPrt | SwTabFrameInvFlags::InvalidatePrevPrt | SwTabFrameInvFlags::SetIndNextCompletePaint;
             [[fallthrough]];
 
         default: