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

Miklos Vajna vmiklos at collabora.co.uk
Sat Nov 14 00:08:31 PST 2015


 sw/inc/printdata.hxx               |  198 ++++++++++++-------------
 sw/inc/prtopt.hxx                  |    2 
 sw/source/core/inc/UndoDelete.hxx  |   42 ++---
 sw/source/core/text/porlay.cxx     |   10 -
 sw/source/core/text/porlay.hxx     |  132 ++++++++---------
 sw/source/core/undo/undel.cxx      |  284 ++++++++++++++++++-------------------
 sw/source/core/view/printdata.cxx  |   32 ++--
 sw/source/core/view/vprint.cxx     |   16 +-
 sw/source/ui/config/optpage.cxx    |   86 +++++------
 sw/source/ui/dialog/uiregionsw.cxx |  204 +++++++++++++-------------
 sw/source/uibase/config/prtopt.cxx |   84 +++++-----
 sw/source/uibase/inc/cfgitems.hxx  |    2 
 sw/source/uibase/inc/regionsw.hxx  |   50 +++---
 sw/source/uibase/uno/unotxdoc.cxx  |    4 
 14 files changed, 573 insertions(+), 573 deletions(-)

New commits:
commit f23d092cc08176f470c4d3d9cd3eb1bcf6f2a487
Author: Miklos Vajna <vmiklos at collabora.co.uk>
Date:   Sat Nov 14 09:07:13 2015 +0100

    sw: prefix members of SwLineLayout
    
    Change-Id: If6ced6d33dbb9c82c75aa05a9f7b018926c0ffb0

diff --git a/sw/source/core/text/porlay.cxx b/sw/source/core/text/porlay.cxx
index ae62a55..b1ca592 100644
--- a/sw/source/core/text/porlay.cxx
+++ b/sw/source/core/text/porlay.cxx
@@ -124,11 +124,11 @@ static  bool lcl_HasStrongLTR ( const OUString& rText, sal_Int32 nStart, sal_Int
 SwLineLayout::~SwLineLayout()
 {
     Truncate();
-    delete pNext;
+    delete m_pNext;
     if( pBlink )
         pBlink->Delete( this );
-    delete pLLSpaceAdd;
-    delete pKanaComp;
+    delete m_pLLSpaceAdd;
+    delete m_pKanaComp;
 }
 
 SwLinePortion *SwLineLayout::Insert( SwLinePortion *pIns )
@@ -220,7 +220,7 @@ SwMarginPortion *SwLineLayout::CalcLeftMargin()
 
 void SwLineLayout::InitSpaceAdd()
 {
-    if ( !pLLSpaceAdd )
+    if ( !m_pLLSpaceAdd )
         CreateSpaceAdd();
     else
         SetLLSpaceAdd( 0, 0 );
@@ -228,7 +228,7 @@ void SwLineLayout::InitSpaceAdd()
 
 void SwLineLayout::CreateSpaceAdd( const long nInit )
 {
-    pLLSpaceAdd = new std::vector<long>;
+    m_pLLSpaceAdd = new std::vector<long>;
     SetLLSpaceAdd( nInit, 0 );
 }
 
diff --git a/sw/source/core/text/porlay.hxx b/sw/source/core/text/porlay.hxx
index 2e090cc..868087d 100644
--- a/sw/source/core/text/porlay.hxx
+++ b/sw/source/core/text/porlay.hxx
@@ -77,25 +77,25 @@ public:
 class SwLineLayout : public SwTextPortion
 {
 private:
-    SwLineLayout *pNext;                // The next Line
-    std::vector<long>* pLLSpaceAdd;     // Used for justified alignment
-    std::deque<sal_uInt16>* pKanaComp;  // Used for Kana compression
-    sal_uInt16 nRealHeight;                 // The height resulting from line spacing and register
-    bool bFormatAdj : 1;
-    bool bDummy     : 1;
-    bool bFntChg    : 1;
-    bool bEndHyph   : 1;
-    bool bMidHyph   : 1;
-    bool bTab       : 1;
-    bool bFly       : 1;
-    bool bRest      : 1;
-    bool bBlinking  : 1;
-    bool bClipping  : 1; // Clipping needed for exact line height
-    bool bContent   : 1; // Text for line numbering
-    bool bRedline   : 1; // The Redlining
-    bool bForcedLeftMargin : 1; // Left adjustment moved by the Fly
-    bool bHanging : 1; // Contains a hanging portion in the margin
-    bool bUnderscore : 1;
+    SwLineLayout *m_pNext;                // The next Line
+    std::vector<long>* m_pLLSpaceAdd;     // Used for justified alignment
+    std::deque<sal_uInt16>* m_pKanaComp;  // Used for Kana compression
+    sal_uInt16 m_nRealHeight;                 // The height resulting from line spacing and register
+    bool m_bFormatAdj : 1;
+    bool m_bDummy     : 1;
+    bool m_bFntChg    : 1;
+    bool m_bEndHyph   : 1;
+    bool m_bMidHyph   : 1;
+    bool m_bTab       : 1;
+    bool m_bFly       : 1;
+    bool m_bRest      : 1;
+    bool m_bBlinking  : 1;
+    bool m_bClipping  : 1; // Clipping needed for exact line height
+    bool m_bContent   : 1; // Text for line numbering
+    bool m_bRedline   : 1; // The Redlining
+    bool m_bForcedLeftMargin : 1; // Left adjustment moved by the Fly
+    bool m_bHanging : 1; // Contains a hanging portion in the margin
+    bool m_bUnderscore : 1;
 
     SwTwips _GetHangingMargin() const;
 
@@ -107,49 +107,49 @@ public:
 
     // Flags
     inline void ResetFlags();
-    inline void SetFormatAdj( const bool bNew ) { bFormatAdj = bNew; }
-    inline bool IsFormatAdj() const { return bFormatAdj; }
-    inline void SetEndHyph( const bool bNew ) { bEndHyph = bNew; }
-    inline bool IsEndHyph() const { return bEndHyph; }
-    inline void SetMidHyph( const bool bNew ) { bMidHyph = bNew; }
-    inline bool IsMidHyph() const { return bMidHyph; }
-    inline void SetFly( const bool bNew ) { bFly = bNew; }
-    inline bool IsFly() const { return bFly; }
-    inline void SetRest( const bool bNew ) { bRest = bNew; }
-    inline bool IsRest() const { return bRest; }
-    inline void SetBlinking( const bool bNew = true ) { bBlinking = bNew; }
-    inline bool IsBlinking() const { return bBlinking; }
-    inline void SetContent( const bool bNew = true ) { bContent = bNew; }
-    inline bool HasContent() const { return bContent; }
-    inline void SetRedline( const bool bNew = true ) { bRedline = bNew; }
-    inline bool HasRedline() const { return bRedline; }
-    inline void SetForcedLeftMargin( const bool bNew = true ) { bForcedLeftMargin = bNew; }
-    inline bool HasForcedLeftMargin() const { return bForcedLeftMargin; }
-    inline void SetHanging( const bool bNew = true ) { bHanging = bNew; }
-    inline void SetUnderscore( const bool bNew = true ) { bUnderscore = bNew; }
-    inline bool HasUnderscore() const { return bUnderscore; }
+    inline void SetFormatAdj( const bool bNew ) { m_bFormatAdj = bNew; }
+    inline bool IsFormatAdj() const { return m_bFormatAdj; }
+    inline void SetEndHyph( const bool bNew ) { m_bEndHyph = bNew; }
+    inline bool IsEndHyph() const { return m_bEndHyph; }
+    inline void SetMidHyph( const bool bNew ) { m_bMidHyph = bNew; }
+    inline bool IsMidHyph() const { return m_bMidHyph; }
+    inline void SetFly( const bool bNew ) { m_bFly = bNew; }
+    inline bool IsFly() const { return m_bFly; }
+    inline void SetRest( const bool bNew ) { m_bRest = bNew; }
+    inline bool IsRest() const { return m_bRest; }
+    inline void SetBlinking( const bool bNew = true ) { m_bBlinking = bNew; }
+    inline bool IsBlinking() const { return m_bBlinking; }
+    inline void SetContent( const bool bNew = true ) { m_bContent = bNew; }
+    inline bool HasContent() const { return m_bContent; }
+    inline void SetRedline( const bool bNew = true ) { m_bRedline = bNew; }
+    inline bool HasRedline() const { return m_bRedline; }
+    inline void SetForcedLeftMargin( const bool bNew = true ) { m_bForcedLeftMargin = bNew; }
+    inline bool HasForcedLeftMargin() const { return m_bForcedLeftMargin; }
+    inline void SetHanging( const bool bNew = true ) { m_bHanging = bNew; }
+    inline void SetUnderscore( const bool bNew = true ) { m_bUnderscore = bNew; }
+    inline bool HasUnderscore() const { return m_bUnderscore; }
 
     // Respecting empty dummy lines
-    inline void SetDummy( const bool bNew ) { bDummy = bNew; }
-    inline bool IsDummy() const { return bDummy; }
+    inline void SetDummy( const bool bNew ) { m_bDummy = bNew; }
+    inline bool IsDummy() const { return m_bDummy; }
 
-    inline void SetClipping( const bool bNew ) { bClipping = bNew; }
-    inline bool IsClipping() const { return bClipping; }
+    inline void SetClipping( const bool bNew ) { m_bClipping = bNew; }
+    inline bool IsClipping() const { return m_bClipping; }
 
     inline SwLineLayout();
     virtual ~SwLineLayout();
 
-    inline SwLineLayout *GetNext() { return pNext; }
-    inline const SwLineLayout *GetNext() const { return pNext; }
-    inline void SetNext( SwLineLayout *pNew ) { pNext = pNew; }
+    inline SwLineLayout *GetNext() { return m_pNext; }
+    inline const SwLineLayout *GetNext() const { return m_pNext; }
+    inline void SetNext( SwLineLayout *pNew ) { m_pNext = pNew; }
 
     void Init( SwLinePortion *pNextPortion = nullptr);
 
     // Collects the data for the line
     void CalcLine( SwTextFormatter &rLine, SwTextFormatInfo &rInf );
 
-    inline void SetRealHeight( sal_uInt16 nNew ) { nRealHeight = nNew; }
-    inline sal_uInt16 GetRealHeight() const { return nRealHeight; }
+    inline void SetRealHeight( sal_uInt16 nNew ) { m_nRealHeight = nNew; }
+    inline sal_uInt16 GetRealHeight() const { return m_nRealHeight; }
 
     // Creates the glue chain for short lines
     SwMarginPortion *CalcLeftMargin();
@@ -161,27 +161,27 @@ public:
     virtual bool Format( SwTextFormatInfo &rInf ) override;
 
     // Stuff for justified alignment
-    inline bool IsSpaceAdd() { return pLLSpaceAdd != nullptr; }
+    inline bool IsSpaceAdd() { return m_pLLSpaceAdd != nullptr; }
     void InitSpaceAdd();     // Creates pLLSpaceAdd if necessary
     void CreateSpaceAdd( const long nInit = 0 );
-    inline void FinishSpaceAdd() { delete pLLSpaceAdd; pLLSpaceAdd = nullptr; }
-    inline sal_uInt16 GetLLSpaceAddCount() const { return sal::static_int_cast< sal_uInt16 >(pLLSpaceAdd->size()); }
+    inline void FinishSpaceAdd() { delete m_pLLSpaceAdd; m_pLLSpaceAdd = nullptr; }
+    inline sal_uInt16 GetLLSpaceAddCount() const { return sal::static_int_cast< sal_uInt16 >(m_pLLSpaceAdd->size()); }
     inline void SetLLSpaceAdd( long nNew, sal_uInt16 nIdx )
     {
         if ( nIdx == GetLLSpaceAddCount() )
-            pLLSpaceAdd->push_back( nNew );
+            m_pLLSpaceAdd->push_back( nNew );
         else
-            (*pLLSpaceAdd)[ nIdx ] = nNew;
+            (*m_pLLSpaceAdd)[ nIdx ] = nNew;
     }
-    inline long GetLLSpaceAdd( sal_uInt16 nIdx ) { return (*pLLSpaceAdd)[ nIdx ]; }
-    inline void RemoveFirstLLSpaceAdd() { pLLSpaceAdd->erase( pLLSpaceAdd->begin() ); }
-    inline std::vector<long>* GetpLLSpaceAdd() const { return pLLSpaceAdd; }
+    inline long GetLLSpaceAdd( sal_uInt16 nIdx ) { return (*m_pLLSpaceAdd)[ nIdx ]; }
+    inline void RemoveFirstLLSpaceAdd() { m_pLLSpaceAdd->erase( m_pLLSpaceAdd->begin() ); }
+    inline std::vector<long>* GetpLLSpaceAdd() const { return m_pLLSpaceAdd; }
 
     // Stuff for Kana compression
-    inline void SetKanaComp( std::deque<sal_uInt16>* pNew ){ pKanaComp = pNew; }
-    inline void FinishKanaComp() { delete pKanaComp; pKanaComp = nullptr; }
-    inline std::deque<sal_uInt16>* GetpKanaComp() const { return pKanaComp; }
-    inline std::deque<sal_uInt16>& GetKanaComp() { return *pKanaComp; }
+    inline void SetKanaComp( std::deque<sal_uInt16>* pNew ){ m_pKanaComp = pNew; }
+    inline void FinishKanaComp() { delete m_pKanaComp; m_pKanaComp = nullptr; }
+    inline std::deque<sal_uInt16>* GetpKanaComp() const { return m_pKanaComp; }
+    inline std::deque<sal_uInt16>& GetKanaComp() { return *m_pKanaComp; }
 
     /** determine ascent and descent for positioning of as-character anchored
         object
@@ -318,14 +318,14 @@ public:
 
 inline void SwLineLayout::ResetFlags()
 {
-    bFormatAdj = bDummy = bFntChg = bTab = bEndHyph = bMidHyph = bFly
-    = bRest = bBlinking = bClipping = bContent = bRedline
-    = bForcedLeftMargin = bHanging = false;
+    m_bFormatAdj = m_bDummy = m_bFntChg = m_bTab = m_bEndHyph = m_bMidHyph = m_bFly
+    = m_bRest = m_bBlinking = m_bClipping = m_bContent = m_bRedline
+    = m_bForcedLeftMargin = m_bHanging = false;
 }
 
 inline SwLineLayout::SwLineLayout()
-    : pNext( nullptr ), pLLSpaceAdd( nullptr ), pKanaComp( nullptr ), nRealHeight( 0 ),
-      bUnderscore( false )
+    : m_pNext( nullptr ), m_pLLSpaceAdd( nullptr ), m_pKanaComp( nullptr ), m_nRealHeight( 0 ),
+      m_bUnderscore( false )
 {
     ResetFlags();
     SetWhichPor( POR_LAY );
commit 97355c222b51bc69adccf149f95744e5d33de788
Author: Miklos Vajna <vmiklos at collabora.co.uk>
Date:   Sat Nov 14 09:06:13 2015 +0100

    sw: prefix members of SwPrintData
    
    Change-Id: I3e6e18a0bc224ba6d201b476d77e82571ef2ee69

diff --git a/sw/inc/printdata.hxx b/sw/inc/printdata.hxx
index adac3de..5271089 100644
--- a/sw/inc/printdata.hxx
+++ b/sw/inc/printdata.hxx
@@ -61,49 +61,49 @@ class SwPrintData
 
 public:
 
-    bool bPrintGraphic, bPrintTable, bPrintDraw, bPrintControl, bPrintPageBackground,
-             bPrintBlackFont,
+    bool m_bPrintGraphic, m_bPrintTable, m_bPrintDraw, m_bPrintControl, m_bPrintPageBackground,
+             m_bPrintBlackFont,
              //#i81434# - printing of hidden text
-             bPrintHiddenText, bPrintTextPlaceholder,
-             bPrintLeftPages, bPrintRightPages, bPrintReverse, bPrintProspect,
-             bPrintProspectRTL,
-             bPrintSingleJobs, bPaperFromSetup,
+             m_bPrintHiddenText, m_bPrintTextPlaceholder,
+             m_bPrintLeftPages, m_bPrintRightPages, m_bPrintReverse, m_bPrintProspect,
+             m_bPrintProspectRTL,
+             m_bPrintSingleJobs, m_bPaperFromSetup,
              /// Print empty pages
-             bPrintEmptyPages,
+             m_bPrintEmptyPages,
 
              /// #i56195# no field update while printing mail merge documents
-             bUpdateFieldsInPrinting,
-             bModified;
+             m_bUpdateFieldsInPrinting,
+             m_bModified;
 
-    SwPostItMode    nPrintPostIts;
-    OUString       sFaxName;
+    SwPostItMode    m_nPrintPostIts;
+    OUString       m_sFaxName;
 
     SwPrintData()
     {
         m_pPrintUIOptions       = nullptr;
         m_pRenderData        = nullptr;
 
-        bPrintGraphic           =
-        bPrintTable             =
-        bPrintDraw              =
-        bPrintControl           =
-        bPrintLeftPages         =
-        bPrintRightPages        =
-        bPrintPageBackground    =
-        bPrintEmptyPages        =
-        bUpdateFieldsInPrinting = true;
-
-        bPaperFromSetup         =
-        bPrintReverse           =
-        bPrintProspect          =
-        bPrintProspectRTL       =
-        bPrintSingleJobs        =
-        bModified               =
-        bPrintBlackFont         =
-        bPrintHiddenText        =
-        bPrintTextPlaceholder   = false;
-
-        nPrintPostIts           = SwPostItMode::NONE;
+        m_bPrintGraphic           =
+        m_bPrintTable             =
+        m_bPrintDraw              =
+        m_bPrintControl           =
+        m_bPrintLeftPages         =
+        m_bPrintRightPages        =
+        m_bPrintPageBackground    =
+        m_bPrintEmptyPages        =
+        m_bUpdateFieldsInPrinting = true;
+
+        m_bPaperFromSetup         =
+        m_bPrintReverse           =
+        m_bPrintProspect          =
+        m_bPrintProspectRTL       =
+        m_bPrintSingleJobs        =
+        m_bModified               =
+        m_bPrintBlackFont         =
+        m_bPrintHiddenText        =
+        m_bPrintTextPlaceholder   = false;
+
+        m_nPrintPostIts           = SwPostItMode::NONE;
     }
 
     virtual ~SwPrintData() {}
@@ -111,25 +111,25 @@ public:
     bool operator==(const SwPrintData& rData)const
     {
         return
-        bPrintGraphic       ==   rData.bPrintGraphic        &&
-        bPrintTable         ==   rData.bPrintTable          &&
-        bPrintDraw          ==   rData.bPrintDraw           &&
-        bPrintControl       ==   rData.bPrintControl        &&
-        bPrintPageBackground==   rData.bPrintPageBackground &&
-        bPrintBlackFont     ==   rData.bPrintBlackFont      &&
-        bPrintLeftPages     ==   rData.bPrintLeftPages      &&
-        bPrintRightPages    ==   rData.bPrintRightPages     &&
-        bPrintReverse       ==   rData.bPrintReverse        &&
-        bPrintProspect      ==   rData.bPrintProspect       &&
-        bPrintProspectRTL   ==   rData.bPrintProspectRTL    &&
-        bPrintSingleJobs    ==   rData.bPrintSingleJobs     &&
-        bPaperFromSetup     ==   rData.bPaperFromSetup      &&
-        bPrintEmptyPages    ==   rData.bPrintEmptyPages     &&
-        bUpdateFieldsInPrinting == rData.bUpdateFieldsInPrinting &&
-        nPrintPostIts       ==   rData.nPrintPostIts        &&
-        sFaxName            ==   rData.sFaxName             &&
-        bPrintHiddenText    ==   rData.bPrintHiddenText     &&
-        bPrintTextPlaceholder   ==   rData.bPrintTextPlaceholder;
+        m_bPrintGraphic       ==   rData.m_bPrintGraphic        &&
+        m_bPrintTable         ==   rData.m_bPrintTable          &&
+        m_bPrintDraw          ==   rData.m_bPrintDraw           &&
+        m_bPrintControl       ==   rData.m_bPrintControl        &&
+        m_bPrintPageBackground==   rData.m_bPrintPageBackground &&
+        m_bPrintBlackFont     ==   rData.m_bPrintBlackFont      &&
+        m_bPrintLeftPages     ==   rData.m_bPrintLeftPages      &&
+        m_bPrintRightPages    ==   rData.m_bPrintRightPages     &&
+        m_bPrintReverse       ==   rData.m_bPrintReverse        &&
+        m_bPrintProspect      ==   rData.m_bPrintProspect       &&
+        m_bPrintProspectRTL   ==   rData.m_bPrintProspectRTL    &&
+        m_bPrintSingleJobs    ==   rData.m_bPrintSingleJobs     &&
+        m_bPaperFromSetup     ==   rData.m_bPaperFromSetup      &&
+        m_bPrintEmptyPages    ==   rData.m_bPrintEmptyPages     &&
+        m_bUpdateFieldsInPrinting == rData.m_bUpdateFieldsInPrinting &&
+        m_nPrintPostIts       ==   rData.m_nPrintPostIts        &&
+        m_sFaxName            ==   rData.m_sFaxName             &&
+        m_bPrintHiddenText    ==   rData.m_bPrintHiddenText     &&
+        m_bPrintTextPlaceholder   ==   rData.m_bPrintTextPlaceholder;
     }
 
     /** Note: in the context where this class is used the pointers should always be valid
@@ -138,45 +138,45 @@ public:
     void  SetPrintUIOptions( const SwPrintUIOptions *pOpt )     { m_pPrintUIOptions = pOpt; }
     void  SetRenderData( const SwRenderData *pData )            { m_pRenderData = pData; }
 
-    bool IsPrintGraphic() const             { return bPrintGraphic; }
-    bool IsPrintTable() const               { return bPrintTable; }
-    bool IsPrintDraw() const                { return bPrintDraw; }
-    bool IsPrintControl() const             { return bPrintControl; }
-    bool IsPrintLeftPage() const            { return bPrintLeftPages; }
-    bool IsPrintRightPage() const           { return bPrintRightPages; }
-    bool IsPrintReverse() const             { return bPrintReverse; }
-    bool IsPaperFromSetup() const           { return bPaperFromSetup; }
-    bool IsPrintEmptyPages() const          { return bPrintEmptyPages; }
-    bool IsPrintProspect() const            { return bPrintProspect; }
-    bool IsPrintProspectRTL() const         { return bPrintProspectRTL; }
-    bool IsPrintPageBackground() const      { return bPrintPageBackground; }
-    bool IsPrintBlackFont() const           { return bPrintBlackFont; }
-    bool IsPrintSingleJobs() const          { return bPrintSingleJobs; }
-    SwPostItMode GetPrintPostIts() const           { return nPrintPostIts; }
-    const OUString GetFaxName() const      { return sFaxName; }
-    bool IsPrintHiddenText() const          { return bPrintHiddenText; }
-    bool IsPrintTextPlaceholder() const     { return bPrintTextPlaceholder; }
-
-    void SetPrintGraphic( bool b )              { doSetModified(); bPrintGraphic = b; }
-    void SetPrintTable( bool b )                { doSetModified(); bPrintTable = b; }
-    void SetPrintDraw( bool b )                 { doSetModified(); bPrintDraw = b; }
-    void SetPrintControl( bool b )              { doSetModified(); bPrintControl = b; }
-    void SetPrintLeftPage( bool b )             { doSetModified(); bPrintLeftPages = b; }
-    void SetPrintRightPage( bool b )            { doSetModified(); bPrintRightPages = b; }
-    void SetPrintReverse( bool b )              { doSetModified(); bPrintReverse = b; }
-    void SetPaperFromSetup( bool b )            { doSetModified(); bPaperFromSetup = b; }
-    void SetPrintEmptyPages( bool b )           { doSetModified(); bPrintEmptyPages = b; }
-    void SetPrintPostIts( SwPostItMode n )      { doSetModified(); nPrintPostIts = n; }
-    void SetPrintProspect( bool b )             { doSetModified(); bPrintProspect = b; }
-    void SetPrintProspect_RTL( bool b )         { doSetModified(); bPrintProspectRTL = b; }
-    void SetPrintPageBackground( bool b )       { doSetModified(); bPrintPageBackground = b; }
-    void SetPrintBlackFont( bool b )            { doSetModified(); bPrintBlackFont = b; }
-    void SetPrintSingleJobs( bool b )           { doSetModified(); bPrintSingleJobs = b; }
-    void SetFaxName( const OUString& rSet )     { sFaxName = rSet; }
-    void SetPrintHiddenText( bool b )           { doSetModified(); bPrintHiddenText = b; }
-    void SetPrintTextPlaceholder( bool b )      { doSetModified(); bPrintTextPlaceholder = b; }
-
-    virtual void doSetModified () { bModified = true;}
+    bool IsPrintGraphic() const             { return m_bPrintGraphic; }
+    bool IsPrintTable() const               { return m_bPrintTable; }
+    bool IsPrintDraw() const                { return m_bPrintDraw; }
+    bool IsPrintControl() const             { return m_bPrintControl; }
+    bool IsPrintLeftPage() const            { return m_bPrintLeftPages; }
+    bool IsPrintRightPage() const           { return m_bPrintRightPages; }
+    bool IsPrintReverse() const             { return m_bPrintReverse; }
+    bool IsPaperFromSetup() const           { return m_bPaperFromSetup; }
+    bool IsPrintEmptyPages() const          { return m_bPrintEmptyPages; }
+    bool IsPrintProspect() const            { return m_bPrintProspect; }
+    bool IsPrintProspectRTL() const         { return m_bPrintProspectRTL; }
+    bool IsPrintPageBackground() const      { return m_bPrintPageBackground; }
+    bool IsPrintBlackFont() const           { return m_bPrintBlackFont; }
+    bool IsPrintSingleJobs() const          { return m_bPrintSingleJobs; }
+    SwPostItMode GetPrintPostIts() const           { return m_nPrintPostIts; }
+    const OUString GetFaxName() const      { return m_sFaxName; }
+    bool IsPrintHiddenText() const          { return m_bPrintHiddenText; }
+    bool IsPrintTextPlaceholder() const     { return m_bPrintTextPlaceholder; }
+
+    void SetPrintGraphic( bool b )              { doSetModified(); m_bPrintGraphic = b; }
+    void SetPrintTable( bool b )                { doSetModified(); m_bPrintTable = b; }
+    void SetPrintDraw( bool b )                 { doSetModified(); m_bPrintDraw = b; }
+    void SetPrintControl( bool b )              { doSetModified(); m_bPrintControl = b; }
+    void SetPrintLeftPage( bool b )             { doSetModified(); m_bPrintLeftPages = b; }
+    void SetPrintRightPage( bool b )            { doSetModified(); m_bPrintRightPages = b; }
+    void SetPrintReverse( bool b )              { doSetModified(); m_bPrintReverse = b; }
+    void SetPaperFromSetup( bool b )            { doSetModified(); m_bPaperFromSetup = b; }
+    void SetPrintEmptyPages( bool b )           { doSetModified(); m_bPrintEmptyPages = b; }
+    void SetPrintPostIts( SwPostItMode n )      { doSetModified(); m_nPrintPostIts = n; }
+    void SetPrintProspect( bool b )             { doSetModified(); m_bPrintProspect = b; }
+    void SetPrintProspect_RTL( bool b )         { doSetModified(); m_bPrintProspectRTL = b; }
+    void SetPrintPageBackground( bool b )       { doSetModified(); m_bPrintPageBackground = b; }
+    void SetPrintBlackFont( bool b )            { doSetModified(); m_bPrintBlackFont = b; }
+    void SetPrintSingleJobs( bool b )           { doSetModified(); m_bPrintSingleJobs = b; }
+    void SetFaxName( const OUString& rSet )     { m_sFaxName = rSet; }
+    void SetPrintHiddenText( bool b )           { doSetModified(); m_bPrintHiddenText = b; }
+    void SetPrintTextPlaceholder( bool b )      { doSetModified(); m_bPrintTextPlaceholder = b; }
+
+    virtual void doSetModified () { m_bModified = true;}
 };
 
 class SwPrintUIOptions : public vcl::PrinterOptionsHelper
@@ -190,15 +190,15 @@ public:
 
     bool processPropertiesAndCheckFormat( const css::uno::Sequence< css::beans::PropertyValue >& i_rNewProp );
 
-    bool IsPrintFormControls() const            { return getBoolValue( "PrintControls",        m_rDefaultPrintData.bPrintControl ); }
-    bool IsPrintPageBackground() const          { return getBoolValue( "PrintPageBackground",  m_rDefaultPrintData.bPrintPageBackground ); }
-    bool IsPrintProspect() const                { return getBoolValue( "PrintProspect",        m_rDefaultPrintData.bPrintProspect ); }
-    bool IsPrintProspectRTL() const             { return getIntValue( "PrintProspectRTL",      m_rDefaultPrintData.bPrintProspectRTL ? 1 : 0 ) != 0; }
-    bool IsPrintTextPlaceholders() const        { return getBoolValue( "PrintTextPlaceholder", m_rDefaultPrintData.bPrintTextPlaceholder ); }
-    bool IsPrintHiddenText() const              { return getBoolValue( "PrintHiddenText",      m_rDefaultPrintData.bPrintHiddenText ); }
-    bool IsPrintWithBlackTextColor() const      { return getBoolValue( "PrintBlackFonts",      m_rDefaultPrintData.bPrintBlackFont ); }
-    SwPostItMode GetPrintPostItsType() const       { return static_cast< SwPostItMode >(getIntValue( "PrintAnnotationMode", static_cast<sal_uInt16>(m_rDefaultPrintData.nPrintPostIts) )); }
-    bool IsPaperFromSetup() const               { return getBoolValue( "PrintPaperFromSetup",  m_rDefaultPrintData.bPaperFromSetup ); }
+    bool IsPrintFormControls() const            { return getBoolValue( "PrintControls",        m_rDefaultPrintData.m_bPrintControl ); }
+    bool IsPrintPageBackground() const          { return getBoolValue( "PrintPageBackground",  m_rDefaultPrintData.m_bPrintPageBackground ); }
+    bool IsPrintProspect() const                { return getBoolValue( "PrintProspect",        m_rDefaultPrintData.m_bPrintProspect ); }
+    bool IsPrintProspectRTL() const             { return getIntValue( "PrintProspectRTL",      m_rDefaultPrintData.m_bPrintProspectRTL ? 1 : 0 ) != 0; }
+    bool IsPrintTextPlaceholders() const        { return getBoolValue( "PrintTextPlaceholder", m_rDefaultPrintData.m_bPrintTextPlaceholder ); }
+    bool IsPrintHiddenText() const              { return getBoolValue( "PrintHiddenText",      m_rDefaultPrintData.m_bPrintHiddenText ); }
+    bool IsPrintWithBlackTextColor() const      { return getBoolValue( "PrintBlackFonts",      m_rDefaultPrintData.m_bPrintBlackFont ); }
+    SwPostItMode GetPrintPostItsType() const       { return static_cast< SwPostItMode >(getIntValue( "PrintAnnotationMode", static_cast<sal_uInt16>(m_rDefaultPrintData.m_nPrintPostIts) )); }
+    bool IsPaperFromSetup() const               { return getBoolValue( "PrintPaperFromSetup",  m_rDefaultPrintData.m_bPaperFromSetup ); }
 
     bool IsPrintLeftPages() const;
     bool IsPrintRightPages() const;
diff --git a/sw/inc/prtopt.hxx b/sw/inc/prtopt.hxx
index 52c83b3..0e4004b 100644
--- a/sw/inc/prtopt.hxx
+++ b/sw/inc/prtopt.hxx
@@ -36,7 +36,7 @@ public:
     virtual ~SwPrintOptions();
 
     virtual void Notify( const css::uno::Sequence< OUString >& aPropertyNames ) override;
-    virtual void            doSetModified( ) override { bModified = true; SetModified();}
+    virtual void            doSetModified( ) override { m_bModified = true; SetModified();}
 
     SwPrintOptions& operator=(const SwPrintData& rData)
     {
diff --git a/sw/source/core/view/printdata.cxx b/sw/source/core/view/printdata.cxx
index cabd06f..9d0ab63 100644
--- a/sw/source/core/view/printdata.cxx
+++ b/sw/source/core/view/printdata.cxx
@@ -124,25 +124,25 @@ void SwRenderData::MakeSwPrtOptions(
     ::sw::InitPrintOptionsFromApplication(rOptions, bWeb);
 
     // get print options to use from provided properties
-    rOptions.bPrintGraphic          = pOpt->IsPrintGraphics();
-    rOptions.bPrintTable            = true; // for now it was decided that tables should always be printed
-    rOptions.bPrintDraw             = pOpt->IsPrintDrawings();
-    rOptions.bPrintControl          = pOpt->IsPrintFormControls();
-    rOptions.bPrintLeftPages        = pOpt->IsPrintLeftPages();
-    rOptions.bPrintRightPages       = pOpt->IsPrintRightPages();
-    rOptions.bPrintPageBackground   = pOpt->IsPrintPageBackground();
-    rOptions.bPrintEmptyPages       = pOpt->IsPrintEmptyPages( bIsPDFExport );
+    rOptions.m_bPrintGraphic          = pOpt->IsPrintGraphics();
+    rOptions.m_bPrintTable            = true; // for now it was decided that tables should always be printed
+    rOptions.m_bPrintDraw             = pOpt->IsPrintDrawings();
+    rOptions.m_bPrintControl          = pOpt->IsPrintFormControls();
+    rOptions.m_bPrintLeftPages        = pOpt->IsPrintLeftPages();
+    rOptions.m_bPrintRightPages       = pOpt->IsPrintRightPages();
+    rOptions.m_bPrintPageBackground   = pOpt->IsPrintPageBackground();
+    rOptions.m_bPrintEmptyPages       = pOpt->IsPrintEmptyPages( bIsPDFExport );
     // bUpdateFieldsInPrinting  <-- not set here; mail merge only
-    rOptions.bPaperFromSetup        = pOpt->IsPaperFromSetup();
-    rOptions.bPrintReverse          = false; /*handled by print dialog now*/
-    rOptions.bPrintProspect         = pOpt->IsPrintProspect();
-    rOptions.bPrintProspectRTL      = pOpt->IsPrintProspectRTL();
+    rOptions.m_bPaperFromSetup        = pOpt->IsPaperFromSetup();
+    rOptions.m_bPrintReverse          = false; /*handled by print dialog now*/
+    rOptions.m_bPrintProspect         = pOpt->IsPrintProspect();
+    rOptions.m_bPrintProspectRTL      = pOpt->IsPrintProspectRTL();
     // bPrintSingleJobs         <-- not set here; mail merge and or configuration
     // bModified                <-- not set here; mail merge only
-    rOptions.bPrintBlackFont        = pOpt->IsPrintWithBlackTextColor();
-    rOptions.bPrintHiddenText       = pOpt->IsPrintHiddenText();
-    rOptions.bPrintTextPlaceholder  = pOpt->IsPrintTextPlaceholders();
-    rOptions.nPrintPostIts          = pOpt->GetPrintPostItsType();
+    rOptions.m_bPrintBlackFont        = pOpt->IsPrintWithBlackTextColor();
+    rOptions.m_bPrintHiddenText       = pOpt->IsPrintHiddenText();
+    rOptions.m_bPrintTextPlaceholder  = pOpt->IsPrintTextPlaceholders();
+    rOptions.m_nPrintPostIts          = pOpt->GetPrintPostItsType();
 
     //! needs to be set after MakeOptions since the assignment operation in that
     //! function will destroy the pointers
diff --git a/sw/source/core/view/vprint.cxx b/sw/source/core/view/vprint.cxx
index f42c0de..4c0b4fd 100644
--- a/sw/source/core/view/vprint.cxx
+++ b/sw/source/core/view/vprint.cxx
@@ -682,12 +682,12 @@ SwDrawViewSave::~SwDrawViewSave()
 // OD 09.01.2003 #i6467# - method also called for page preview
 void SwViewShell::PrepareForPrint( const SwPrintData &rOptions )
  {
-    mpOpt->SetGraphic  ( rOptions.bPrintGraphic );
-    mpOpt->SetTable    ( rOptions.bPrintTable );
-    mpOpt->SetDraw     ( rOptions.bPrintDraw  );
-    mpOpt->SetControl  ( rOptions.bPrintControl );
-    mpOpt->SetPageBack ( rOptions.bPrintPageBackground );
-    mpOpt->SetBlackFont( rOptions.bPrintBlackFont );
+    mpOpt->SetGraphic  ( rOptions.m_bPrintGraphic );
+    mpOpt->SetTable    ( rOptions.m_bPrintTable );
+    mpOpt->SetDraw     ( rOptions.m_bPrintDraw  );
+    mpOpt->SetControl  ( rOptions.m_bPrintControl );
+    mpOpt->SetPageBack ( rOptions.m_bPrintPageBackground );
+    mpOpt->SetBlackFont( rOptions.m_bPrintBlackFont );
 
     if ( HasDrawView() )
     {
@@ -695,11 +695,11 @@ void SwViewShell::PrepareForPrint( const SwPrintData &rOptions )
         // OD 09.01.2003 #i6467# - consider, if view shell belongs to page preview
         if ( !IsPreview() )
         {
-            pDrawView->SetLayerPrintable( "Controls", rOptions.bPrintControl );
+            pDrawView->SetLayerPrintable( "Controls", rOptions.m_bPrintControl );
         }
         else
         {
-            pDrawView->SetLayerVisible( "Controls", rOptions.bPrintControl );
+            pDrawView->SetLayerVisible( "Controls", rOptions.m_bPrintControl );
         }
     }
 }
diff --git a/sw/source/ui/config/optpage.cxx b/sw/source/ui/config/optpage.cxx
index 5ba7605..34a6ec1 100644
--- a/sw/source/ui/config/optpage.cxx
+++ b/sw/source/ui/config/optpage.cxx
@@ -408,37 +408,37 @@ bool    SwAddPrinterTabPage::FillItemSet( SfxItemSet* rCoreSet )
     if ( bAttrModified )
     {
         SwAddPrinterItem aAddPrinterAttr (FN_PARAM_ADDPRINTER);
-        aAddPrinterAttr.bPrintGraphic   = m_pGrfCB->IsChecked();
-        aAddPrinterAttr.bPrintTable     = true; // always enabled since CWS printerpullgpages /*aTabCB.IsChecked();*/
-        aAddPrinterAttr.bPrintDraw      = m_pGrfCB->IsChecked(); // UI merged with m_pGrfCB in CWS printerpullgpages
-        aAddPrinterAttr.bPrintControl   = m_pCtrlFieldCB->IsChecked();
-        aAddPrinterAttr.bPrintPageBackground = m_pBackgroundCB->IsChecked();
-        aAddPrinterAttr.bPrintBlackFont = m_pBlackFontCB->IsChecked();
-        aAddPrinterAttr.bPrintHiddenText = m_pPrintHiddenTextCB->IsChecked();
-        aAddPrinterAttr.bPrintTextPlaceholder = m_pPrintTextPlaceholderCB->IsChecked();
-
-        aAddPrinterAttr.bPrintLeftPages     = m_pLeftPageCB->IsChecked();
-        aAddPrinterAttr.bPrintRightPages    = m_pRightPageCB->IsChecked();
-        aAddPrinterAttr.bPrintReverse       = false; // handled by vcl itself since CWS printerpullpages /*aReverseCB.IsChecked()*/;
-        aAddPrinterAttr.bPrintProspect      = m_pProspectCB->IsChecked();
-        aAddPrinterAttr.bPrintProspectRTL   = m_pProspectCB_RTL->IsChecked();
-        aAddPrinterAttr.bPaperFromSetup     = m_pPaperFromSetupCB->IsChecked();
-        aAddPrinterAttr.bPrintEmptyPages    = m_pPrintEmptyPagesCB->IsChecked();
-        aAddPrinterAttr.bPrintSingleJobs    = true; // handled by vcl in new print dialog since CWS printerpullpages /*aSingleJobsCB.IsChecked()*/;
-
-        if (m_pNoRB->IsChecked())  aAddPrinterAttr.nPrintPostIts =
+        aAddPrinterAttr.m_bPrintGraphic   = m_pGrfCB->IsChecked();
+        aAddPrinterAttr.m_bPrintTable     = true; // always enabled since CWS printerpullgpages /*aTabCB.IsChecked();*/
+        aAddPrinterAttr.m_bPrintDraw      = m_pGrfCB->IsChecked(); // UI merged with m_pGrfCB in CWS printerpullgpages
+        aAddPrinterAttr.m_bPrintControl   = m_pCtrlFieldCB->IsChecked();
+        aAddPrinterAttr.m_bPrintPageBackground = m_pBackgroundCB->IsChecked();
+        aAddPrinterAttr.m_bPrintBlackFont = m_pBlackFontCB->IsChecked();
+        aAddPrinterAttr.m_bPrintHiddenText = m_pPrintHiddenTextCB->IsChecked();
+        aAddPrinterAttr.m_bPrintTextPlaceholder = m_pPrintTextPlaceholderCB->IsChecked();
+
+        aAddPrinterAttr.m_bPrintLeftPages     = m_pLeftPageCB->IsChecked();
+        aAddPrinterAttr.m_bPrintRightPages    = m_pRightPageCB->IsChecked();
+        aAddPrinterAttr.m_bPrintReverse       = false; // handled by vcl itself since CWS printerpullpages /*aReverseCB.IsChecked()*/;
+        aAddPrinterAttr.m_bPrintProspect      = m_pProspectCB->IsChecked();
+        aAddPrinterAttr.m_bPrintProspectRTL   = m_pProspectCB_RTL->IsChecked();
+        aAddPrinterAttr.m_bPaperFromSetup     = m_pPaperFromSetupCB->IsChecked();
+        aAddPrinterAttr.m_bPrintEmptyPages    = m_pPrintEmptyPagesCB->IsChecked();
+        aAddPrinterAttr.m_bPrintSingleJobs    = true; // handled by vcl in new print dialog since CWS printerpullpages /*aSingleJobsCB.IsChecked()*/;
+
+        if (m_pNoRB->IsChecked())  aAddPrinterAttr.m_nPrintPostIts =
                                                         SwPostItMode::NONE;
-        if (m_pOnlyRB->IsChecked()) aAddPrinterAttr.nPrintPostIts =
+        if (m_pOnlyRB->IsChecked()) aAddPrinterAttr.m_nPrintPostIts =
                                                         SwPostItMode::Only;
-        if (m_pEndRB->IsChecked()) aAddPrinterAttr.nPrintPostIts =
+        if (m_pEndRB->IsChecked()) aAddPrinterAttr.m_nPrintPostIts =
                                                         SwPostItMode::EndDoc;
-        if (m_pEndPageRB->IsChecked()) aAddPrinterAttr.nPrintPostIts =
+        if (m_pEndPageRB->IsChecked()) aAddPrinterAttr.m_nPrintPostIts =
                                                         SwPostItMode::EndPage;
-        if (m_pInMarginsRB->IsChecked()) aAddPrinterAttr.nPrintPostIts =
+        if (m_pInMarginsRB->IsChecked()) aAddPrinterAttr.m_nPrintPostIts =
                                                         SwPostItMode::InMargins;
 
         const OUString sFax = m_pFaxLB->GetSelectEntry();
-        aAddPrinterAttr.sFaxName = sNone == sFax ? aEmptyOUStr : sFax;
+        aAddPrinterAttr.m_sFaxName = sNone == sFax ? aEmptyOUStr : sFax;
         rCoreSet->Put(aAddPrinterAttr);
     }
     return bAttrModified;
@@ -452,25 +452,25 @@ void    SwAddPrinterTabPage::Reset( const SfxItemSet*  )
     if( SfxItemState::SET == rSet.GetItemState( FN_PARAM_ADDPRINTER , false,
                                     reinterpret_cast<const SfxPoolItem**>(&pAddPrinterAttr) ))
     {
-        m_pGrfCB->Check(pAddPrinterAttr->bPrintGraphic || pAddPrinterAttr->bPrintDraw);
-        m_pCtrlFieldCB->Check(       pAddPrinterAttr->bPrintControl);
-        m_pBackgroundCB->Check(    pAddPrinterAttr->bPrintPageBackground);
-        m_pBlackFontCB->Check(     pAddPrinterAttr->bPrintBlackFont);
-        m_pPrintHiddenTextCB->Check( pAddPrinterAttr->bPrintHiddenText);
-        m_pPrintTextPlaceholderCB->Check(pAddPrinterAttr->bPrintTextPlaceholder);
-        m_pLeftPageCB->Check(      pAddPrinterAttr->bPrintLeftPages);
-        m_pRightPageCB->Check(     pAddPrinterAttr->bPrintRightPages);
-        m_pPaperFromSetupCB->Check(pAddPrinterAttr->bPaperFromSetup);
-        m_pPrintEmptyPagesCB->Check(pAddPrinterAttr->bPrintEmptyPages);
-        m_pProspectCB->Check(      pAddPrinterAttr->bPrintProspect);
-        m_pProspectCB_RTL->Check(      pAddPrinterAttr->bPrintProspectRTL);
-
-        m_pNoRB->Check (pAddPrinterAttr->nPrintPostIts== SwPostItMode::NONE ) ;
-        m_pOnlyRB->Check (pAddPrinterAttr->nPrintPostIts== SwPostItMode::Only ) ;
-        m_pEndRB->Check (pAddPrinterAttr->nPrintPostIts== SwPostItMode::EndDoc ) ;
-        m_pEndPageRB->Check (pAddPrinterAttr->nPrintPostIts== SwPostItMode::EndPage ) ;
-        m_pInMarginsRB->Check (pAddPrinterAttr->nPrintPostIts== SwPostItMode::InMargins ) ;
-        m_pFaxLB->SelectEntry( pAddPrinterAttr->sFaxName );
+        m_pGrfCB->Check(pAddPrinterAttr->m_bPrintGraphic || pAddPrinterAttr->m_bPrintDraw);
+        m_pCtrlFieldCB->Check(       pAddPrinterAttr->m_bPrintControl);
+        m_pBackgroundCB->Check(    pAddPrinterAttr->m_bPrintPageBackground);
+        m_pBlackFontCB->Check(     pAddPrinterAttr->m_bPrintBlackFont);
+        m_pPrintHiddenTextCB->Check( pAddPrinterAttr->m_bPrintHiddenText);
+        m_pPrintTextPlaceholderCB->Check(pAddPrinterAttr->m_bPrintTextPlaceholder);
+        m_pLeftPageCB->Check(      pAddPrinterAttr->m_bPrintLeftPages);
+        m_pRightPageCB->Check(     pAddPrinterAttr->m_bPrintRightPages);
+        m_pPaperFromSetupCB->Check(pAddPrinterAttr->m_bPaperFromSetup);
+        m_pPrintEmptyPagesCB->Check(pAddPrinterAttr->m_bPrintEmptyPages);
+        m_pProspectCB->Check(      pAddPrinterAttr->m_bPrintProspect);
+        m_pProspectCB_RTL->Check(      pAddPrinterAttr->m_bPrintProspectRTL);
+
+        m_pNoRB->Check (pAddPrinterAttr->m_nPrintPostIts== SwPostItMode::NONE ) ;
+        m_pOnlyRB->Check (pAddPrinterAttr->m_nPrintPostIts== SwPostItMode::Only ) ;
+        m_pEndRB->Check (pAddPrinterAttr->m_nPrintPostIts== SwPostItMode::EndDoc ) ;
+        m_pEndPageRB->Check (pAddPrinterAttr->m_nPrintPostIts== SwPostItMode::EndPage ) ;
+        m_pInMarginsRB->Check (pAddPrinterAttr->m_nPrintPostIts== SwPostItMode::InMargins ) ;
+        m_pFaxLB->SelectEntry( pAddPrinterAttr->m_sFaxName );
     }
     if (m_pProspectCB->IsChecked())
     {
diff --git a/sw/source/uibase/config/prtopt.cxx b/sw/source/uibase/config/prtopt.cxx
index f182ca2..bb0af27 100644
--- a/sw/source/uibase/config/prtopt.cxx
+++ b/sw/source/uibase/config/prtopt.cxx
@@ -67,11 +67,11 @@ SwPrintOptions::SwPrintOptions(bool bWeb) :
         ConfigItemMode::DelayedUpdate|ConfigItemMode::ReleaseTree),
     bIsWeb(bWeb)
 {
-    bPrintPageBackground = !bWeb;
-    bPrintBlackFont = bWeb;
-    bPrintTextPlaceholder = bPrintHiddenText = false;
+    m_bPrintPageBackground = !bWeb;
+    m_bPrintBlackFont = bWeb;
+    m_bPrintTextPlaceholder = m_bPrintHiddenText = false;
     if (bWeb)
-        bPrintEmptyPages = false;
+        m_bPrintEmptyPages = false;
 
     Sequence<OUString> aNames = GetPropertyNames();
     Sequence<Any> aValues = GetProperties(aNames);
@@ -85,30 +85,30 @@ SwPrintOptions::SwPrintOptions(bool bWeb) :
             {
                 switch(nProp)
                 {
-                    case  0: bPrintGraphic      = *static_cast<sal_Bool const *>(pValues[nProp].getValue()); break;
-                    case  1: bPrintTable            = *static_cast<sal_Bool const *>(pValues[nProp].getValue());  break;
-                    case  2: bPrintControl      = *static_cast<sal_Bool const *>(pValues[nProp].getValue());  break;
-                    case  3: bPrintPageBackground= *static_cast<sal_Bool const *>(pValues[nProp].getValue());  break;
-                    case  4: bPrintBlackFont        = *static_cast<sal_Bool const *>(pValues[nProp].getValue());  break;
+                    case  0: m_bPrintGraphic      = *static_cast<sal_Bool const *>(pValues[nProp].getValue()); break;
+                    case  1: m_bPrintTable            = *static_cast<sal_Bool const *>(pValues[nProp].getValue());  break;
+                    case  2: m_bPrintControl      = *static_cast<sal_Bool const *>(pValues[nProp].getValue());  break;
+                    case  3: m_bPrintPageBackground= *static_cast<sal_Bool const *>(pValues[nProp].getValue());  break;
+                    case  4: m_bPrintBlackFont        = *static_cast<sal_Bool const *>(pValues[nProp].getValue());  break;
                     case  5:
                     {
                         sal_Int32 nTmp = 0;
                         pValues[nProp] >>=  nTmp;
-                        nPrintPostIts = static_cast<SwPostItMode>(nTmp);
+                        m_nPrintPostIts = static_cast<SwPostItMode>(nTmp);
                     }
                     break;
-                    case  6: bPrintReverse      = *static_cast<sal_Bool const *>(pValues[nProp].getValue());  break;
-                    case  7: bPrintProspect      = *static_cast<sal_Bool const *>(pValues[nProp].getValue());  break;
-                    case  8: bPrintProspectRTL  = *static_cast<sal_Bool const *>(pValues[nProp].getValue());  break;
-                    case  9: bPrintSingleJobs   = *static_cast<sal_Bool const *>(pValues[nProp].getValue());  break;
-                    case 10: pValues[nProp] >>= sFaxName;  break;
-                    case 11: bPaperFromSetup    = *static_cast<sal_Bool const *>(pValues[nProp].getValue());  break;
-                    case 12: bPrintDraw         = *static_cast<sal_Bool const *>(pValues[nProp].getValue());  break;
-                    case 13: bPrintLeftPages    = *static_cast<sal_Bool const *>(pValues[nProp].getValue());  break;
-                    case 14: bPrintRightPages       = *static_cast<sal_Bool const *>(pValues[nProp].getValue());  break;
-                    case 15: bPrintEmptyPages       = *static_cast<sal_Bool const *>(pValues[nProp].getValue());  break;
-                    case 16: bPrintTextPlaceholder = *static_cast<sal_Bool const *>(pValues[nProp].getValue());  break;
-                    case 17: bPrintHiddenText = *static_cast<sal_Bool const *>(pValues[nProp].getValue());  break;
+                    case  6: m_bPrintReverse      = *static_cast<sal_Bool const *>(pValues[nProp].getValue());  break;
+                    case  7: m_bPrintProspect      = *static_cast<sal_Bool const *>(pValues[nProp].getValue());  break;
+                    case  8: m_bPrintProspectRTL  = *static_cast<sal_Bool const *>(pValues[nProp].getValue());  break;
+                    case  9: m_bPrintSingleJobs   = *static_cast<sal_Bool const *>(pValues[nProp].getValue());  break;
+                    case 10: pValues[nProp] >>= m_sFaxName;  break;
+                    case 11: m_bPaperFromSetup    = *static_cast<sal_Bool const *>(pValues[nProp].getValue());  break;
+                    case 12: m_bPrintDraw         = *static_cast<sal_Bool const *>(pValues[nProp].getValue());  break;
+                    case 13: m_bPrintLeftPages    = *static_cast<sal_Bool const *>(pValues[nProp].getValue());  break;
+                    case 14: m_bPrintRightPages       = *static_cast<sal_Bool const *>(pValues[nProp].getValue());  break;
+                    case 15: m_bPrintEmptyPages       = *static_cast<sal_Bool const *>(pValues[nProp].getValue());  break;
+                    case 16: m_bPrintTextPlaceholder = *static_cast<sal_Bool const *>(pValues[nProp].getValue());  break;
+                    case 17: m_bPrintHiddenText = *static_cast<sal_Bool const *>(pValues[nProp].getValue());  break;
                 }
             }
         }
@@ -118,7 +118,7 @@ SwPrintOptions::SwPrintOptions(bool bWeb) :
     // In the UI. (File/Print dialog and Tools/Options/.../Print)
     // And since print graphics is the only available in Writer and WrtierWeb ...
 
-    bPrintDraw = bPrintGraphic;
+    m_bPrintDraw = m_bPrintGraphic;
 }
 
 SwPrintOptions::~SwPrintOptions()
@@ -138,31 +138,31 @@ void SwPrintOptions::ImplCommit()
     {
         switch(nProp)
         {
-            case  0: pValues[nProp] <<= bPrintGraphic; break;
-            case  1: pValues[nProp] <<= bPrintTable; break;
-            case  2: pValues[nProp] <<= bPrintControl; break;
-            case  3: pValues[nProp] <<= bPrintPageBackground; break;
-            case  4: pValues[nProp] <<= bPrintBlackFont; break;
-            case  5: pValues[nProp] <<=  (sal_Int32)nPrintPostIts       ; break;
-            case  6: pValues[nProp] <<= bPrintReverse; break;
-            case  7: pValues[nProp] <<= bPrintProspect; break;
-            case  8: pValues[nProp] <<= bPrintProspectRTL; break;
-            case  9: pValues[nProp] <<= bPrintSingleJobs; break;
-            case 10: pValues[nProp] <<= sFaxName;  break;
-            case 11: pValues[nProp] <<= bPaperFromSetup; break;
-            case 12: pValues[nProp] <<= bPrintDraw; break;
-            case 13: pValues[nProp] <<= bPrintLeftPages; break;
-            case 14: pValues[nProp] <<= bPrintRightPages; break;
-            case 15: pValues[nProp] <<= bPrintEmptyPages; break;
-            case 16: pValues[nProp] <<= bPrintTextPlaceholder; break;
-            case 17: pValues[nProp] <<= bPrintHiddenText; break;
+            case  0: pValues[nProp] <<= m_bPrintGraphic; break;
+            case  1: pValues[nProp] <<= m_bPrintTable; break;
+            case  2: pValues[nProp] <<= m_bPrintControl; break;
+            case  3: pValues[nProp] <<= m_bPrintPageBackground; break;
+            case  4: pValues[nProp] <<= m_bPrintBlackFont; break;
+            case  5: pValues[nProp] <<=  (sal_Int32)m_nPrintPostIts       ; break;
+            case  6: pValues[nProp] <<= m_bPrintReverse; break;
+            case  7: pValues[nProp] <<= m_bPrintProspect; break;
+            case  8: pValues[nProp] <<= m_bPrintProspectRTL; break;
+            case  9: pValues[nProp] <<= m_bPrintSingleJobs; break;
+            case 10: pValues[nProp] <<= m_sFaxName;  break;
+            case 11: pValues[nProp] <<= m_bPaperFromSetup; break;
+            case 12: pValues[nProp] <<= m_bPrintDraw; break;
+            case 13: pValues[nProp] <<= m_bPrintLeftPages; break;
+            case 14: pValues[nProp] <<= m_bPrintRightPages; break;
+            case 15: pValues[nProp] <<= m_bPrintEmptyPages; break;
+            case 16: pValues[nProp] <<= m_bPrintTextPlaceholder; break;
+            case 17: pValues[nProp] <<= m_bPrintHiddenText; break;
         }
     }
 
     // currently there is just one checkbox for print drawings and print graphics
     // In the UI. (File/Print dialog and Tools/Options/.../Print)
     // And since print graphics is the only available in Writer and WrtierWeb ...
-    bPrintDraw = bPrintGraphic;
+    m_bPrintDraw = m_bPrintGraphic;
 
     PutProperties(aNames, aValues);
 }
diff --git a/sw/source/uibase/inc/cfgitems.hxx b/sw/source/uibase/inc/cfgitems.hxx
index 5092848..2677f98 100644
--- a/sw/source/uibase/inc/cfgitems.hxx
+++ b/sw/source/uibase/inc/cfgitems.hxx
@@ -118,7 +118,7 @@ public:
 
     virtual bool         operator==( const SfxPoolItem& ) const override;
 
-    const OUString &GetFax() const              { return sFaxName; }
+    const OUString &GetFax() const              { return m_sFaxName; }
 };
 
 // Item for settings dialog, ShadowCursorPage
diff --git a/sw/source/uibase/uno/unotxdoc.cxx b/sw/source/uibase/uno/unotxdoc.cxx
index 104a5d2..094e374 100644
--- a/sw/source/uibase/uno/unotxdoc.cxx
+++ b/sw/source/uibase/uno/unotxdoc.cxx
@@ -4266,7 +4266,7 @@ SwViewOptionAdjust_Impl::AdjustViewOptions(SwPrintData const*const pPrtOptions)
     aRenderViewOptions.SetLineBreak( false );             // breaks (type 1)
     aRenderViewOptions.SetPageBreak( false );             // breaks (type 2)
     aRenderViewOptions.SetColumnBreak( false );           // breaks (type 3)
-    bool bVal = pPrtOptions && pPrtOptions->bPrintHiddenText;
+    bool bVal = pPrtOptions && pPrtOptions->m_bPrintHiddenText;
     if (bContainsHiddenChars)
         aRenderViewOptions.SetShowHiddenChar( bVal );     // hidden text
     if (bContainsHiddenFields)
@@ -4277,7 +4277,7 @@ SwViewOptionAdjust_Impl::AdjustViewOptions(SwPrintData const*const pPrtOptions)
     if (bContainsPlaceHolders)
     {
         // should always be printed in PDF export!
-        bVal = !pPrtOptions || pPrtOptions->bPrintTextPlaceholder;
+        bVal = !pPrtOptions || pPrtOptions->m_bPrintTextPlaceholder;
         aRenderViewOptions.SetShowPlaceHolderFields( bVal );
     }
 
commit 441412a0bc33bd9c6fe6a19a901213e113104a14
Author: Miklos Vajna <vmiklos at collabora.co.uk>
Date:   Sat Nov 14 09:05:15 2015 +0100

    sw: prefix members of SwSectionFootnoteEndTabPage
    
    Change-Id: I785b50d6f09db79d3184a19c01e71ef26c11b1d1

diff --git a/sw/source/ui/dialog/uiregionsw.cxx b/sw/source/ui/dialog/uiregionsw.cxx
index a185fbd..35a7b60 100644
--- a/sw/source/ui/dialog/uiregionsw.cxx
+++ b/sw/source/ui/dialog/uiregionsw.cxx
@@ -1817,39 +1817,39 @@ SwSectionFootnoteEndTabPage::SwSectionFootnoteEndTabPage( vcl::Window *pParent,
     : SfxTabPage( pParent, "FootnotesEndnotesTabPage", "modules/swriter/ui/footnotesendnotestabpage.ui", &rAttrSet )
 
 {
-    get(pFootnoteNtAtTextEndCB,"ftnntattextend");
+    get(m_pFootnoteNtAtTextEndCB,"ftnntattextend");
 
-    get(pFootnoteNtNumCB,"ftnntnum");
-    get(pFootnoteOffsetLbl,"ftnoffset_label");
-    get(pFootnoteOffsetField,"ftnoffset");
+    get(m_pFootnoteNtNumCB,"ftnntnum");
+    get(m_pFootnoteOffsetLbl,"ftnoffset_label");
+    get(m_pFootnoteOffsetField,"ftnoffset");
 
-    get(pFootnoteNtNumFormatCB,"ftnntnumfmt");
-    get(pFootnotePrefixFT,"ftnprefix_label");
-    get(pFootnotePrefixED,"ftnprefix");
-    get(pFootnoteNumViewBox,"ftnnumviewbox");
-    get(pFootnoteSuffixFT,"ftnsuffix_label");
-    get(pFootnoteSuffixED,"ftnsuffix");
+    get(m_pFootnoteNtNumFormatCB,"ftnntnumfmt");
+    get(m_pFootnotePrefixFT,"ftnprefix_label");
+    get(m_pFootnotePrefixED,"ftnprefix");
+    get(m_pFootnoteNumViewBox,"ftnnumviewbox");
+    get(m_pFootnoteSuffixFT,"ftnsuffix_label");
+    get(m_pFootnoteSuffixED,"ftnsuffix");
 
-    get(pEndNtAtTextEndCB,"endntattextend");
+    get(m_pEndNtAtTextEndCB,"endntattextend");
 
-    get(pEndNtNumCB,"endntnum");
-    get(pEndOffsetLbl,"endoffset_label");
-    get(pEndOffsetField,"endoffset");
+    get(m_pEndNtNumCB,"endntnum");
+    get(m_pEndOffsetLbl,"endoffset_label");
+    get(m_pEndOffsetField,"endoffset");
 
-    get(pEndNtNumFormatCB,"endntnumfmt");
-    get(pEndPrefixFT,"endprefix_label");
-    get(pEndPrefixED,"endprefix");
-    get(pEndNumViewBox,"endnumviewbox");
-    get(pEndSuffixFT,"endsuffix_label");
-    get(pEndSuffixED,"endsuffix");
+    get(m_pEndNtNumFormatCB,"endntnumfmt");
+    get(m_pEndPrefixFT,"endprefix_label");
+    get(m_pEndPrefixED,"endprefix");
+    get(m_pEndNumViewBox,"endnumviewbox");
+    get(m_pEndSuffixFT,"endsuffix_label");
+    get(m_pEndSuffixED,"endsuffix");
 
     Link<Button*,void> aLk( LINK( this, SwSectionFootnoteEndTabPage, FootEndHdl));
-    pFootnoteNtAtTextEndCB->SetClickHdl( aLk );
-    pFootnoteNtNumCB->SetClickHdl( aLk );
-    pEndNtAtTextEndCB->SetClickHdl( aLk );
-    pEndNtNumCB->SetClickHdl( aLk );
-    pFootnoteNtNumFormatCB->SetClickHdl( aLk );
-    pEndNtNumFormatCB->SetClickHdl( aLk );
+    m_pFootnoteNtAtTextEndCB->SetClickHdl( aLk );
+    m_pFootnoteNtNumCB->SetClickHdl( aLk );
+    m_pEndNtAtTextEndCB->SetClickHdl( aLk );
+    m_pEndNtNumCB->SetClickHdl( aLk );
+    m_pFootnoteNtNumFormatCB->SetClickHdl( aLk );
+    m_pEndNtNumFormatCB->SetClickHdl( aLk );
 }
 
 SwSectionFootnoteEndTabPage::~SwSectionFootnoteEndTabPage()
@@ -1859,34 +1859,34 @@ SwSectionFootnoteEndTabPage::~SwSectionFootnoteEndTabPage()
 
 void SwSectionFootnoteEndTabPage::dispose()
 {
-    pFootnoteNtAtTextEndCB.clear();
-    pFootnoteNtNumCB.clear();
-    pFootnoteOffsetLbl.clear();
-    pFootnoteOffsetField.clear();
-    pFootnoteNtNumFormatCB.clear();
-    pFootnotePrefixFT.clear();
-    pFootnotePrefixED.clear();
-    pFootnoteNumViewBox.clear();
-    pFootnoteSuffixFT.clear();
-    pFootnoteSuffixED.clear();
-    pEndNtAtTextEndCB.clear();
-    pEndNtNumCB.clear();
-    pEndOffsetLbl.clear();
-    pEndOffsetField.clear();
-    pEndNtNumFormatCB.clear();
-    pEndPrefixFT.clear();
-    pEndPrefixED.clear();
-    pEndNumViewBox.clear();
-    pEndSuffixFT.clear();
-    pEndSuffixED.clear();
+    m_pFootnoteNtAtTextEndCB.clear();
+    m_pFootnoteNtNumCB.clear();
+    m_pFootnoteOffsetLbl.clear();
+    m_pFootnoteOffsetField.clear();
+    m_pFootnoteNtNumFormatCB.clear();
+    m_pFootnotePrefixFT.clear();
+    m_pFootnotePrefixED.clear();
+    m_pFootnoteNumViewBox.clear();
+    m_pFootnoteSuffixFT.clear();
+    m_pFootnoteSuffixED.clear();
+    m_pEndNtAtTextEndCB.clear();
+    m_pEndNtNumCB.clear();
+    m_pEndOffsetLbl.clear();
+    m_pEndOffsetField.clear();
+    m_pEndNtNumFormatCB.clear();
+    m_pEndPrefixFT.clear();
+    m_pEndPrefixED.clear();
+    m_pEndNumViewBox.clear();
+    m_pEndSuffixFT.clear();
+    m_pEndSuffixED.clear();
     SfxTabPage::dispose();
 }
 
 bool SwSectionFootnoteEndTabPage::FillItemSet( SfxItemSet* rSet )
 {
-    SwFormatFootnoteAtTextEnd aFootnote( pFootnoteNtAtTextEndCB->IsChecked()
-                            ? ( pFootnoteNtNumCB->IsChecked()
-                                ? ( pFootnoteNtNumFormatCB->IsChecked()
+    SwFormatFootnoteAtTextEnd aFootnote( m_pFootnoteNtAtTextEndCB->IsChecked()
+                            ? ( m_pFootnoteNtNumCB->IsChecked()
+                                ? ( m_pFootnoteNtNumFormatCB->IsChecked()
                                     ? FTNEND_ATTXTEND_OWNNUMANDFMT
                                     : FTNEND_ATTXTEND_OWNNUMSEQ )
                                 : FTNEND_ATTXTEND )
@@ -1895,19 +1895,19 @@ bool SwSectionFootnoteEndTabPage::FillItemSet( SfxItemSet* rSet )
     switch( aFootnote.GetValue() )
     {
     case FTNEND_ATTXTEND_OWNNUMANDFMT:
-        aFootnote.SetNumType( pFootnoteNumViewBox->GetSelectedNumberingType() );
-        aFootnote.SetPrefix( pFootnotePrefixED->GetText().replaceAll("\\t", "\t") ); // fdo#65666
-        aFootnote.SetSuffix( pFootnoteSuffixED->GetText().replaceAll("\\t", "\t") );
+        aFootnote.SetNumType( m_pFootnoteNumViewBox->GetSelectedNumberingType() );
+        aFootnote.SetPrefix( m_pFootnotePrefixED->GetText().replaceAll("\\t", "\t") ); // fdo#65666
+        aFootnote.SetSuffix( m_pFootnoteSuffixED->GetText().replaceAll("\\t", "\t") );
         // no break;
 
     case FTNEND_ATTXTEND_OWNNUMSEQ:
-        aFootnote.SetOffset( static_cast< sal_uInt16 >( pFootnoteOffsetField->GetValue()-1 ) );
+        aFootnote.SetOffset( static_cast< sal_uInt16 >( m_pFootnoteOffsetField->GetValue()-1 ) );
         // no break;
     }
 
-    SwFormatEndAtTextEnd aEnd( pEndNtAtTextEndCB->IsChecked()
-                            ? ( pEndNtNumCB->IsChecked()
-                                ? ( pEndNtNumFormatCB->IsChecked()
+    SwFormatEndAtTextEnd aEnd( m_pEndNtAtTextEndCB->IsChecked()
+                            ? ( m_pEndNtNumCB->IsChecked()
+                                ? ( m_pEndNtNumFormatCB->IsChecked()
                                     ? FTNEND_ATTXTEND_OWNNUMANDFMT
                                     : FTNEND_ATTXTEND_OWNNUMSEQ )
                                 : FTNEND_ATTXTEND )
@@ -1916,13 +1916,13 @@ bool SwSectionFootnoteEndTabPage::FillItemSet( SfxItemSet* rSet )
     switch( aEnd.GetValue() )
     {
     case FTNEND_ATTXTEND_OWNNUMANDFMT:
-        aEnd.SetNumType( pEndNumViewBox->GetSelectedNumberingType() );
-        aEnd.SetPrefix( pEndPrefixED->GetText().replaceAll("\\t", "\t") );
-        aEnd.SetSuffix( pEndSuffixED->GetText().replaceAll("\\t", "\t") );
+        aEnd.SetNumType( m_pEndNumViewBox->GetSelectedNumberingType() );
+        aEnd.SetPrefix( m_pEndPrefixED->GetText().replaceAll("\\t", "\t") );
+        aEnd.SetSuffix( m_pEndSuffixED->GetText().replaceAll("\\t", "\t") );
         // no break;
 
     case FTNEND_ATTXTEND_OWNNUMSEQ:
-        aEnd.SetOffset( static_cast< sal_uInt16 >( pEndOffsetField->GetValue()-1 ) );
+        aEnd.SetOffset( static_cast< sal_uInt16 >( m_pEndOffsetField->GetValue()-1 ) );
         // no break;
     }
 
@@ -1944,29 +1944,29 @@ void SwSectionFootnoteEndTabPage::ResetState( bool bFootnote,
 
     if( bFootnote )
     {
-        pNtAtTextEndCB = pFootnoteNtAtTextEndCB;
-        pNtNumCB = pFootnoteNtNumCB;
-        pNtNumFormatCB = pFootnoteNtNumFormatCB;
-        pPrefixFT = pFootnotePrefixFT;
-        pPrefixED = pFootnotePrefixED;
-        pSuffixFT = pFootnoteSuffixFT;
-        pSuffixED = pFootnoteSuffixED;
-        pNumViewBox = pFootnoteNumViewBox;
-        pOffsetText = pFootnoteOffsetLbl;
-        pOffsetField = pFootnoteOffsetField;
+        pNtAtTextEndCB = m_pFootnoteNtAtTextEndCB;
+        pNtNumCB = m_pFootnoteNtNumCB;
+        pNtNumFormatCB = m_pFootnoteNtNumFormatCB;
+        pPrefixFT = m_pFootnotePrefixFT;
+        pPrefixED = m_pFootnotePrefixED;
+        pSuffixFT = m_pFootnoteSuffixFT;
+        pSuffixED = m_pFootnoteSuffixED;
+        pNumViewBox = m_pFootnoteNumViewBox;
+        pOffsetText = m_pFootnoteOffsetLbl;
+        pOffsetField = m_pFootnoteOffsetField;
     }
     else
     {
-        pNtAtTextEndCB = pEndNtAtTextEndCB;
-        pNtNumCB = pEndNtNumCB;
-        pNtNumFormatCB = pEndNtNumFormatCB;
-        pPrefixFT = pEndPrefixFT;
-        pPrefixED = pEndPrefixED;
-        pSuffixFT = pEndSuffixFT;
-        pSuffixED = pEndSuffixED;
-        pNumViewBox = pEndNumViewBox;
-        pOffsetText = pEndOffsetLbl;
-        pOffsetField = pEndOffsetField;
+        pNtAtTextEndCB = m_pEndNtAtTextEndCB;
+        pNtNumCB = m_pEndNtNumCB;
+        pNtNumFormatCB = m_pEndNtNumFormatCB;
+        pPrefixFT = m_pEndPrefixFT;
+        pPrefixED = m_pEndPrefixED;
+        pSuffixFT = m_pEndSuffixFT;
+        pSuffixED = m_pEndSuffixED;
+        pNumViewBox = m_pEndNumViewBox;
+        pOffsetText = m_pEndOffsetLbl;
+        pOffsetField = m_pEndOffsetField;
     }
 
     const sal_uInt16 eState = rAttr.GetValue();
@@ -2028,8 +2028,8 @@ VclPtr<SfxTabPage> SwSectionFootnoteEndTabPage::Create( vcl::Window* pParent,
 
 IMPL_LINK_TYPED( SwSectionFootnoteEndTabPage, FootEndHdl, Button *, pBox, void )
 {
-    bool bFoot = pFootnoteNtAtTextEndCB == pBox || pFootnoteNtNumCB == pBox ||
-                    pFootnoteNtNumFormatCB == pBox ;
+    bool bFoot = m_pFootnoteNtAtTextEndCB == pBox || m_pFootnoteNtNumCB == pBox ||
+                    m_pFootnoteNtNumFormatCB == pBox ;
 
     CheckBox *pNumBox, *pNumFormatBox, *pEndBox;
     SwNumberingTypeListBox* pNumViewBox;
@@ -2040,29 +2040,29 @@ IMPL_LINK_TYPED( SwSectionFootnoteEndTabPage, FootEndHdl, Button *, pBox, void )
 
     if( bFoot )
     {
-        pEndBox = pFootnoteNtAtTextEndCB;
-        pNumBox = pFootnoteNtNumCB;
-        pNumFormatBox = pFootnoteNtNumFormatCB;
-        pNumViewBox = pFootnoteNumViewBox;
-        pOffsetText = pFootnoteOffsetLbl;
-        pOffsetField = pFootnoteOffsetField;
-        pPrefixFT = pFootnotePrefixFT;
-        pSuffixFT = pFootnoteSuffixFT;
-        pPrefixED = pFootnotePrefixED;
-        pSuffixED = pFootnoteSuffixED;
+        pEndBox = m_pFootnoteNtAtTextEndCB;
+        pNumBox = m_pFootnoteNtNumCB;
+        pNumFormatBox = m_pFootnoteNtNumFormatCB;
+        pNumViewBox = m_pFootnoteNumViewBox;
+        pOffsetText = m_pFootnoteOffsetLbl;
+        pOffsetField = m_pFootnoteOffsetField;
+        pPrefixFT = m_pFootnotePrefixFT;
+        pSuffixFT = m_pFootnoteSuffixFT;
+        pPrefixED = m_pFootnotePrefixED;
+        pSuffixED = m_pFootnoteSuffixED;
     }
     else
     {
-        pEndBox = pEndNtAtTextEndCB;
-        pNumBox = pEndNtNumCB;
-        pNumFormatBox = pEndNtNumFormatCB;
-        pNumViewBox = pEndNumViewBox;
-        pOffsetText = pEndOffsetLbl;
-        pOffsetField = pEndOffsetField;
-        pPrefixFT = pEndPrefixFT;
-        pSuffixFT = pEndSuffixFT;
-        pPrefixED = pEndPrefixED;
-        pSuffixED = pEndSuffixED;
+        pEndBox = m_pEndNtAtTextEndCB;
+        pNumBox = m_pEndNtNumCB;
+        pNumFormatBox = m_pEndNtNumFormatCB;
+        pNumViewBox = m_pEndNumViewBox;
+        pOffsetText = m_pEndOffsetLbl;
+        pOffsetField = m_pEndOffsetField;
+        pPrefixFT = m_pEndPrefixFT;
+        pSuffixFT = m_pEndSuffixFT;
+        pPrefixED = m_pEndPrefixED;
+        pSuffixED = m_pEndSuffixED;
     }
 
     bool bEnableAtEnd = TRISTATE_TRUE == pEndBox->GetState();
diff --git a/sw/source/uibase/inc/regionsw.hxx b/sw/source/uibase/inc/regionsw.hxx
index fa4453d..53da375 100644
--- a/sw/source/uibase/inc/regionsw.hxx
+++ b/sw/source/uibase/inc/regionsw.hxx
@@ -196,31 +196,31 @@ public:
 
 class SwSectionFootnoteEndTabPage : public SfxTabPage
 {
-    VclPtr<CheckBox>        pFootnoteNtAtTextEndCB;
-
-    VclPtr<CheckBox>        pFootnoteNtNumCB;
-    VclPtr<FixedText>       pFootnoteOffsetLbl;
-    VclPtr<NumericField>    pFootnoteOffsetField;
-
-    VclPtr<CheckBox>        pFootnoteNtNumFormatCB;
-    VclPtr<FixedText>       pFootnotePrefixFT;
-    VclPtr<Edit>            pFootnotePrefixED;
-    VclPtr<SwNumberingTypeListBox> pFootnoteNumViewBox;
-    VclPtr<FixedText>       pFootnoteSuffixFT;
-    VclPtr<Edit>            pFootnoteSuffixED;
-
-    VclPtr<CheckBox>        pEndNtAtTextEndCB;
-
-    VclPtr<CheckBox>        pEndNtNumCB;
-    VclPtr<FixedText>       pEndOffsetLbl;
-    VclPtr<NumericField>    pEndOffsetField;
-
-    VclPtr<CheckBox>        pEndNtNumFormatCB;
-    VclPtr<FixedText>       pEndPrefixFT;
-    VclPtr<Edit>            pEndPrefixED;
-    VclPtr<SwNumberingTypeListBox> pEndNumViewBox;
-    VclPtr<FixedText>       pEndSuffixFT;
-    VclPtr<Edit>            pEndSuffixED;
+    VclPtr<CheckBox>        m_pFootnoteNtAtTextEndCB;
+
+    VclPtr<CheckBox>        m_pFootnoteNtNumCB;
+    VclPtr<FixedText>       m_pFootnoteOffsetLbl;
+    VclPtr<NumericField>    m_pFootnoteOffsetField;
+
+    VclPtr<CheckBox>        m_pFootnoteNtNumFormatCB;
+    VclPtr<FixedText>       m_pFootnotePrefixFT;
+    VclPtr<Edit>            m_pFootnotePrefixED;
+    VclPtr<SwNumberingTypeListBox> m_pFootnoteNumViewBox;
+    VclPtr<FixedText>       m_pFootnoteSuffixFT;
+    VclPtr<Edit>            m_pFootnoteSuffixED;
+
+    VclPtr<CheckBox>        m_pEndNtAtTextEndCB;
+
+    VclPtr<CheckBox>        m_pEndNtNumCB;
+    VclPtr<FixedText>       m_pEndOffsetLbl;
+    VclPtr<NumericField>    m_pEndOffsetField;
+
+    VclPtr<CheckBox>        m_pEndNtNumFormatCB;
+    VclPtr<FixedText>       m_pEndPrefixFT;
+    VclPtr<Edit>            m_pEndPrefixED;
+    VclPtr<SwNumberingTypeListBox> m_pEndNumViewBox;
+    VclPtr<FixedText>       m_pEndSuffixFT;
+    VclPtr<Edit>            m_pEndSuffixED;
 
     DECL_LINK_TYPED( FootEndHdl, Button*, void );
     void ResetState( bool bFootnote, const SwFormatFootnoteEndAtTextEnd& );
commit 412d3a8f46bcd3db1953497cec23b5d60317fd1a
Author: Miklos Vajna <vmiklos at collabora.co.uk>
Date:   Sat Nov 14 09:04:34 2015 +0100

    sw: prefix members of SwUndoDelete
    
    Change-Id: I9a31850ac3524b8f2fae6513f4d378f8616eb30d

diff --git a/sw/source/core/inc/UndoDelete.hxx b/sw/source/core/inc/UndoDelete.hxx
index dd67242..92513a2 100644
--- a/sw/source/core/inc/UndoDelete.hxx
+++ b/sw/source/core/inc/UndoDelete.hxx
@@ -36,29 +36,29 @@ class SwUndoDelete
     , private SwUndRng
     , private SwUndoSaveContent
 {
-    SwNodeIndex* pMvStt;            // Position of Nodes in UndoNodes-Array
-    OUString *pSttStr, *pEndStr;
-    SwRedlineData* pRedlData;
-    SwRedlineSaveDatas* pRedlSaveData;
+    SwNodeIndex* m_pMvStt;            // Position of Nodes in UndoNodes-Array
+    OUString *m_pSttStr, *m_pEndStr;
+    SwRedlineData* m_pRedlData;
+    SwRedlineSaveDatas* m_pRedlSaveData;
     std::shared_ptr< ::sfx2::MetadatableUndo > m_pMetadataUndoStart;
     std::shared_ptr< ::sfx2::MetadatableUndo > m_pMetadataUndoEnd;
 
-    OUString sTableName;
+    OUString m_sTableName;
 
-    sal_uLong nNode;
-    sal_uLong nNdDiff;           // difference of Nodes before/after Delete
-    sal_uLong nSectDiff;         // diff. of Nodes before/after Move w/ SectionNodes
-    sal_uLong nReplaceDummy;     // diff. to a temporary dummy object
-    sal_uInt16 nSetPos;
+    sal_uLong m_nNode;
+    sal_uLong m_nNdDiff;           // difference of Nodes before/after Delete
+    sal_uLong m_nSectDiff;         // diff. of Nodes before/after Move w/ SectionNodes
+    sal_uLong m_nReplaceDummy;     // diff. to a temporary dummy object
+    sal_uInt16 m_nSetPos;
 
-    bool bGroup : 1;         // TRUE: is already Grouped; see CanGrouping()
-    bool bBackSp : 1;        // TRUE: if Grouped and preceding content deleted
-    bool bJoinNext: 1;       // TRUE: if range is selected forwards
-    bool bTableDelLastNd : 1;  // TRUE: TextNode following Table inserted/deleted
-    bool bDelFullPara : 1;   // TRUE: entire Nodes were deleted
-    bool bResetPgDesc : 1;   // TRUE: reset PgDsc on following node
-    bool bResetPgBrk : 1;    // TRUE: reset PgBreak on following node
-    bool bFromTableCopy : 1; // TRUE: called by SwUndoTableCpyTable
+    bool m_bGroup : 1;         // TRUE: is already Grouped; see CanGrouping()
+    bool m_bBackSp : 1;        // TRUE: if Grouped and preceding content deleted
+    bool m_bJoinNext: 1;       // TRUE: if range is selected forwards
+    bool m_bTableDelLastNd : 1;  // TRUE: TextNode following Table inserted/deleted
+    bool m_bDelFullPara : 1;   // TRUE: entire Nodes were deleted
+    bool m_bResetPgDesc : 1;   // TRUE: reset PgDsc on following node
+    bool m_bResetPgBrk : 1;    // TRUE: reset PgBreak on following node
+    bool m_bFromTableCopy : 1; // TRUE: called by SwUndoTableCpyTable
 
     bool SaveContent( const SwPosition* pStt, const SwPosition* pEnd,
                     SwTextNode* pSttTextNd, SwTextNode* pEndTextNd );
@@ -88,16 +88,16 @@ public:
 
     bool CanGrouping( SwDoc*, const SwPaM& );
 
-    void SetTableDelLastNd()      { bTableDelLastNd = true; }
+    void SetTableDelLastNd()      { m_bTableDelLastNd = true; }
 
     // for PageDesc/PageBreak Attributes of a table
     void SetPgBrkFlags( bool bPageBreak, bool bPageDesc )
-        { bResetPgDesc = bPageDesc; bResetPgBrk = bPageBreak; }
+        { m_bResetPgDesc = bPageDesc; m_bResetPgBrk = bPageBreak; }
 
     void SetTableName(const OUString & rName);
 
     // SwUndoTableCpyTable needs this information:
-    bool IsDelFullPara() const { return bDelFullPara; }
+    bool IsDelFullPara() const { return m_bDelFullPara; }
 
     DECL_FIXEDMEMPOOL_NEWDEL(SwUndoDelete)
 };
diff --git a/sw/source/core/undo/undel.cxx b/sw/source/core/undo/undel.cxx
index ab19ecc..e68ce6d 100644
--- a/sw/source/core/undo/undel.cxx
+++ b/sw/source/core/undo/undel.cxx
@@ -99,25 +99,25 @@ SwUndoDelete::SwUndoDelete(
     bool bCalledByTableCpy )
     : SwUndo(UNDO_DELETE),
     SwUndRng( rPam ),
-    pMvStt( nullptr ),
-    pSttStr(nullptr),
-    pEndStr(nullptr),
-    pRedlData(nullptr),
-    pRedlSaveData(nullptr),
-    nNode(0),
-    nNdDiff(0),
-    nSectDiff(0),
-    nReplaceDummy(0),
-    nSetPos(0),
-    bGroup( false ),
-    bBackSp( false ),
-    bJoinNext( false ),
-    bTableDelLastNd( false ),
+    m_pMvStt( nullptr ),
+    m_pSttStr(nullptr),
+    m_pEndStr(nullptr),
+    m_pRedlData(nullptr),
+    m_pRedlSaveData(nullptr),
+    m_nNode(0),
+    m_nNdDiff(0),
+    m_nSectDiff(0),
+    m_nReplaceDummy(0),
+    m_nSetPos(0),
+    m_bGroup( false ),
+    m_bBackSp( false ),
+    m_bJoinNext( false ),
+    m_bTableDelLastNd( false ),
     // bFullPara is set e.g. if an empty paragraph before a table is deleted
-    bDelFullPara( bFullPara ),
-    bResetPgDesc( false ),
-    bResetPgBrk( false ),
-    bFromTableCopy( bCalledByTableCpy )
+    m_bDelFullPara( bFullPara ),
+    m_bResetPgDesc( false ),
+    m_bResetPgBrk( false ),
+    m_bFromTableCopy( bCalledByTableCpy )
 {
 
     bCacheComment = false;
@@ -126,9 +126,9 @@ SwUndoDelete::SwUndoDelete(
 
     if( !pDoc->getIDocumentRedlineAccess().IsIgnoreRedline() && !pDoc->getIDocumentRedlineAccess().GetRedlineTable().empty() )
     {
-        pRedlSaveData = new SwRedlineSaveDatas;
-        if( !FillSaveData( rPam, *pRedlSaveData ))
-            delete pRedlSaveData, pRedlSaveData = nullptr;
+        m_pRedlSaveData = new SwRedlineSaveDatas;
+        if( !FillSaveData( rPam, *m_pRedlSaveData ))
+            delete m_pRedlSaveData, m_pRedlSaveData = nullptr;
     }
 
     if( !pHistory )
@@ -141,7 +141,7 @@ SwUndoDelete::SwUndoDelete(
                         : rPam.GetPoint();
 
     // Step 1. deletion/record of content indices
-    if( bDelFullPara )
+    if( m_bDelFullPara )
     {
         OSL_ENSURE( rPam.HasMark(), "PaM ohne Mark" );
         DelContentIndex( *rPam.GetMark(), *rPam.GetPoint(),
@@ -161,13 +161,13 @@ SwUndoDelete::SwUndoDelete(
         }
     }
 
-    nSetPos = pHistory ? pHistory->Count() : 0;
+    m_nSetPos = pHistory ? pHistory->Count() : 0;
 
     // Is already anything deleted?
-    nNdDiff = nSttNode - pStt->nNode.GetIndex();
+    m_nNdDiff = nSttNode - pStt->nNode.GetIndex();
 
-    bJoinNext = !bFullPara && pEnd == rPam.GetPoint();
-    bBackSp = !bFullPara && !bJoinNext;
+    m_bJoinNext = !bFullPara && pEnd == rPam.GetPoint();
+    m_bBackSp = !bFullPara && !m_bJoinNext;
 
     SwTextNode *pSttTextNd = nullptr, *pEndTextNd = nullptr;
     if( !bFullPara )
@@ -179,7 +179,7 @@ SwUndoDelete::SwUndoDelete(
     }
 
     bool bMoveNds = *pStt != *pEnd      // any area still existent?
-                && ( SaveContent( pStt, pEnd, pSttTextNd, pEndTextNd ) || bFromTableCopy );
+                && ( SaveContent( pStt, pEnd, pSttTextNd, pEndTextNd ) || m_bFromTableCopy );
 
     if( pSttTextNd && pEndTextNd && pSttTextNd != pEndTextNd )
     {
@@ -187,7 +187,7 @@ SwUndoDelete::SwUndoDelete(
         pHistory->Add( pSttTextNd->GetTextColl(),pStt->nNode.GetIndex(), ND_TEXTNODE );
         pHistory->Add( pEndTextNd->GetTextColl(),pEnd->nNode.GetIndex(), ND_TEXTNODE );
 
-        if( !bJoinNext )        // Selection from bottom to top
+        if( !m_bJoinNext )        // Selection from bottom to top
         {
             // When using JoinPrev() all AUTO-PageBreak's will be copied
             // correctly. To restore them with UNDO, Auto-PageBreak of the
@@ -223,18 +223,18 @@ SwUndoDelete::SwUndoDelete(
     {
         SwNodes& rNds = pDoc->GetUndoManager().GetUndoNodes();
         SwNodes& rDocNds = pDoc->GetNodes();
-        SwNodeRange aRg( rDocNds, nSttNode - nNdDiff,
-                         rDocNds, nEndNode - nNdDiff );
+        SwNodeRange aRg( rDocNds, nSttNode - m_nNdDiff,
+                         rDocNds, nEndNode - m_nNdDiff );
         if( !bFullPara && !pEndTextNd &&
             &aRg.aEnd.GetNode() != &pDoc->GetNodes().GetEndOfContent() )
         {
             SwNode* pNode = aRg.aEnd.GetNode().StartOfSectionNode();
-            if( pNode->GetIndex() >= nSttNode - nNdDiff )
+            if( pNode->GetIndex() >= nSttNode - m_nNdDiff )
                 ++aRg.aEnd; // Deletion of a complete table
         }
         SwNode* pTmpNd;
         // Step 2: Expand selection if necessary
-        if( bJoinNext || bFullPara )
+        if( m_bJoinNext || bFullPara )
         {
             // If all content of a section will be moved into Undo, the section
             // itself should be moved completely.
@@ -243,8 +243,8 @@ SwUndoDelete::SwUndoDelete(
                 pTmpNd->StartOfSectionNode()->IsSectionNode() &&
                 pTmpNd->StartOfSectionNode()->GetIndex() >= aRg.aStart.GetIndex() ) )
                 ++aRg.aEnd;
-            nReplaceDummy = aRg.aEnd.GetIndex() + nNdDiff - nEndNode;
-            if( nReplaceDummy )
+            m_nReplaceDummy = aRg.aEnd.GetIndex() + m_nNdDiff - nEndNode;
+            if( m_nReplaceDummy )
             {   // The selection has been expanded, because
                 ++aRg.aEnd;
                 if( pEndTextNd )
@@ -252,7 +252,7 @@ SwUndoDelete::SwUndoDelete(
                     // The end text node has to leave the (expanded) selection
                     // The dummy is needed because _MoveNodes deletes empty
                     // sections
-                    ++nReplaceDummy;
+                    ++m_nReplaceDummy;
                     SwNodeRange aMvRg( *pEndTextNd, 0, *pEndTextNd, 1 );
                     SwPosition aSplitPos( *pEndTextNd );
                     ::sw::UndoGuard const ug(pDoc->GetIDocumentUndoRedo());
@@ -261,10 +261,10 @@ SwUndoDelete::SwUndoDelete(
                     --aRg.aEnd;
                 }
                 else
-                    nReplaceDummy = 0;
+                    m_nReplaceDummy = 0;
             }
         }
-        if( bBackSp || bFullPara )
+        if( m_bBackSp || bFullPara )
         {
             // See above, the selection has to be expanded if there are "nearly
             // empty" sections and a replacement dummy has to be set if needed.
@@ -274,8 +274,8 @@ SwUndoDelete::SwUndoDelete(
                 --aRg.aStart;
             if( pSttTextNd )
             {
-                nReplaceDummy = nSttNode - nNdDiff - aRg.aStart.GetIndex();
-                if( nReplaceDummy )
+                m_nReplaceDummy = nSttNode - m_nNdDiff - aRg.aStart.GetIndex();
+                if( m_nReplaceDummy )
                 {
                     SwNodeRange aMvRg( *pSttTextNd, 0, *pSttTextNd, 1 );
                     SwPosition aSplitPos( *pSttTextNd );
@@ -287,7 +287,7 @@ SwUndoDelete::SwUndoDelete(
             }
         }
 
-        if( bFromTableCopy )
+        if( m_bFromTableCopy )
         {
             if( !pEndTextNd )
             {
@@ -301,22 +301,22 @@ SwUndoDelete::SwUndoDelete(
             ++aRg.aStart;
 
         // Step 3: Moving into UndoArray...
-        nNode = rNds.GetEndOfContent().GetIndex();
+        m_nNode = rNds.GetEndOfContent().GetIndex();
         rDocNds._MoveNodes( aRg, rNds, SwNodeIndex( rNds.GetEndOfContent() ));
-        pMvStt = new SwNodeIndex( rNds, nNode );
+        m_pMvStt = new SwNodeIndex( rNds, m_nNode );
         // remember difference!
-        nNode = rNds.GetEndOfContent().GetIndex() - nNode;
+        m_nNode = rNds.GetEndOfContent().GetIndex() - m_nNode;
 
         if( pSttTextNd && pEndTextNd )
         {
             //Step 4: Moving around sections
-            nSectDiff = aRg.aEnd.GetIndex() - aRg.aStart.GetIndex();
+            m_nSectDiff = aRg.aEnd.GetIndex() - aRg.aStart.GetIndex();
             // nSect is the number of sections which starts(ends) between start
             // and end node of the selection. The "loser" paragraph has to be
             // moved into the section(s) of the "winner" paragraph
-            if( nSectDiff )
+            if( m_nSectDiff )
             {
-                if( bJoinNext )
+                if( m_bJoinNext )
                 {
                     SwNodeRange aMvRg( *pEndTextNd, 0, *pEndTextNd, 1 );
                     rDocNds._MoveNodes( aMvRg, rDocNds, aRg.aStart );
@@ -328,26 +328,26 @@ SwUndoDelete::SwUndoDelete(
                 }
             }
         }
-        if( nSectDiff || nReplaceDummy )
+        if( m_nSectDiff || m_nReplaceDummy )
             lcl_MakeAutoFrms( *pDoc->GetSpzFrameFormats(),
-                bJoinNext ? pEndTextNd->GetIndex() : pSttTextNd->GetIndex() );
+                m_bJoinNext ? pEndTextNd->GetIndex() : pSttTextNd->GetIndex() );
     }
     else
-        nNode = 0;      // moved no node -> no difference at the end
+        m_nNode = 0;      // moved no node -> no difference at the end
 
     // Are there any Nodes that got deleted before that (FootNotes
     // have ContentNodes)?
     if( !pSttTextNd && !pEndTextNd )
     {
-        nNdDiff = nSttNode - rPam.GetPoint()->nNode.GetIndex() - (bFullPara ? 0 : 1);
+        m_nNdDiff = nSttNode - rPam.GetPoint()->nNode.GetIndex() - (bFullPara ? 0 : 1);
         rPam.Move( fnMoveForward, fnGoNode );
     }
     else
     {
-        nNdDiff = nSttNode;
-        if( nSectDiff && bBackSp )
-            nNdDiff += nSectDiff;
-        nNdDiff -= rPam.GetPoint()->nNode.GetIndex();
+        m_nNdDiff = nSttNode;
+        if( m_nSectDiff && m_bBackSp )
+            m_nNdDiff += m_nSectDiff;
+        m_nNdDiff -= rPam.GetPoint()->nNode.GetIndex();
     }
 
     if( !rPam.GetNode().IsContentNode() )
@@ -382,13 +382,13 @@ bool SwUndoDelete::SaveContent( const SwPosition* pStt, const SwPosition* pEnd,
 
         // delete now also the text (all attribute changes are added to
         // UNDO history)
-        pSttStr = new OUString( pSttTextNd->GetText().copy(nSttContent, nLen));
+        m_pSttStr = new OUString( pSttTextNd->GetText().copy(nSttContent, nLen));
         pSttTextNd->EraseText( pStt->nContent, nLen );
         if( pSttTextNd->GetpSwpHints() )
             pSttTextNd->GetpSwpHints()->DeRegister();
 
         // METADATA: store
-        bool emptied( !pSttStr->isEmpty() && !pSttTextNd->Len() );
+        bool emptied( !m_pSttStr->isEmpty() && !pSttTextNd->Len() );
         if (!bOneNode || emptied) // merging may overwrite xmlids...
         {
             m_pMetadataUndoStart = (emptied)
@@ -417,14 +417,14 @@ bool SwUndoDelete::SaveContent( const SwPosition* pStt, const SwPosition* pEnd,
 
         // delete now also the text (all attribute changes are added to
         // UNDO history)
-        pEndStr = new OUString( pEndTextNd->GetText().copy( 0,
+        m_pEndStr = new OUString( pEndTextNd->GetText().copy( 0,
                                     pEnd->nContent.GetIndex() ));
         pEndTextNd->EraseText( aEndIdx, pEnd->nContent.GetIndex() );
         if( pEndTextNd->GetpSwpHints() )
             pEndTextNd->GetpSwpHints()->DeRegister();
 
         // METADATA: store
-        bool emptied = !pEndStr->isEmpty() && !pEndTextNd->Len();
+        bool emptied = !m_pEndStr->isEmpty() && !pEndTextNd->Len();
 
         m_pMetadataUndoEnd = (emptied)
             ? pEndTextNd->CreateUndoForDelete()
@@ -441,11 +441,11 @@ bool SwUndoDelete::SaveContent( const SwPosition* pStt, const SwPosition* pEnd,
 bool SwUndoDelete::CanGrouping( SwDoc* pDoc, const SwPaM& rDelPam )
 {
     // Is Undo greater than one Node (that is Start and EndString)?
-    if( !pSttStr || pSttStr->isEmpty() || pEndStr )
+    if( !m_pSttStr || m_pSttStr->isEmpty() || m_pEndStr )
         return false;
 
     // only the deletion of single char's can be condensed
-    if( nSttNode != nEndNode || ( !bGroup && nSttContent+1 != nEndContent ))
+    if( nSttNode != nEndNode || ( !m_bGroup && nSttContent+1 != nEndContent ))
         return false;
 
     const SwPosition *pStt = rDelPam.Start(),
@@ -462,13 +462,13 @@ bool SwUndoDelete::CanGrouping( SwDoc* pDoc, const SwPaM& rDelPam )
     // be constructed differently!
     if( pEnd->nContent == nSttContent )
     {
-        if( bGroup && !bBackSp ) return false;
-        bBackSp = true;
+        if( m_bGroup && !m_bBackSp ) return false;
+        m_bBackSp = true;
     }
     else if( pStt->nContent == nSttContent )
     {
-        if( bGroup && bBackSp ) return false;
-        bBackSp = false;
+        if( m_bGroup && m_bBackSp ) return false;
+        m_bBackSp = false;
     }
     else
         return false;
@@ -477,21 +477,21 @@ bool SwUndoDelete::CanGrouping( SwDoc* pDoc, const SwPaM& rDelPam )
     SwTextNode * pDelTextNd = pStt->nNode.GetNode().GetTextNode();
     if( !pDelTextNd ) return false;
 
-    sal_Int32 nUChrPos = bBackSp ? 0 : pSttStr->getLength()-1;
+    sal_Int32 nUChrPos = m_bBackSp ? 0 : m_pSttStr->getLength()-1;
     sal_Unicode cDelChar = pDelTextNd->GetText()[ pStt->nContent.GetIndex() ];
     CharClass& rCC = GetAppCharClass();
     if( ( CH_TXTATR_BREAKWORD == cDelChar || CH_TXTATR_INWORD == cDelChar ) ||
         rCC.isLetterNumeric( OUString( cDelChar ), 0 ) !=
-        rCC.isLetterNumeric( *pSttStr, nUChrPos ) )
+        rCC.isLetterNumeric( *m_pSttStr, nUChrPos ) )
         return false;
 
     {
         SwRedlineSaveDatas aTmpSav;
         const bool bSaved = FillSaveData( rDelPam, aTmpSav, false );
 
-        bool bOk = ( !pRedlSaveData && !bSaved ) ||
-                   ( pRedlSaveData && bSaved &&
-                SwUndo::CanRedlineGroup( *pRedlSaveData, aTmpSav, bBackSp ));
+        bool bOk = ( !m_pRedlSaveData && !bSaved ) ||
+                   ( m_pRedlSaveData && bSaved &&
+                SwUndo::CanRedlineGroup( *m_pRedlSaveData, aTmpSav, m_bBackSp ));
         // aTmpSav.DeleteAndDestroyAll();
         if( !bOk )
             return false;
@@ -500,32 +500,32 @@ bool SwUndoDelete::CanGrouping( SwDoc* pDoc, const SwPaM& rDelPam )
     }
 
     // Both 'deletes' can be consolidated, so 'move' the related character
-    if( bBackSp )
+    if( m_bBackSp )
         nSttContent--;    // BackSpace: add char to array!
     else
     {
         nEndContent++;    // Delete: attach char at the end
         nUChrPos++;
     }
-    (*pSttStr) = pSttStr->replaceAt( nUChrPos, 0, OUString(cDelChar) );
+    (*m_pSttStr) = m_pSttStr->replaceAt( nUChrPos, 0, OUString(cDelChar) );
     pDelTextNd->EraseText( pStt->nContent, 1 );
 
-    bGroup = true;
+    m_bGroup = true;
     return true;
 }
 
 SwUndoDelete::~SwUndoDelete()
 {
-    delete pSttStr;
-    delete pEndStr;
-    if( pMvStt )        // Delete also the selection from UndoNodes array
+    delete m_pSttStr;
+    delete m_pEndStr;
+    if( m_pMvStt )        // Delete also the selection from UndoNodes array
     {
         // Insert saves content in IconSection
-        pMvStt->GetNode().GetNodes().Delete( *pMvStt, nNode );
-        delete pMvStt;
+        m_pMvStt->GetNode().GetNodes().Delete( *m_pMvStt, m_nNode );
+        delete m_pMvStt;
     }
-    delete pRedlData;
-    delete pRedlSaveData;
+    delete m_pRedlData;
+    delete m_pRedlSaveData;
 }
 
 static SwRewriter lcl_RewriterFromHistory(SwHistory & rHistory)
@@ -661,14 +661,14 @@ SwRewriter SwUndoDelete::GetRewriter() const
 {
     SwRewriter aResult;
 
-    if (nNode != 0)
+    if (m_nNode != 0)
     {
-        if (!sTableName.isEmpty())
+        if (!m_sTableName.isEmpty())
         {
 
             SwRewriter aRewriter;
             aRewriter.AddRule(UndoArg1, SW_RESSTR(STR_START_QUOTE));
-            aRewriter.AddRule(UndoArg2, sTableName);
+            aRewriter.AddRule(UndoArg2, m_sTableName);
             aRewriter.AddRule(UndoArg3, SW_RESSTR(STR_END_QUOTE));
 
             OUString sTmp = aRewriter.Apply(SW_RES(STR_TABLE_NAME));
@@ -681,18 +681,18 @@ SwRewriter SwUndoDelete::GetRewriter() const
     {
         OUString aStr;
 
-        if (pSttStr != nullptr && pEndStr != nullptr && pSttStr->isEmpty() &&
-            pEndStr->isEmpty())
+        if (m_pSttStr != nullptr && m_pEndStr != nullptr && m_pSttStr->isEmpty() &&
+            m_pEndStr->isEmpty())
         {
             aStr = SW_RESSTR(STR_PARAGRAPH_UNDO);
         }
         else
         {
             OUString * pStr = nullptr;
-            if (pSttStr != nullptr)
-                pStr = pSttStr;
-            else if (pEndStr != nullptr)
-                pStr = pEndStr;
+            if (m_pSttStr != nullptr)
+                pStr = m_pSttStr;
+            else if (m_pEndStr != nullptr)
+                pStr = m_pEndStr;
 
             if (pStr != nullptr)
             {
@@ -747,17 +747,17 @@ void SwUndoDelete::UndoImpl(::sw::UndoRedoContext & rContext)
 {
     SwDoc& rDoc = rContext.GetDoc();
 
-    sal_uLong nCalcStt = nSttNode - nNdDiff;
+    sal_uLong nCalcStt = nSttNode - m_nNdDiff;
 
-    if( nSectDiff && bBackSp )
-        nCalcStt += nSectDiff;
+    if( m_nSectDiff && m_bBackSp )
+        nCalcStt += m_nSectDiff;
 
     SwNodeIndex aIdx(rDoc.GetNodes(), nCalcStt);
     SwNode* pInsNd = &aIdx.GetNode();
 
     {   // code block so that SwPosition is detached when deleting a Node
         SwPosition aPos( aIdx );
-        if( !bDelFullPara )
+        if( !m_bDelFullPara )
         {
             if( pInsNd->IsTableNode() )
             {
@@ -771,16 +771,16 @@ void SwUndoDelete::UndoImpl(::sw::UndoRedoContext & rContext)
             {
                 if( pInsNd->IsContentNode() )
                     aPos.nContent.Assign( static_cast<SwContentNode*>(pInsNd), nSttContent );
-                if( !bTableDelLastNd )
+                if( !m_bTableDelLastNd )
                     pInsNd = nullptr;         // do not delete Node!
             }
         }
         else
             pInsNd = nullptr;         // do not delete Node!
 
-        bool bNodeMove = 0 != nNode;
+        bool bNodeMove = 0 != m_nNode;
 
-        if( pEndStr )
+        if( m_pEndStr )
         {
             // discard attributes since they all saved!
             SwTextNode* pTextNd = aPos.nNode.GetNode().GetTextNode();
@@ -791,7 +791,7 @@ void SwUndoDelete::UndoImpl(::sw::UndoRedoContext & rContext)
             if( pTextNd && pTextNd->GetpSwpHints() )
                 pTextNd->ClearSwpHintsArr( true );
 
-            if( pSttStr && !bFromTableCopy )
+            if( m_pSttStr && !m_bFromTableCopy )
             {
                 sal_uLong nOldIdx = aPos.nNode.GetIndex();
                 rDoc.getIDocumentContentOperations().SplitNode( aPos, false );
@@ -799,21 +799,21 @@ void SwUndoDelete::UndoImpl(::sw::UndoRedoContext & rContext)
                 // paragraph, but the pHistory of the fly frame formats relies
                 // on anchoring at the start of the selection
                 // => selection backwards needs a correction.
-                if( bBackSp )
+                if( m_bBackSp )
                     lcl_ReAnchorAtContentFlyFrames(*rDoc.GetSpzFrameFormats(), aPos, nOldIdx);
                 pTextNd = aPos.nNode.GetNode().GetTextNode();
             }
             if( pTextNd )
             {
-                OUString const ins( pTextNd->InsertText(*pEndStr, aPos.nContent,
+                OUString const ins( pTextNd->InsertText(*m_pEndStr, aPos.nContent,
                         SwInsertFlags::NOHINTEXPAND) );
-                assert(ins.getLength() == pEndStr->getLength()); // must succeed
+                assert(ins.getLength() == m_pEndStr->getLength()); // must succeed
                 (void) ins;
                 // METADATA: restore
                 pTextNd->RestoreMetadata(m_pMetadataUndoEnd);
             }
         }
-        else if( pSttStr && bNodeMove )
+        else if( m_pSttStr && bNodeMove )
         {
             SwTextNode * pNd = aPos.nNode.GetNode().GetTextNode();
             if( pNd )
@@ -822,7 +822,7 @@ void SwUndoDelete::UndoImpl(::sw::UndoRedoContext & rContext)
                 {
                     sal_uLong nOldIdx = aPos.nNode.GetIndex();
                     rDoc.getIDocumentContentOperations().SplitNode( aPos, false );
-                    if( bBackSp )
+                    if( m_bBackSp )
                         lcl_ReAnchorAtContentFlyFrames(*rDoc.GetSpzFrameFormats(), aPos, nOldIdx);
                 }
                 else
@@ -830,24 +830,24 @@ void SwUndoDelete::UndoImpl(::sw::UndoRedoContext & rContext)
             }
         }
         SwNode* pMovedNode = nullptr;
-        if( nSectDiff )
+        if( m_nSectDiff )
         {
             sal_uLong nMoveIndex = aPos.nNode.GetIndex();
             int nDiff = 0;
-            if( bJoinNext )
+            if( m_bJoinNext )
             {
-                nMoveIndex += nSectDiff + 1;
+                nMoveIndex += m_nSectDiff + 1;
                 pMovedNode = &aPos.nNode.GetNode();
             }
             else
             {
-                nMoveIndex -= nSectDiff + 1;
+                nMoveIndex -= m_nSectDiff + 1;
                 ++nDiff;
             }
             SwNodeIndex aMvIdx(rDoc.GetNodes(), nMoveIndex);
             SwNodeRange aRg( aPos.nNode, 0 - nDiff, aPos.nNode, 1 - nDiff );
             --aPos.nNode;
-            if( !bJoinNext )
+            if( !m_bJoinNext )
                 pMovedNode = &aPos.nNode.GetNode();
             rDoc.GetNodes()._MoveNodes(aRg, rDoc.GetNodes(), aMvIdx);
             ++aPos.nNode;
@@ -855,22 +855,22 @@ void SwUndoDelete::UndoImpl(::sw::UndoRedoContext & rContext)
 
         if( bNodeMove )
         {
-            SwNodeRange aRange( *pMvStt, 0, *pMvStt, nNode );
+            SwNodeRange aRange( *m_pMvStt, 0, *m_pMvStt, m_nNode );
             SwNodeIndex aCopyIndex( aPos.nNode, -1 );
             rDoc.GetUndoManager().GetUndoNodes()._Copy( aRange, aPos.nNode );
 
-            if( nReplaceDummy )
+            if( m_nReplaceDummy )
             {
                 sal_uLong nMoveIndex;
-                if( bJoinNext )
+                if( m_bJoinNext )
                 {
-                    nMoveIndex = nEndNode - nNdDiff;
-                    aPos.nNode = nMoveIndex + nReplaceDummy;
+                    nMoveIndex = nEndNode - m_nNdDiff;
+                    aPos.nNode = nMoveIndex + m_nReplaceDummy;
                 }
                 else
                 {
                     aPos = SwPosition( aCopyIndex );
-                    nMoveIndex = aPos.nNode.GetIndex() + nReplaceDummy + 1;
+                    nMoveIndex = aPos.nNode.GetIndex() + m_nReplaceDummy + 1;
                 }
                 SwNodeIndex aMvIdx(rDoc.GetNodes(), nMoveIndex);
                 SwNodeRange aRg( aPos.nNode, 0, aPos.nNode, 1 );
@@ -883,15 +883,15 @@ void SwUndoDelete::UndoImpl(::sw::UndoRedoContext & rContext)
         if( pMovedNode )
             lcl_MakeAutoFrms(*rDoc.GetSpzFrameFormats(), pMovedNode->GetIndex());
 
-        if( pSttStr )
+        if( m_pSttStr )
         {
-            aPos.nNode = nSttNode - nNdDiff + ( bJoinNext ? 0 : nReplaceDummy );
+            aPos.nNode = nSttNode - m_nNdDiff + ( m_bJoinNext ? 0 : m_nReplaceDummy );
             SwTextNode * pTextNd = aPos.nNode.GetNode().GetTextNode();
             // If more than a single Node got deleted, also all "Node"
             // attributes were saved
             if (pTextNd != nullptr)
             {
-                if( pTextNd->HasSwAttrSet() && bNodeMove && !pEndStr )
+                if( pTextNd->HasSwAttrSet() && bNodeMove && !m_pEndStr )
                     pTextNd->ResetAllAttr();
 
                 if( pTextNd->GetpSwpHints() )
@@ -900,9 +900,9 @@ void SwUndoDelete::UndoImpl(::sw::UndoRedoContext & rContext)
                 // SectionNode mode and selection from top to bottom:
                 //  -> in StartNode is still the rest of the Join => delete
                 aPos.nContent.Assign( pTextNd, nSttContent );
-                OUString const ins( pTextNd->InsertText(*pSttStr, aPos.nContent,
+                OUString const ins( pTextNd->InsertText(*m_pSttStr, aPos.nContent,
                         SwInsertFlags::NOHINTEXPAND) );
-                assert(ins.getLength() == pSttStr->getLength()); // must succeed
+                assert(ins.getLength() == m_pSttStr->getLength()); // must succeed
                 (void) ins;
                 // METADATA: restore
                 pTextNd->RestoreMetadata(m_pMetadataUndoStart);
@@ -911,15 +911,15 @@ void SwUndoDelete::UndoImpl(::sw::UndoRedoContext & rContext)
 
         if( pHistory )
         {
-            pHistory->TmpRollback(&rDoc, nSetPos, false);
-            if( nSetPos )       // there were Footnodes/FlyFrames
+            pHistory->TmpRollback(&rDoc, m_nSetPos, false);
+            if( m_nSetPos )       // there were Footnodes/FlyFrames
             {
                 // are there others than these ones?
-                if( nSetPos < pHistory->Count() )
+                if( m_nSetPos < pHistory->Count() )
                 {
                     // if so save the attributes of the others
                     SwHistory aHstr;
-                    aHstr.Move( 0, pHistory, nSetPos );
+                    aHstr.Move( 0, pHistory, m_nSetPos );
                     pHistory->Rollback(&rDoc);
                     pHistory->Move( 0, &aHstr );
                 }
@@ -931,10 +931,10 @@ void SwUndoDelete::UndoImpl(::sw::UndoRedoContext & rContext)
             }
         }
 
-        if( bResetPgDesc || bResetPgBrk )
+        if( m_bResetPgDesc || m_bResetPgBrk )
         {
-            sal_uInt16 nStt = static_cast<sal_uInt16>( bResetPgDesc ? RES_PAGEDESC : RES_BREAK );
-            sal_uInt16 nEnd = static_cast<sal_uInt16>( bResetPgBrk ? RES_BREAK : RES_PAGEDESC );
+            sal_uInt16 nStt = static_cast<sal_uInt16>( m_bResetPgDesc ? RES_PAGEDESC : RES_BREAK );
+            sal_uInt16 nEnd = static_cast<sal_uInt16>( m_bResetPgBrk ? RES_BREAK : RES_PAGEDESC );
 
             SwNode* pNode = rDoc.GetNodes()[ nEndNode + 1 ];
             if( pNode->IsContentNode() )
@@ -946,8 +946,8 @@ void SwUndoDelete::UndoImpl(::sw::UndoRedoContext & rContext)
     // delete the temporarily added Node
     if( pInsNd )
         rDoc.GetNodes().Delete( aIdx );
-    if( pRedlSaveData )
-        SetSaveData(rDoc, *pRedlSaveData);
+    if( m_pRedlSaveData )
+        SetSaveData(rDoc, *m_pRedlSaveData);
 
     AddUndoRedoPaM(rContext, true);
 }
@@ -957,24 +957,24 @@ void SwUndoDelete::RedoImpl(::sw::UndoRedoContext & rContext)
     SwPaM & rPam = AddUndoRedoPaM(rContext);
     SwDoc& rDoc = *rPam.GetDoc();
 
-    if( pRedlSaveData )
+    if( m_pRedlSaveData )
     {
-        const bool bSuccess = FillSaveData(rPam, *pRedlSaveData);
+        const bool bSuccess = FillSaveData(rPam, *m_pRedlSaveData);
         OSL_ENSURE(bSuccess,
             "SwUndoDelete::Redo: used to have redline data, but now none?");
         if (!bSuccess)
         {
-            delete pRedlSaveData, pRedlSaveData = nullptr;
+            delete m_pRedlSaveData, m_pRedlSaveData = nullptr;
         }
     }
 
-    if( !bDelFullPara )
+    if( !m_bDelFullPara )
     {
         SwUndRng aTmpRng( rPam );
         RemoveIdxFromRange( rPam, false );
         aTmpRng.SetPaM( rPam );
 
-        if( !bJoinNext )           // then restore selection from bottom to top
+        if( !m_bJoinNext )           // then restore selection from bottom to top
             rPam.Exchange();
     }
 
@@ -984,7 +984,7 @@ void SwUndoDelete::RedoImpl(::sw::UndoRedoContext & rContext)
         SwHistory aHstr;
         aHstr.Move( 0, pHistory );
 
-        if( bDelFullPara )
+        if( m_bDelFullPara )
         {
             OSL_ENSURE( rPam.HasMark(), "PaM without Mark" );
             DelContentIndex( *rPam.GetMark(), *rPam.GetPoint(),
@@ -994,13 +994,13 @@ void SwUndoDelete::RedoImpl(::sw::UndoRedoContext & rContext)
         }
         else
             DelContentIndex( *rPam.GetMark(), *rPam.GetPoint() );
-        nSetPos = pHistory ? pHistory->Count() : 0;
+        m_nSetPos = pHistory ? pHistory->Count() : 0;
 
-        pHistory->Move( nSetPos, &aHstr );
+        pHistory->Move( m_nSetPos, &aHstr );
     }
     else
     {
-        if( bDelFullPara )
+        if( m_bDelFullPara )
         {
             OSL_ENSURE( rPam.HasMark(), "PaM without Mark" );
             DelContentIndex( *rPam.GetMark(), *rPam.GetPoint(),
@@ -1010,18 +1010,18 @@ void SwUndoDelete::RedoImpl(::sw::UndoRedoContext & rContext)
         }
         else
             DelContentIndex( *rPam.GetMark(), *rPam.GetPoint() );
-        nSetPos = pHistory ? pHistory->Count() : 0;
+        m_nSetPos = pHistory ? pHistory->Count() : 0;
     }
 
-    if( !pSttStr && !pEndStr )
+    if( !m_pSttStr && !m_pEndStr )
     {
-        SwNodeIndex aSttIdx = ( bDelFullPara || bJoinNext )
+        SwNodeIndex aSttIdx = ( m_bDelFullPara || m_bJoinNext )
                                     ? rPam.GetMark()->nNode
                                     : rPam.GetPoint()->nNode;
         SwTableNode* pTableNd = aSttIdx.GetNode().GetTableNode();
         if( pTableNd )
         {
-            if( bTableDelLastNd )
+            if( m_bTableDelLastNd )
             {
                 // than add again a Node at the end
                 const SwNodeIndex aTmpIdx( *pTableNd->EndOfSectionNode(), 1 );
@@ -1062,7 +1062,7 @@ void SwUndoDelete::RedoImpl(::sw::UndoRedoContext & rContext)
 
         rDoc.GetNodes().Delete( aSttIdx, nEndNode - nSttNode );
     }
-    else if( bDelFullPara )
+    else if( m_bDelFullPara )
     {
         // The Pam was incremented by one at Point (== end) to provide space
         // for UNDO. This now needs to be reverted!
@@ -1090,7 +1090,7 @@ void SwUndoDelete::RepeatImpl(::sw::RepeatContext & rContext)
         rPam.SetMark();
         rPam.Move( fnMoveForward, fnGoContent );
     }
-    if( bDelFullPara )
+    if( m_bDelFullPara )
         rDoc.getIDocumentContentOperations().DelFullPara( rPam );
     else
         rDoc.getIDocumentContentOperations().DeleteAndJoin( rPam );
@@ -1099,7 +1099,7 @@ void SwUndoDelete::RepeatImpl(::sw::RepeatContext & rContext)
 
 void SwUndoDelete::SetTableName(const OUString & rName)
 {
-    sTableName = rName;
+    m_sTableName = rName;
 }
 
 /* vim:set shiftwidth=4 softtabstop=4 expandtab: */


More information about the Libreoffice-commits mailing list