diff options
author | Miklos Vajna <vmiklos@collabora.co.uk> | 2018-09-10 09:06:15 +0200 |
---|---|---|
committer | Miklos Vajna <vmiklos@collabora.co.uk> | 2018-09-10 12:34:44 +0200 |
commit | a909d67f476945ec51d597dcd281a6f2a2b0b9b8 (patch) | |
tree | d6d6beb58759cbdc568a0eec2bc322dc1f3bd512 | |
parent | d1afb4c4218ae0e41b790b0c0e19a1df86ca9b5f (diff) |
sw: prefix members of SwRedlineData
Change-Id: Idf21b7f3330fdc4ecca6245494a788a33efee2a7
Reviewed-on: https://gerrit.libreoffice.org/60236
Reviewed-by: Miklos Vajna <vmiklos@collabora.co.uk>
Tested-by: Jenkins
-rw-r--r-- | sw/inc/redline.hxx | 52 | ||||
-rw-r--r-- | sw/source/core/doc/docredln.cxx | 88 |
2 files changed, 70 insertions, 70 deletions
diff --git a/sw/inc/redline.hxx b/sw/inc/redline.hxx index cde78e743b0b..c307792d80b0 100644 --- a/sw/inc/redline.hxx +++ b/sw/inc/redline.hxx @@ -106,14 +106,14 @@ public: class SW_DLLPUBLIC SwRedlineData { friend class SwRangeRedline; - SwRedlineData* pNext; // Points to other data. - SwRedlineExtraData* pExtraData; + SwRedlineData* m_pNext; // Points to other data. + SwRedlineExtraData* m_pExtraData; - OUString sComment; - DateTime aStamp; - RedlineType_t eType; - std::size_t nAuthor; - sal_uInt16 nSeqNo; + OUString m_sComment; + DateTime m_aStamp; + RedlineType_t m_eType; + std::size_t m_nAuthor; + sal_uInt16 m_nSeqNo; public: SwRedlineData( RedlineType_t eT, std::size_t nAut ); @@ -127,43 +127,43 @@ public: bool operator==( const SwRedlineData& rCmp ) const { - return nAuthor == rCmp.nAuthor && - eType == rCmp.eType && - sComment == rCmp.sComment && - (( !pNext && !rCmp.pNext ) || - ( pNext && rCmp.pNext && *pNext == *rCmp.pNext )) && - (( !pExtraData && !rCmp.pExtraData ) || - ( pExtraData && rCmp.pExtraData && - *pExtraData == *rCmp.pExtraData )); + return m_nAuthor == rCmp.m_nAuthor && + m_eType == rCmp.m_eType && + m_sComment == rCmp.m_sComment && + (( !m_pNext && !rCmp.m_pNext ) || + ( m_pNext && rCmp.m_pNext && *m_pNext == *rCmp.m_pNext )) && + (( !m_pExtraData && !rCmp.m_pExtraData ) || + ( m_pExtraData && rCmp.m_pExtraData && + *m_pExtraData == *rCmp.m_pExtraData )); } bool operator!=( const SwRedlineData& rCmp ) const { return !operator==( rCmp ); } RedlineType_t GetType() const - { return static_cast<RedlineType_t>(eType & nsRedlineType_t::REDLINE_NO_FLAG_MASK); } + { return static_cast<RedlineType_t>(m_eType & nsRedlineType_t::REDLINE_NO_FLAG_MASK); } - std::size_t GetAuthor() const { return nAuthor; } - const OUString& GetComment() const { return sComment; } - const DateTime& GetTimeStamp() const { return aStamp; } - const SwRedlineData* Next() const{ return pNext; } + std::size_t GetAuthor() const { return m_nAuthor; } + const OUString& GetComment() const { return m_sComment; } + const DateTime& GetTimeStamp() const { return m_aStamp; } + const SwRedlineData* Next() const{ return m_pNext; } - void SetComment( const OUString& rS ) { sComment = rS; } - void SetTimeStamp( const DateTime& rDT ) { aStamp = rDT; } + void SetComment( const OUString& rS ) { m_sComment = rS; } + void SetTimeStamp( const DateTime& rDT ) { m_aStamp = rDT; } void SetAutoFormatFlag() - { eType = static_cast<RedlineType_t>(eType | nsRedlineType_t::REDLINE_FORM_AUTOFMT); } + { m_eType = static_cast<RedlineType_t>(m_eType | nsRedlineType_t::REDLINE_FORM_AUTOFMT); } bool CanCombine( const SwRedlineData& rCmp ) const; // ExtraData gets copied, the pointer is therefore not taken over by // the RedlineObject void SetExtraData( const SwRedlineExtraData* pData ); - const SwRedlineExtraData* GetExtraData() const { return pExtraData; } + const SwRedlineExtraData* GetExtraData() const { return m_pExtraData; } // For UI-side pooling of Redline-actions. // At the moment only used for Autoformat with Redline. // Value != 0 means there can be others! - sal_uInt16 GetSeqNo() const { return nSeqNo; } - void SetSeqNo( sal_uInt16 nNo ) { nSeqNo = nNo; } + sal_uInt16 GetSeqNo() const { return m_nSeqNo; } + void SetSeqNo( sal_uInt16 nNo ) { m_nSeqNo = nNo; } OUString GetDescr() const; }; diff --git a/sw/source/core/doc/docredln.cxx b/sw/source/core/doc/docredln.cxx index 0947b6a0cbfd..5c36d8be9b1c 100644 --- a/sw/source/core/doc/docredln.cxx +++ b/sw/source/core/doc/docredln.cxx @@ -924,38 +924,38 @@ bool SwRedlineExtraData_FormattingChanges::operator == ( const SwRedlineExtraDat } SwRedlineData::SwRedlineData( RedlineType_t eT, std::size_t nAut ) - : pNext( nullptr ), pExtraData( nullptr ), - aStamp( DateTime::SYSTEM ), - eType( eT ), nAuthor( nAut ), nSeqNo( 0 ) + : m_pNext( nullptr ), m_pExtraData( nullptr ), + m_aStamp( DateTime::SYSTEM ), + m_eType( eT ), m_nAuthor( nAut ), m_nSeqNo( 0 ) { - aStamp.SetNanoSec( 0 ); + m_aStamp.SetNanoSec( 0 ); } SwRedlineData::SwRedlineData( const SwRedlineData& rCpy, bool bCpyNext ) - : pNext( ( bCpyNext && rCpy.pNext ) ? new SwRedlineData( *rCpy.pNext ) : nullptr ) - , pExtraData( rCpy.pExtraData ? rCpy.pExtraData->CreateNew() : nullptr ) - , sComment( rCpy.sComment ) - , aStamp( rCpy.aStamp ) - , eType( rCpy.eType ) - , nAuthor( rCpy.nAuthor ) - , nSeqNo( rCpy.nSeqNo ) + : m_pNext( ( bCpyNext && rCpy.m_pNext ) ? new SwRedlineData( *rCpy.m_pNext ) : nullptr ) + , m_pExtraData( rCpy.m_pExtraData ? rCpy.m_pExtraData->CreateNew() : nullptr ) + , m_sComment( rCpy.m_sComment ) + , m_aStamp( rCpy.m_aStamp ) + , m_eType( rCpy.m_eType ) + , m_nAuthor( rCpy.m_nAuthor ) + , m_nSeqNo( rCpy.m_nSeqNo ) { } // For sw3io: We now own pNext! SwRedlineData::SwRedlineData(RedlineType_t eT, std::size_t nAut, const DateTime& rDT, const OUString& rCmnt, SwRedlineData *pNxt) - : pNext(pNxt), pExtraData(nullptr), sComment(rCmnt), aStamp(rDT), - eType(eT), nAuthor(nAut), nSeqNo(0) + : m_pNext(pNxt), m_pExtraData(nullptr), m_sComment(rCmnt), m_aStamp(rDT), + m_eType(eT), m_nAuthor(nAut), m_nSeqNo(0) { } SwRedlineData::~SwRedlineData() { - delete pExtraData; - delete pNext; + delete m_pExtraData; + delete m_pNext; } bool SwRedlineData::CanCombine(const SwRedlineData& rCmp) const @@ -964,29 +964,29 @@ bool SwRedlineData::CanCombine(const SwRedlineData& rCmp) const aTime.SetSec(0); DateTime aCompareTime = rCmp.GetTimeStamp(); aCompareTime.SetSec(0); - return nAuthor == rCmp.nAuthor && - eType == rCmp.eType && - sComment == rCmp.sComment && + return m_nAuthor == rCmp.m_nAuthor && + m_eType == rCmp.m_eType && + m_sComment == rCmp.m_sComment && aTime == aCompareTime && - (( !pNext && !rCmp.pNext ) || - ( pNext && rCmp.pNext && - pNext->CanCombine( *rCmp.pNext ))) && - (( !pExtraData && !rCmp.pExtraData ) || - ( pExtraData && rCmp.pExtraData && - *pExtraData == *rCmp.pExtraData )); + (( !m_pNext && !rCmp.m_pNext ) || + ( m_pNext && rCmp.m_pNext && + m_pNext->CanCombine( *rCmp.m_pNext ))) && + (( !m_pExtraData && !rCmp.m_pExtraData ) || + ( m_pExtraData && rCmp.m_pExtraData && + *m_pExtraData == *rCmp.m_pExtraData )); } /// ExtraData is copied. The Pointer's ownership is thus NOT transferred /// to the Redline Object! void SwRedlineData::SetExtraData( const SwRedlineExtraData* pData ) { - delete pExtraData; + delete m_pExtraData; // Check if there is data - and if so - delete it if( pData ) - pExtraData = pData->CreateNew(); + m_pExtraData = pData->CreateNew(); else - pExtraData = nullptr; + m_pExtraData = nullptr; } static const char* STR_REDLINE_ARY[] = @@ -1217,10 +1217,10 @@ void SwRangeRedline::ShowOriginal(sal_uInt16 nLoop, size_t nMyPos) ::sw::UndoGuard const undoGuard(pDoc->GetIDocumentUndoRedo()); // Determine the Type, it's the first on Stack - for( pCur = m_pRedlineData; pCur->pNext; ) - pCur = pCur->pNext; + for( pCur = m_pRedlineData; pCur->m_pNext; ) + pCur = pCur->m_pNext; - switch( pCur->eType ) + switch( pCur->m_eType ) { case nsRedlineType_t::REDLINE_INSERT: // Content has been inserted m_bIsVisible = false; @@ -1710,23 +1710,23 @@ void SwRangeRedline::PushData( const SwRangeRedline& rRedl, bool bOwnAsNext ) SwRedlineData* pNew = new SwRedlineData( *rRedl.m_pRedlineData, false ); if( bOwnAsNext ) { - pNew->pNext = m_pRedlineData; + pNew->m_pNext = m_pRedlineData; m_pRedlineData = pNew; } else { - pNew->pNext = m_pRedlineData->pNext; - m_pRedlineData->pNext = pNew; + pNew->m_pNext = m_pRedlineData->m_pNext; + m_pRedlineData->m_pNext = pNew; } } bool SwRangeRedline::PopData() { - if( !m_pRedlineData->pNext ) + if( !m_pRedlineData->m_pNext ) return false; SwRedlineData* pCur = m_pRedlineData; - m_pRedlineData = pCur->pNext; - pCur->pNext = nullptr; + m_pRedlineData = pCur->m_pNext; + pCur->m_pNext = nullptr; delete pCur; return true; } @@ -1734,34 +1734,34 @@ bool SwRangeRedline::PopData() sal_uInt16 SwRangeRedline::GetStackCount() const { sal_uInt16 nRet = 1; - for( SwRedlineData* pCur = m_pRedlineData; pCur->pNext; pCur = pCur->pNext ) + for( SwRedlineData* pCur = m_pRedlineData; pCur->m_pNext; pCur = pCur->m_pNext ) ++nRet; return nRet; } std::size_t SwRangeRedline::GetAuthor( sal_uInt16 nPos ) const { - return GetRedlineData(nPos).nAuthor; + return GetRedlineData(nPos).m_nAuthor; } OUString const & SwRangeRedline::GetAuthorString( sal_uInt16 nPos ) const { - return SW_MOD()->GetRedlineAuthor(GetRedlineData(nPos).nAuthor); + return SW_MOD()->GetRedlineAuthor(GetRedlineData(nPos).m_nAuthor); } const DateTime& SwRangeRedline::GetTimeStamp( sal_uInt16 nPos ) const { - return GetRedlineData(nPos).aStamp; + return GetRedlineData(nPos).m_aStamp; } RedlineType_t SwRangeRedline::GetRealType( sal_uInt16 nPos ) const { - return GetRedlineData(nPos).eType; + return GetRedlineData(nPos).m_eType; } const OUString& SwRangeRedline::GetComment( sal_uInt16 nPos ) const { - return GetRedlineData(nPos).sComment; + return GetRedlineData(nPos).m_sComment; } bool SwRangeRedline::operator<( const SwRangeRedline& rCmp ) const @@ -1778,9 +1778,9 @@ const SwRedlineData & SwRangeRedline::GetRedlineData(const sal_uInt16 nPos) cons sal_uInt16 nP = nPos; - while (nP > 0 && nullptr != pCur->pNext) + while (nP > 0 && nullptr != pCur->m_pNext) { - pCur = pCur->pNext; + pCur = pCur->m_pNext; nP--; } |