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

Miklos Vajna (via logerrit) logerrit at kemper.freedesktop.org
Tue Jan 12 19:30:54 UTC 2021


 sw/source/core/text/itrcrsr.cxx |   52 +++++++++----------
 sw/source/core/text/itrtxt.hxx  |   62 +++++++++++------------
 sw/source/core/text/porexp.cxx  |    4 -
 sw/source/core/text/porfld.cxx  |    2 
 sw/source/core/text/porfly.cxx  |   12 ++--
 sw/source/core/text/porlay.cxx  |   30 +++++------
 sw/source/core/text/porlay.hxx  |   14 ++---
 sw/source/core/text/porlin.cxx  |    6 +-
 sw/source/core/text/porlin.hxx  |  106 ++++++++++++++++++++--------------------
 sw/source/core/text/porrst.cxx  |    8 +--
 sw/source/core/text/possiz.hxx  |   34 ++++++------
 sw/source/core/text/txttab.cxx  |    2 
 12 files changed, 166 insertions(+), 166 deletions(-)

New commits:
commit e56a6707137c20c3626fc697a08548bd9951e85a
Author:     Miklos Vajna <vmiklos at collabora.com>
AuthorDate: Mon Jan 11 09:07:13 2021 +0100
Commit:     Miklos Vajna <vmiklos at collabora.com>
CommitDate: Tue Jan 12 20:30:10 2021 +0100

    sw: prefix members of SwLinePortion, SwPosSize, SwRepaint and SwTextMargin
    
    See tdf#94879 for motivation.
    
    Change-Id: I83fc138714ad170fad0465b884d6297134a0d806
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/109076
    Tested-by: Jenkins
    Reviewed-by: Miklos Vajna <vmiklos at collabora.com>

diff --git a/sw/source/core/text/itrcrsr.cxx b/sw/source/core/text/itrcrsr.cxx
index a4e3dca7c1ec..327c178497e2 100644
--- a/sw/source/core/text/itrcrsr.cxx
+++ b/sw/source/core/text/itrcrsr.cxx
@@ -184,7 +184,7 @@ void SwTextMargin::CtorInitTextMargin( SwTextFrame *pNewFrame, SwTextSizeInfo *p
     if ( m_pFrame->IsRightToLeft() )
     {
         // this calculation is identical this the calculation for L2R layout - see below
-        nLeft = m_pFrame->getFrameArea().Left() +
+        mnLeft = m_pFrame->getFrameArea().Left() +
                 m_pFrame->getFramePrintArea().Left() +
                 nLMWithNum -
                 pNode->GetLeftMarginWithNum() -
@@ -203,7 +203,7 @@ void SwTextMargin::CtorInitTextMargin( SwTextFrame *pNewFrame, SwTextSizeInfo *p
              !pNode->getIDocumentSettingAccess()->get(DocumentSettingId::IGNORE_FIRST_LINE_INDENT_IN_NUMBERING) )
         {
             // this calculation is identical this the calculation for R2L layout - see above
-            nLeft = m_pFrame->getFrameArea().Left() +
+            mnLeft = m_pFrame->getFrameArea().Left() +
                     m_pFrame->getFramePrintArea().Left() +
                     nLMWithNum -
                     pNode->GetLeftMarginWithNum() -
@@ -215,28 +215,28 @@ void SwTextMargin::CtorInitTextMargin( SwTextFrame *pNewFrame, SwTextSizeInfo *p
         }
         else
         {
-            nLeft = m_pFrame->getFrameArea().Left() +
+            mnLeft = m_pFrame->getFrameArea().Left() +
                     std::max( tools::Long( rSpace.GetTextLeft() + nLMWithNum ),
                          m_pFrame->getFramePrintArea().Left() );
         }
     }
 
-    nRight = m_pFrame->getFrameArea().Left() + m_pFrame->getFramePrintArea().Left() + m_pFrame->getFramePrintArea().Width();
+    mnRight = m_pFrame->getFrameArea().Left() + m_pFrame->getFramePrintArea().Left() + m_pFrame->getFramePrintArea().Width();
 
-    if( nLeft >= nRight &&
+    if( mnLeft >= mnRight &&
          // #i53066# Omit adjustment of nLeft for numbered
          // paras inside cells inside new documents:
         ( pNode->getIDocumentSettingAccess()->get(DocumentSettingId::IGNORE_FIRST_LINE_INDENT_IN_NUMBERING) ||
           !m_pFrame->IsInTab() ||
           ( !nLMWithNum && (!bLabelAlignmentActive || bListLevelIndentsApplicable) ) ) )
     {
-        nLeft = m_pFrame->getFramePrintArea().Left() + m_pFrame->getFrameArea().Left();
-        if( nLeft >= nRight )   // e.g. with large paragraph indentations in slim table columns
-            nRight = nLeft + 1; // einen goennen wir uns immer
+        mnLeft = m_pFrame->getFramePrintArea().Left() + m_pFrame->getFrameArea().Left();
+        if( mnLeft >= mnRight )   // e.g. with large paragraph indentations in slim table columns
+            mnRight = mnLeft + 1; // einen goennen wir uns immer
     }
 
     if( m_pFrame->IsFollow() && m_pFrame->GetOffset() )
-        nFirst = nLeft;
+        mnFirst = mnLeft;
     else
     {
         short nFLOfst = 0;
@@ -306,20 +306,20 @@ void SwTextMargin::CtorInitTextMargin( SwTextFrame *pNewFrame, SwTextSizeInfo *p
              !pNode->getIDocumentSettingAccess()->get(DocumentSettingId::IGNORE_FIRST_LINE_INDENT_IN_NUMBERING) )
         {
             if ( nFirstLineOfs < 0 && m_pFrame->IsInTab() &&
-                 nLeft == m_pFrame->getFramePrintArea().Left() + m_pFrame->getFrameArea().Left() &&
+                 mnLeft == m_pFrame->getFramePrintArea().Left() + m_pFrame->getFrameArea().Left() &&
                  !m_pFrame->IsRightToLeft() &&
                  !bListLevelIndentsApplicableAndLabelAlignmentActive )
             {
                 // tdf#130218 always show hanging indent in narrow table cells
                 // to avoid hiding the text content of the first line
-                nLeft -= nFirstLineOfs;
+                mnLeft -= nFirstLineOfs;
             }
 
-            nFirst = nLeft + nFirstLineOfs;
+            mnFirst = mnLeft + nFirstLineOfs;
         }
         else
         {
-              nFirst = m_pFrame->getFrameArea().Left() +
+              mnFirst = m_pFrame->getFrameArea().Left() +
                      std::max( rSpace.GetTextLeft() + nLMWithNum+ nFirstLineOfs,
                           m_pFrame->getFramePrintArea().Left() );
         }
@@ -328,21 +328,21 @@ void SwTextMargin::CtorInitTextMargin( SwTextFrame *pNewFrame, SwTextSizeInfo *p
         //       value for the new list label position and space mode LABEL_ALIGNMENT
         //       and label alignment CENTER and RIGHT in L2R layout respectively
         //       label alignment LEFT and CENTER in R2L layout
-        nFirst += m_pFrame->GetAdditionalFirstLineOffset();
+        mnFirst += m_pFrame->GetAdditionalFirstLineOffset();
 
-        if( nFirst >= nRight )
-            nFirst = nRight - 1;
+        if( mnFirst >= mnRight )
+            mnFirst = mnRight - 1;
     }
     const SvxAdjustItem& rAdjust = m_pFrame->GetTextNodeForParaProps()->GetSwAttrSet().GetAdjust();
-    nAdjust = rAdjust.GetAdjust();
+    mnAdjust = rAdjust.GetAdjust();
 
     // left is left and right is right
     if ( m_pFrame->IsRightToLeft() )
     {
-        if ( SvxAdjust::Left == nAdjust )
-            nAdjust = SvxAdjust::Right;
-        else if ( SvxAdjust::Right == nAdjust )
-            nAdjust = SvxAdjust::Left;
+        if ( SvxAdjust::Left == mnAdjust )
+            mnAdjust = SvxAdjust::Right;
+        else if ( SvxAdjust::Right == mnAdjust )
+            mnAdjust = SvxAdjust::Left;
     }
 
     m_bOneBlock = rAdjust.GetOneWord() == SvxAdjust::Block;
@@ -357,17 +357,17 @@ void SwTextMargin::CtorInitTextMargin( SwTextFrame *pNewFrame, SwTextSizeInfo *p
 
 void SwTextMargin::DropInit()
 {
-    nDropLeft = nDropLines = nDropHeight = nDropDescent = 0;
+    mnDropLeft = mnDropLines = mnDropHeight = mnDropDescent = 0;
     const SwParaPortion *pPara = GetInfo().GetParaPortion();
     if( pPara )
     {
         const SwDropPortion *pPorDrop = pPara->FindDropPortion();
         if ( pPorDrop )
         {
-            nDropLeft = pPorDrop->GetDropLeft();
-            nDropLines = pPorDrop->GetLines();
-            nDropHeight = pPorDrop->GetDropHeight();
-            nDropDescent = pPorDrop->GetDropDescent();
+            mnDropLeft = pPorDrop->GetDropLeft();
+            mnDropLines = pPorDrop->GetLines();
+            mnDropHeight = pPorDrop->GetDropHeight();
+            mnDropDescent = pPorDrop->GetDropDescent();
         }
     }
 }
diff --git a/sw/source/core/text/itrtxt.hxx b/sw/source/core/text/itrtxt.hxx
index 52bbbe06fd4a..8090b23a60a8 100644
--- a/sw/source/core/text/itrtxt.hxx
+++ b/sw/source/core/text/itrtxt.hxx
@@ -141,32 +141,32 @@ public:
 class SwTextMargin : public SwTextIter
 {
 private:
-          SwTwips nLeft;
-          SwTwips nRight;
-          SwTwips nFirst;
-          sal_uInt16  nDropLeft;
-          sal_uInt16  nDropHeight;
-          sal_uInt16  nDropDescent;
-          sal_uInt16  nDropLines;
-          SvxAdjust  nAdjust;
+          SwTwips mnLeft;
+          SwTwips mnRight;
+          SwTwips mnFirst;
+          sal_uInt16  mnDropLeft;
+          sal_uInt16  mnDropHeight;
+          sal_uInt16  mnDropDescent;
+          sal_uInt16  mnDropLines;
+          SvxAdjust  mnAdjust;
           // #i91133#
           SwTwips mnTabLeft;
 
 protected:
     // For FormatQuoVadis
-    void Right( const SwTwips nNew ) { nRight = nNew; }
+    void Right( const SwTwips nNew ) { mnRight = nNew; }
 
     void CtorInitTextMargin( SwTextFrame *pFrame, SwTextSizeInfo *pInf );
     explicit SwTextMargin(SwTextNode const * pTextNode)
         : SwTextIter(pTextNode)
-        , nLeft(0)
-        , nRight(0)
-        , nFirst(0)
-        , nDropLeft(0)
-        , nDropHeight(0)
-        , nDropDescent(0)
-        , nDropLines(0)
-        , nAdjust(SvxAdjust::Left)
+        , mnLeft(0)
+        , mnRight(0)
+        , mnFirst(0)
+        , mnDropLeft(0)
+        , mnDropHeight(0)
+        , mnDropDescent(0)
+        , mnDropLines(0)
+        , mnAdjust(SvxAdjust::Left)
         , mnTabLeft(0)
     {
     }
@@ -178,8 +178,8 @@ public:
     }
     inline SwTwips GetLeftMargin() const;
     inline SwTwips Left() const;
-    SwTwips Right() const { return nRight; }
-    SwTwips FirstLeft() const { return nFirst; }
+    SwTwips Right() const { return mnRight; }
+    SwTwips FirstLeft() const { return mnFirst; }
     SwTwips CurrWidth() const { return m_pCurr->PrtWidth(); }
            SwTwips GetLineStart() const;
     SwTwips GetLineEnd() const { return GetLineStart() + CurrWidth(); }
@@ -187,11 +187,11 @@ public:
     bool IsOneBlock() const { return m_bOneBlock; }
     bool IsLastBlock() const { return m_bLastBlock; }
     bool IsLastCenter() const { return m_bLastCenter; }
-    SvxAdjust GetAdjust() const { return nAdjust; }
+    SvxAdjust GetAdjust() const { return mnAdjust; }
     sal_uInt16 GetLineWidth() const
            { return sal_uInt16( Right() - GetLeftMargin() + 1 ); }
-    SwTwips GetLeftMin() const { return std::min(nFirst, nLeft); }
-    bool HasNegFirst() const { return nFirst < nLeft; }
+    SwTwips GetLeftMin() const { return std::min(mnFirst, mnLeft); }
+    bool HasNegFirst() const { return mnFirst < mnLeft; }
 
     // #i91133#
     SwTwips GetTabLeft() const
@@ -199,13 +199,13 @@ public:
         return mnTabLeft;
     }
     // DropCaps
-    sal_uInt16 GetDropLines() const { return nDropLines; }
-    void SetDropLines( const sal_uInt16 nNew ) { nDropLines = nNew; }
-    sal_uInt16 GetDropLeft() const { return nDropLeft; }
-    sal_uInt16 GetDropHeight() const { return nDropHeight; }
-    void SetDropHeight( const sal_uInt16 nNew ) { nDropHeight = nNew; }
-    sal_uInt16 GetDropDescent() const { return nDropDescent; }
-    void SetDropDescent( const sal_uInt16 nNew ) { nDropDescent = nNew; }
+    sal_uInt16 GetDropLines() const { return mnDropLines; }
+    void SetDropLines( const sal_uInt16 nNew ) { mnDropLines = nNew; }
+    sal_uInt16 GetDropLeft() const { return mnDropLeft; }
+    sal_uInt16 GetDropHeight() const { return mnDropHeight; }
+    void SetDropHeight( const sal_uInt16 nNew ) { mnDropHeight = nNew; }
+    sal_uInt16 GetDropDescent() const { return mnDropDescent; }
+    void SetDropDescent( const sal_uInt16 nNew ) { mnDropDescent = nNew; }
     void DropInit();
 
     // Returns the TextPos for start and end of the current line without whitespace
@@ -327,12 +327,12 @@ inline bool SwTextIter::SeekStartAndChg( SwTextSizeInfo &rInf, const bool bPara
 
 inline SwTwips SwTextMargin::GetLeftMargin() const
 {
-    return IsFirstTextLine() ? nFirst : Left();
+    return IsFirstTextLine() ? mnFirst : Left();
 }
 
 inline SwTwips SwTextMargin::Left() const
 {
-    return (nDropLines >= m_nLineNr && 1 != m_nLineNr) ? nFirst + nDropLeft : nLeft;
+    return (mnDropLines >= m_nLineNr && 1 != m_nLineNr) ? mnFirst + mnDropLeft : mnLeft;
 }
 
 #endif
diff --git a/sw/source/core/text/porexp.cxx b/sw/source/core/text/porexp.cxx
index 0e79a2145e7b..7a72b94e669d 100644
--- a/sw/source/core/text/porexp.cxx
+++ b/sw/source/core/text/porexp.cxx
@@ -159,7 +159,7 @@ sal_uInt16 SwBlankPortion::MayUnderflow( const SwTextFormatInfo &rInf,
  */
 void SwBlankPortion::FormatEOL( SwTextFormatInfo &rInf )
 {
-    sal_uInt16 nMay = MayUnderflow( rInf, rInf.GetIdx() - nLineLength, true );
+    sal_uInt16 nMay = MayUnderflow( rInf, rInf.GetIdx() - mnLineLength, true );
     if( !nMay )
         return;
 
@@ -213,7 +213,7 @@ void SwBlankPortion::HandlePortion( SwPortionHandler& rPH ) const
 SwPostItsPortion::SwPostItsPortion( bool bScrpt )
     : bScript( bScrpt )
 {
-    nLineLength = TextFrameIndex(1);
+    mnLineLength = TextFrameIndex(1);
     SetWhichPor( PortionType::PostIts );
 }
 
diff --git a/sw/source/core/text/porfld.cxx b/sw/source/core/text/porfld.cxx
index 8688490ffe38..1d793dac0d2f 100644
--- a/sw/source/core/text/porfld.cxx
+++ b/sw/source/core/text/porfld.cxx
@@ -544,7 +544,7 @@ bool SwNumberPortion::Format( SwTextFormatInfo &rInf )
     if( rInf.IsNumDone() )
     {
 //        SetAscent( rInf.GetAscent() );
-        OSL_ENSURE( Height() && nAscent, "NumberPortions without Height | Ascent" );
+        OSL_ENSURE( Height() && mnAscent, "NumberPortions without Height | Ascent" );
 
         tools::Long nDiff( 0 );
 
diff --git a/sw/source/core/text/porfly.cxx b/sw/source/core/text/porfly.cxx
index 24961c08b68d..8fcbb7ff4ae9 100644
--- a/sw/source/core/text/porfly.cxx
+++ b/sw/source/core/text/porfly.cxx
@@ -251,7 +251,7 @@ SwFlyCntPortion::SwFlyCntPortion()
     : m_bMax(false)
     , m_eAlign(sw::LineAlign::NONE)
 {
-    nLineLength = TextFrameIndex(1);
+    mnLineLength = TextFrameIndex(1);
     SetWhichPor(PortionType::FlyCnt);
 }
 
@@ -385,20 +385,20 @@ void SwFlyCntPortion::SetBase( const SwTextFrame& rFrame, const Point &rBase,
         SwTwips nRelPos = aObjPositioning.GetRelPosY();
         if ( nRelPos < 0 )
         {
-            nAscent = static_cast<sal_uInt16>(-nRelPos);
-            if( nAscent > Height() )
-                Height( nAscent );
+            mnAscent = static_cast<sal_uInt16>(-nRelPos);
+            if( mnAscent > Height() )
+                Height( mnAscent );
         }
         else
         {
-            nAscent = 0;
+            mnAscent = 0;
             Height( Height() + static_cast<sal_uInt16>(nRelPos) );
         }
     }
     else
     {
         Height( 1 );
-        nAscent = 0;
+        mnAscent = 0;
     }
 }
 
diff --git a/sw/source/core/text/porlay.cxx b/sw/source/core/text/porlay.cxx
index 7787f5c87f40..225fd2081c3c 100644
--- a/sw/source/core/text/porlay.cxx
+++ b/sw/source/core/text/porlay.cxx
@@ -402,8 +402,8 @@ void SwLineLayout::CalcLine( SwTextFormatter &rLine, SwTextFormatInfo &rInf )
                     continue;
                 }
 
-                TextFrameIndex const nPorSttIdx = rInf.GetLineStart() + nLineLength;
-                nLineLength += pPos->GetLen();
+                TextFrameIndex const nPorSttIdx = rInf.GetLineStart() + mnLineLength;
+                mnLineLength += pPos->GetLen();
                 AddPrtWidth( pPos->Width() );
 
                 // #i3952#
@@ -453,7 +453,7 @@ void SwLineLayout::CalcLine( SwTextFormatter &rLine, SwTextFormatInfo &rInf )
                 {
                     if (!pPos->IsPostItsPortion()) bOnlyPostIts = false;
 
-                    if( bTmpDummy && !nLineLength )
+                    if( bTmpDummy && !mnLineLength )
                     {
                         if( pPos->IsFlyPortion() )
                         {
@@ -481,7 +481,7 @@ void SwLineLayout::CalcLine( SwTextFormatter &rLine, SwTextFormatInfo &rInf )
                                     nPosHeight = nTmp;
                             }
                             Height( nPosHeight, false );
-                            nAscent = nPosAscent;
+                            mnAscent = nPosAscent;
                             nMaxDescent = nPosHeight - nPosAscent;
                         }
                     }
@@ -511,8 +511,8 @@ void SwLineLayout::CalcLine( SwTextFormatter &rLine, SwTextFormatInfo &rInf )
                         }
                         else
                         {
-                            if( nAscent < nPosAscent )
-                                nAscent = nPosAscent;
+                            if( mnAscent < nPosAscent )
+                                mnAscent = nPosAscent;
                             if( nMaxDescent < nPosHeight - nPosAscent )
                                 nMaxDescent = nPosHeight - nPosAscent;
                         }
@@ -545,32 +545,32 @@ void SwLineLayout::CalcLine( SwTextFormatter &rLine, SwTextFormatInfo &rInf )
                 if( pFlyCnt->Height() == Height() )
                 {
                     pFlyCnt->SetMax( true );
-                    if( Height() > nMaxDescent + nAscent )
+                    if( Height() > nMaxDescent + mnAscent )
                     {
                         if( sw::LineAlign::BOTTOM == pFlyCnt->GetAlign() )
-                            nAscent = Height() - nMaxDescent;
+                            mnAscent = Height() - nMaxDescent;
                         else if( sw::LineAlign::CENTER == pFlyCnt->GetAlign() )
-                            nAscent = ( Height() + nAscent - nMaxDescent ) / 2;
+                            mnAscent = ( Height() + mnAscent - nMaxDescent ) / 2;
                     }
-                    pFlyCnt->SetAscent( nAscent );
+                    pFlyCnt->SetAscent( mnAscent );
                 }
             }
 
             if( bTmpDummy && nFlyHeight )
             {
-                nAscent = nFlyAscent;
+                mnAscent = nFlyAscent;
                 if( nFlyDescent > nFlyHeight - nFlyAscent )
                     Height( nFlyHeight + nFlyDescent, false );
                 else
                     Height( nFlyHeight, false );
             }
-            else if( nMaxDescent > Height() - nAscent )
-                Height( nMaxDescent + nAscent, false );
+            else if( nMaxDescent > Height() - mnAscent )
+                Height( nMaxDescent + mnAscent, false );
 
             if( bOnlyPostIts && !( bHasBlankPortion && bHasOnlyBlankPortions ) )
             {
                 Height( rInf.GetFont()->GetHeight( rInf.GetVsh(), *rInf.GetOut() ) );
-                nAscent = rInf.GetFont()->GetAscent( rInf.GetVsh(), *rInf.GetOut() );
+                mnAscent = rInf.GetFont()->GetAscent( rInf.GetVsh(), *rInf.GetOut() );
             }
         }
     }
@@ -2463,7 +2463,7 @@ SwTwips SwLineLayout::GetHangingMargin_() const
         }
         // the last post its portion
         else if ( pPor->IsPostItsPortion() && ! pPor->GetNextPortion() )
-            nDiff = nAscent;
+            nDiff = mnAscent;
 
         pPor = pPor->GetNextPortion();
     }
diff --git a/sw/source/core/text/porlay.hxx b/sw/source/core/text/porlay.hxx
index e3f6f882eabb..0c91b53f1465 100644
--- a/sw/source/core/text/porlay.hxx
+++ b/sw/source/core/text/porlay.hxx
@@ -63,15 +63,15 @@ public:
 // nRightOfst gives the right margin
 class SwRepaint : public SwRect
 {
-    SwTwips nOfst;
-    SwTwips nRightOfst;
+    SwTwips m_nOffset;
+    SwTwips m_nRightOffset;
 public:
-    SwRepaint() : SwRect(), nOfst( 0 ), nRightOfst( 0 ) {}
+    SwRepaint() : SwRect(), m_nOffset( 0 ), m_nRightOffset( 0 ) {}
 
-    SwTwips GetOffset() const { return nOfst; }
-    void   SetOffset( const SwTwips nNew ) { nOfst = nNew; }
-    SwTwips GetRightOfst() const { return nRightOfst; }
-    void   SetRightOfst( const SwTwips nNew ) { nRightOfst = nNew; }
+    SwTwips GetOffset() const { return m_nOffset; }
+    void   SetOffset( const SwTwips nNew ) { m_nOffset = nNew; }
+    SwTwips GetRightOfst() const { return m_nRightOffset; }
+    void   SetRightOfst( const SwTwips nNew ) { m_nRightOffset = nNew; }
 };
 
 /// Collection of SwLinePortion instances, representing one line of text.
diff --git a/sw/source/core/text/porlin.cxx b/sw/source/core/text/porlin.cxx
index b94d4b7896c9..035f670d464b 100644
--- a/sw/source/core/text/porlin.cxx
+++ b/sw/source/core/text/porlin.cxx
@@ -65,9 +65,9 @@ sal_uInt16 SwLinePortion::GetViewWidth( const SwTextSizeInfo & ) const
 
 SwLinePortion::SwLinePortion( ) :
     mpNextPortion( nullptr ),
-    nLineLength( 0 ),
-    nAscent( 0 ),
-    nWhichPor( PortionType::NONE ),
+    mnLineLength( 0 ),
+    mnAscent( 0 ),
+    mnWhichPor( PortionType::NONE ),
     m_bJoinBorderWithPrev(false),
     m_bJoinBorderWithNext(false)
 {
diff --git a/sw/source/core/text/porlin.hxx b/sw/source/core/text/porlin.hxx
index 50b19ad0782c..a71c1112a99b 100644
--- a/sw/source/core/text/porlin.hxx
+++ b/sw/source/core/text/porlin.hxx
@@ -53,12 +53,12 @@ protected:
     // Here we have areas with different attributes
     SwLinePortion *mpNextPortion;
     // Count of chars and spaces on the line
-    TextFrameIndex nLineLength;
-    sal_uInt16 nAscent;      // Maximum ascender
+    TextFrameIndex mnLineLength;
+    sal_uInt16 mnAscent;      // Maximum ascender
 
     SwLinePortion();
 private:
-    PortionType nWhichPor;       // Who's who?
+    PortionType mnWhichPor;       // Who's who?
     bool m_bJoinBorderWithPrev;
     bool m_bJoinBorderWithNext;
 
@@ -71,12 +71,12 @@ public:
     // Access methods
     SwLinePortion *GetNextPortion() const { return mpNextPortion; }
     inline SwLinePortion &operator=(const SwLinePortion &rPortion);
-    TextFrameIndex GetLen() const { return nLineLength; }
-    void SetLen(TextFrameIndex const nLen) { nLineLength = nLen; }
+    TextFrameIndex GetLen() const { return mnLineLength; }
+    void SetLen(TextFrameIndex const nLen) { mnLineLength = nLen; }
     void SetNextPortion( SwLinePortion *pNew ){ mpNextPortion = pNew; }
-    sal_uInt16 &GetAscent() { return nAscent; }
-    sal_uInt16 GetAscent() const { return nAscent; }
-    void SetAscent( const sal_uInt16 nNewAsc ) { nAscent = nNewAsc; }
+    sal_uInt16 &GetAscent() { return mnAscent; }
+    sal_uInt16 GetAscent() const { return mnAscent; }
+    void SetAscent( const sal_uInt16 nNewAsc ) { mnAscent = nNewAsc; }
     void  PrtWidth( sal_uInt16 nNewWidth ) { Width( nNewWidth ); }
     sal_uInt16 PrtWidth() const { return Width(); }
     void AddPrtWidth( const sal_uInt16 nNew ) { Width( Width() + nNew ); }
@@ -91,49 +91,49 @@ public:
     // Returns 0, if there's no payload
     virtual SwLinePortion *Compress();
 
-    void SetWhichPor( const PortionType nNew )    { nWhichPor = nNew; }
-    PortionType GetWhichPor( ) const        { return nWhichPor; }
+    void SetWhichPor( const PortionType nNew )    { mnWhichPor = nNew; }
+    PortionType GetWhichPor( ) const        { return mnWhichPor; }
 
 // Group queries
-    bool InTextGrp() const { return (sal_uInt16(nWhichPor) & PORGRP_TXT) != 0; }
-    bool InGlueGrp() const { return (sal_uInt16(nWhichPor) & PORGRP_GLUE) != 0; }
-    bool InTabGrp() const { return (sal_uInt16(nWhichPor) & PORGRP_TAB) != 0; }
-    bool InHyphGrp() const { return (sal_uInt16(nWhichPor) & PORGRP_HYPH) != 0; }
-    bool InNumberGrp() const { return (sal_uInt16(nWhichPor) & PORGRP_NUMBER) != 0; }
-    bool InFixGrp() const { return (sal_uInt16(nWhichPor) & PORGRP_FIX) != 0; }
-    bool InFieldGrp() const { return (sal_uInt16(nWhichPor) & PORGRP_FLD) != 0; }
-    bool InToxRefGrp() const { return (sal_uInt16(nWhichPor) & PORGRP_TOXREF) != 0; }
-    bool InToxRefOrFieldGrp() const { return (sal_uInt16(nWhichPor) & ( PORGRP_FLD | PORGRP_TOXREF )) != 0; }
-    bool InExpGrp() const { return (sal_uInt16(nWhichPor) & PORGRP_EXP) != 0; }
-    bool InFixMargGrp() const { return (sal_uInt16(nWhichPor) & PORGRP_FIXMARG) != 0; }
+    bool InTextGrp() const { return (sal_uInt16(mnWhichPor) & PORGRP_TXT) != 0; }
+    bool InGlueGrp() const { return (sal_uInt16(mnWhichPor) & PORGRP_GLUE) != 0; }
+    bool InTabGrp() const { return (sal_uInt16(mnWhichPor) & PORGRP_TAB) != 0; }
+    bool InHyphGrp() const { return (sal_uInt16(mnWhichPor) & PORGRP_HYPH) != 0; }
+    bool InNumberGrp() const { return (sal_uInt16(mnWhichPor) & PORGRP_NUMBER) != 0; }
+    bool InFixGrp() const { return (sal_uInt16(mnWhichPor) & PORGRP_FIX) != 0; }
+    bool InFieldGrp() const { return (sal_uInt16(mnWhichPor) & PORGRP_FLD) != 0; }
+    bool InToxRefGrp() const { return (sal_uInt16(mnWhichPor) & PORGRP_TOXREF) != 0; }
+    bool InToxRefOrFieldGrp() const { return (sal_uInt16(mnWhichPor) & ( PORGRP_FLD | PORGRP_TOXREF )) != 0; }
+    bool InExpGrp() const { return (sal_uInt16(mnWhichPor) & PORGRP_EXP) != 0; }
+    bool InFixMargGrp() const { return (sal_uInt16(mnWhichPor) & PORGRP_FIXMARG) != 0; }
     bool InSpaceGrp() const { return InTextGrp() || IsMultiPortion(); }
 // Individual queries
-    bool IsGrfNumPortion() const { return nWhichPor == PortionType::GrfNum; }
-    bool IsFlyCntPortion() const { return nWhichPor == PortionType::FlyCnt; }
-    bool IsBlankPortion() const { return nWhichPor == PortionType::Blank; }
-    bool IsBreakPortion() const { return nWhichPor == PortionType::Break; }
-    bool IsErgoSumPortion() const { return nWhichPor == PortionType::ErgoSum; }
-    bool IsQuoVadisPortion() const { return nWhichPor == PortionType::QuoVadis; }
-    bool IsTabLeftPortion() const { return nWhichPor == PortionType::TabLeft; }
-    bool IsTabRightPortion() const { return nWhichPor == PortionType::TabRight; }
-    bool IsFootnoteNumPortion() const { return nWhichPor == PortionType::FootnoteNum; }
-    bool IsFootnotePortion() const { return nWhichPor == PortionType::Footnote; }
-    bool IsDropPortion() const { return nWhichPor == PortionType::Drop; }
-    bool IsLayPortion() const { return nWhichPor == PortionType::Lay; }
-    bool IsParaPortion() const { return nWhichPor == PortionType::Para; }
-    bool IsMarginPortion() const { return nWhichPor == PortionType::Margin; }
-    bool IsFlyPortion() const { return nWhichPor == PortionType::Fly; }
-    bool IsHolePortion() const { return nWhichPor == PortionType::Hole; }
-    bool IsSoftHyphPortion() const { return nWhichPor == PortionType::SoftHyphen; }
-    bool IsPostItsPortion() const { return nWhichPor == PortionType::PostIts; }
-    bool IsCombinedPortion() const { return nWhichPor == PortionType::Combined; }
-    bool IsTextPortion() const { return nWhichPor == PortionType::Text; }
-    bool IsHangingPortion() const { return nWhichPor == PortionType::Hanging; }
-    bool IsKernPortion() const { return nWhichPor == PortionType::Kern; }
-    bool IsArrowPortion() const { return nWhichPor == PortionType::Arrow; }
-    bool IsMultiPortion() const { return nWhichPor == PortionType::Multi; }
-    bool IsNumberPortion() const { return nWhichPor == PortionType::Number; } // #i23726#
-    bool IsControlCharPortion() const { return nWhichPor == PortionType::ControlChar || nWhichPor == PortionType::Bookmark; }
+    bool IsGrfNumPortion() const { return mnWhichPor == PortionType::GrfNum; }
+    bool IsFlyCntPortion() const { return mnWhichPor == PortionType::FlyCnt; }
+    bool IsBlankPortion() const { return mnWhichPor == PortionType::Blank; }
+    bool IsBreakPortion() const { return mnWhichPor == PortionType::Break; }
+    bool IsErgoSumPortion() const { return mnWhichPor == PortionType::ErgoSum; }
+    bool IsQuoVadisPortion() const { return mnWhichPor == PortionType::QuoVadis; }
+    bool IsTabLeftPortion() const { return mnWhichPor == PortionType::TabLeft; }
+    bool IsTabRightPortion() const { return mnWhichPor == PortionType::TabRight; }
+    bool IsFootnoteNumPortion() const { return mnWhichPor == PortionType::FootnoteNum; }
+    bool IsFootnotePortion() const { return mnWhichPor == PortionType::Footnote; }
+    bool IsDropPortion() const { return mnWhichPor == PortionType::Drop; }
+    bool IsLayPortion() const { return mnWhichPor == PortionType::Lay; }
+    bool IsParaPortion() const { return mnWhichPor == PortionType::Para; }
+    bool IsMarginPortion() const { return mnWhichPor == PortionType::Margin; }
+    bool IsFlyPortion() const { return mnWhichPor == PortionType::Fly; }
+    bool IsHolePortion() const { return mnWhichPor == PortionType::Hole; }
+    bool IsSoftHyphPortion() const { return mnWhichPor == PortionType::SoftHyphen; }
+    bool IsPostItsPortion() const { return mnWhichPor == PortionType::PostIts; }
+    bool IsCombinedPortion() const { return mnWhichPor == PortionType::Combined; }
+    bool IsTextPortion() const { return mnWhichPor == PortionType::Text; }
+    bool IsHangingPortion() const { return mnWhichPor == PortionType::Hanging; }
+    bool IsKernPortion() const { return mnWhichPor == PortionType::Kern; }
+    bool IsArrowPortion() const { return mnWhichPor == PortionType::Arrow; }
+    bool IsMultiPortion() const { return mnWhichPor == PortionType::Multi; }
+    bool IsNumberPortion() const { return mnWhichPor == PortionType::Number; } // #i23726#
+    bool IsControlCharPortion() const { return mnWhichPor == PortionType::ControlChar || mnWhichPor == PortionType::Bookmark; }
 
     // Positioning
     SwLinePortion *FindPrevPortion( const SwLinePortion *pRoot );
@@ -174,9 +174,9 @@ public:
 inline SwLinePortion &SwLinePortion::operator=(const SwLinePortion &rPortion)
 {
     *static_cast<SwPosSize*>(this) = rPortion;
-    nLineLength = rPortion.nLineLength;
-    nAscent = rPortion.nAscent;
-    nWhichPor = rPortion.nWhichPor;
+    mnLineLength = rPortion.mnLineLength;
+    mnAscent = rPortion.mnAscent;
+    mnWhichPor = rPortion.mnWhichPor;
     m_bJoinBorderWithPrev = rPortion.m_bJoinBorderWithPrev;
     m_bJoinBorderWithNext = rPortion.m_bJoinBorderWithNext;
     return *this;
@@ -185,9 +185,9 @@ inline SwLinePortion &SwLinePortion::operator=(const SwLinePortion &rPortion)
 inline SwLinePortion::SwLinePortion(const SwLinePortion &rPortion) :
     SwPosSize( rPortion ),
     mpNextPortion( nullptr ),
-    nLineLength( rPortion.nLineLength ),
-    nAscent( rPortion.nAscent ),
-    nWhichPor( rPortion.nWhichPor ),
+    mnLineLength( rPortion.mnLineLength ),
+    mnAscent( rPortion.mnAscent ),
+    mnWhichPor( rPortion.mnWhichPor ),
     m_bJoinBorderWithPrev( rPortion.m_bJoinBorderWithPrev ),
     m_bJoinBorderWithNext( rPortion.m_bJoinBorderWithNext )
 {
diff --git a/sw/source/core/text/porrst.cxx b/sw/source/core/text/porrst.cxx
index 6326a2e7c9df..fe94c2e9aa0b 100644
--- a/sw/source/core/text/porrst.cxx
+++ b/sw/source/core/text/porrst.cxx
@@ -96,7 +96,7 @@ void SwTmpEndPortion::Paint( const SwTextPaintInfo &rInf ) const
 SwBreakPortion::SwBreakPortion( const SwLinePortion &rPortion )
     : SwLinePortion( rPortion )
 {
-    nLineLength = TextFrameIndex(1);
+    mnLineLength = TextFrameIndex(1);
     m_eRedline = RedlineType::None;
     SetWhichPor( PortionType::Break );
 }
@@ -173,7 +173,7 @@ SwKernPortion::SwKernPortion( SwLinePortion &rPortion, short nKrn,
 {
     Height( rPortion.Height() );
     SetAscent( rPortion.GetAscent() );
-    nLineLength = TextFrameIndex(0);
+    mnLineLength = TextFrameIndex(0);
     SetWhichPor( PortionType::Kern );
     if( nKern > 0 )
         Width( nKern );
@@ -186,7 +186,7 @@ SwKernPortion::SwKernPortion( const SwLinePortion& rPortion ) :
     Height( rPortion.Height() );
     SetAscent( rPortion.GetAscent() );
 
-    nLineLength = TextFrameIndex(0);
+    mnLineLength = TextFrameIndex(0);
     SetWhichPor( PortionType::Kern );
 }
 
@@ -236,7 +236,7 @@ SwArrowPortion::SwArrowPortion( const SwLinePortion &rPortion ) :
 {
     Height( rPortion.Height() );
     SetAscent( rPortion.GetAscent() );
-    nLineLength = TextFrameIndex(0);
+    mnLineLength = TextFrameIndex(0);
     SetWhichPor( PortionType::Arrow );
 }
 
diff --git a/sw/source/core/text/possiz.hxx b/sw/source/core/text/possiz.hxx
index 63782e4c91e8..42546305a801 100644
--- a/sw/source/core/text/possiz.hxx
+++ b/sw/source/core/text/possiz.hxx
@@ -25,17 +25,17 @@
 // Compared to the SV sizes SwPosSize is always positive
 class SwPosSize
 {
-    sal_uInt16 nWidth;
-    sal_uInt16 nHeight;
+    sal_uInt16 m_nWidth;
+    sal_uInt16 m_nHeight;
 public:
     SwPosSize( const sal_uInt16 nW = 0, const sal_uInt16 nH = 0 )
-        : nWidth(nW)
-        , nHeight(nH)
+        : m_nWidth(nW)
+        , m_nHeight(nH)
     {
     }
     explicit SwPosSize( const Size &rSize )
-        : nWidth(sal_uInt16(rSize.Width()))
-        ,nHeight(sal_uInt16(rSize.Height()))
+        : m_nWidth(sal_uInt16(rSize.Width()))
+        ,m_nHeight(sal_uInt16(rSize.Height()))
     {
     }
 #if defined(__COVERITY__)
@@ -47,25 +47,25 @@ public:
     SwPosSize(SwPosSize &&) = default;
     SwPosSize & operator =(SwPosSize const &) = default;
     SwPosSize & operator =(SwPosSize &&) = default;
-    sal_uInt16 Height() const { return nHeight; }
-    virtual void Height(const sal_uInt16 nNew, const bool = true) { nHeight = nNew; }
-    sal_uInt16 Width() const { return nWidth; }
-    void Width( const sal_uInt16 nNew ) { nWidth = nNew; }
-    Size SvLSize() const { return Size( nWidth, nHeight ); }
+    sal_uInt16 Height() const { return m_nHeight; }
+    virtual void Height(const sal_uInt16 nNew, const bool = true) { m_nHeight = nNew; }
+    sal_uInt16 Width() const { return m_nWidth; }
+    void Width( const sal_uInt16 nNew ) { m_nWidth = nNew; }
+    Size SvLSize() const { return Size( m_nWidth, m_nHeight ); }
     void SvLSize( const Size &rSize )
     {
-        nWidth  = sal_uInt16(rSize.Width());
-        nHeight = sal_uInt16(rSize.Height());
+        m_nWidth  = sal_uInt16(rSize.Width());
+        m_nHeight = sal_uInt16(rSize.Height());
     }
     void SvXSize( const Size &rSize )
     {
-        nHeight = sal_uInt16(rSize.Width());
-        nWidth = sal_uInt16(rSize.Height());
+        m_nHeight = sal_uInt16(rSize.Width());
+        m_nWidth = sal_uInt16(rSize.Height());
     }
     SwPosSize& operator=( const Size &rSize )
     {
-        nWidth  = sal_uInt16(rSize.Width());
-        nHeight = sal_uInt16(rSize.Height());
+        m_nWidth  = sal_uInt16(rSize.Width());
+        m_nHeight = sal_uInt16(rSize.Height());
         return *this;
     }
 };
diff --git a/sw/source/core/text/txttab.cxx b/sw/source/core/text/txttab.cxx
index 3fde39337859..51cf0567ad81 100644
--- a/sw/source/core/text/txttab.cxx
+++ b/sw/source/core/text/txttab.cxx
@@ -292,7 +292,7 @@ SwTabPortion *SwTextFormatter::NewTabPortion( SwTextFormatInfo &rInf, bool bAuto
 SwTabPortion::SwTabPortion( const sal_uInt16 nTabPosition, const sal_Unicode cFillChar, const bool bAutoTab )
     : SwFixPortion(), nTabPos(nTabPosition), cFill(cFillChar), bAutoTabStop( bAutoTab )
 {
-    nLineLength = TextFrameIndex(1);
+    mnLineLength = TextFrameIndex(1);
     OSL_ENSURE(!IsFilled() || ' ' != cFill, "SwTabPortion::CTOR: blanks ?!");
     SetWhichPor( PortionType::Table );
 }


More information about the Libreoffice-commits mailing list