[Libreoffice-commits] core.git: 2 commits - sw/source
Michael Stahl
Michael.Stahl at cib.de
Wed May 16 08:36:00 UTC 2018
sw/source/core/text/atrhndl.hxx | 37 ++++-----
sw/source/core/text/atrstck.cxx | 159 +++++++++++++++++++---------------------
2 files changed, 98 insertions(+), 98 deletions(-)
New commits:
commit 71f89889b5bf6273067acb4dad925a5c5ecb3239
Author: Michael Stahl <Michael.Stahl at cib.de>
Date: Tue May 15 14:13:12 2018 +0200
sw: prefix members in SwAttrHandler
Change-Id: I46a19e86afdd369c76a34711c7ef11a226309e57
Reviewed-on: https://gerrit.libreoffice.org/54370
Tested-by: Jenkins <ci at libreoffice.org>
Reviewed-by: Michael Stahl <Michael.Stahl at cib.de>
diff --git a/sw/source/core/text/atrhndl.hxx b/sw/source/core/text/atrhndl.hxx
index 2949206d9a22..5c59de630c63 100644
--- a/sw/source/core/text/atrhndl.hxx
+++ b/sw/source/core/text/atrhndl.hxx
@@ -80,16 +80,16 @@ private:
sal_uInt16 Pos( const SwTextAttr& rAttr ) const;
};
- SwAttrStack aAttrStack[ NUM_ATTRIBUTE_STACKS ]; // stack collection
- const SfxPoolItem* pDefaultArray[ NUM_DEFAULT_VALUES ];
- const IDocumentSettingAccess* mpIDocumentSettingAccess;
- const SwViewShell* mpShell;
+ SwAttrStack m_aAttrStack[ NUM_ATTRIBUTE_STACKS ]; // stack collection
+ const SfxPoolItem* m_pDefaultArray[ NUM_DEFAULT_VALUES ];
+ const IDocumentSettingAccess* m_pIDocumentSettingAccess;
+ const SwViewShell* m_pShell;
// This is the base font for the paragraph. It is stored in order to have
// a template, if we have to restart the attribute evaluation
- std::unique_ptr<SwFont> pFnt;
+ std::unique_ptr<SwFont> m_pFnt;
- bool bVertLayout;
+ bool m_bVertLayout;
// change font according to pool item
void FontChg(const SfxPoolItem& rItem, SwFont& rFnt, bool bPush );
@@ -140,14 +140,14 @@ public:
inline void SwAttrHandler::ResetFont( SwFont& rFnt ) const
{
- OSL_ENSURE( pFnt, "ResetFont without a font" );
- if ( pFnt )
- rFnt = *pFnt;
+ OSL_ENSURE(m_pFnt, "ResetFont without a font");
+ if (m_pFnt)
+ rFnt = *m_pFnt;
};
inline const SwFont* SwAttrHandler::GetFont() const
{
- return pFnt.get();
+ return m_pFnt.get();
};
#endif
diff --git a/sw/source/core/text/atrstck.cxx b/sw/source/core/text/atrstck.cxx
index ee43d211bd79..345400cede4f 100644
--- a/sw/source/core/text/atrstck.cxx
+++ b/sw/source/core/text/atrstck.cxx
@@ -343,12 +343,11 @@ sal_uInt16 SwAttrHandler::SwAttrStack::Pos( const SwTextAttr& rAttr ) const
}
SwAttrHandler::SwAttrHandler()
- : mpIDocumentSettingAccess(nullptr)
- , mpShell(nullptr)
- , pFnt(nullptr)
- , bVertLayout(false)
+ : m_pIDocumentSettingAccess(nullptr)
+ , m_pShell(nullptr)
+ , m_bVertLayout(false)
{
- memset( pDefaultArray, 0, NUM_DEFAULT_VALUES * sizeof(SfxPoolItem*) );
+ memset( m_pDefaultArray, 0, NUM_DEFAULT_VALUES * sizeof(SfxPoolItem*) );
}
SwAttrHandler::~SwAttrHandler()
@@ -358,11 +357,11 @@ SwAttrHandler::~SwAttrHandler()
void SwAttrHandler::Init( const SwAttrSet& rAttrSet,
const IDocumentSettingAccess& rIDocumentSettingAcces )
{
- mpIDocumentSettingAccess = &rIDocumentSettingAcces;
- mpShell = nullptr;
+ m_pIDocumentSettingAccess = &rIDocumentSettingAcces;
+ m_pShell = nullptr;
for ( sal_uInt16 i = RES_CHRATR_BEGIN; i < RES_CHRATR_END; i++ )
- pDefaultArray[ StackPos[ i ] ] = &rAttrSet.Get( i );
+ m_pDefaultArray[ StackPos[ i ] ] = &rAttrSet.Get( i );
}
void SwAttrHandler::Init( const SfxPoolItem** pPoolItem, const SwAttrSet* pAS,
@@ -371,14 +370,14 @@ void SwAttrHandler::Init( const SfxPoolItem** pPoolItem, const SwAttrSet* pAS,
SwFont& rFnt, bool bVL )
{
// initialize default array
- memcpy( pDefaultArray, pPoolItem,
+ memcpy( m_pDefaultArray, pPoolItem,
NUM_DEFAULT_VALUES * sizeof(SfxPoolItem*) );
- mpIDocumentSettingAccess = &rIDocumentSettingAcces;
- mpShell = pSh;
+ m_pIDocumentSettingAccess = &rIDocumentSettingAcces;
+ m_pShell = pSh;
// do we have to apply additional paragraph attributes?
- bVertLayout = bVL;
+ m_bVertLayout = bVL;
if ( pAS && pAS->Count() )
{
@@ -390,7 +389,7 @@ void SwAttrHandler::Init( const SfxPoolItem** pPoolItem, const SwAttrSet* pAS,
nWhich = pItem->Which();
if (isCHRATR(nWhich))
{
- pDefaultArray[ StackPos[ nWhich ] ] = pItem;
+ m_pDefaultArray[ StackPos[ nWhich ] ] = pItem;
FontChg( *pItem, rFnt, true );
}
@@ -403,12 +402,12 @@ void SwAttrHandler::Init( const SfxPoolItem** pPoolItem, const SwAttrSet* pAS,
// It is possible, that Init is called more than once, e.g., in a
// SwTextFrame::FormatOnceMore situation.
- pFnt.reset( new SwFont( rFnt ) );
+ m_pFnt.reset( new SwFont(rFnt) );
}
void SwAttrHandler::Reset( )
{
- for (SwAttrStack & i : aAttrStack)
+ for (SwAttrStack & i : m_aAttrStack)
i.Reset();
}
@@ -435,7 +434,7 @@ void SwAttrHandler::PushAndChg( const SwTextAttr& rAttr, SwFont& rFnt )
{
// we let pItem change rFnt
Color aColor;
- if ( lcl_ChgHyperLinkColor( rAttr, *pItem, mpShell, &aColor ) )
+ if (lcl_ChgHyperLinkColor(rAttr, *pItem, m_pShell, &aColor))
{
SvxColorItem aItemNext( aColor, RES_CHRATR_COLOR );
FontChg( aItemNext, rFnt, true );
@@ -469,19 +468,19 @@ bool SwAttrHandler::Push( const SwTextAttr& rAttr, const SfxPoolItem& rItem )
// attributes originating from redlining have highest priority
// second priority are hyperlink attributes, which have a color replacement
- const SwTextAttr* pTopAttr = aAttrStack[ nStack ].Top();
+ const SwTextAttr* pTopAttr = m_aAttrStack[ nStack ].Top();
if ( !pTopAttr
|| rAttr.IsPriorityAttr()
|| ( !pTopAttr->IsPriorityAttr()
- && !lcl_ChgHyperLinkColor( *pTopAttr, rItem, mpShell, nullptr ) ) )
+ && !lcl_ChgHyperLinkColor(*pTopAttr, rItem, m_pShell, nullptr)))
{
- aAttrStack[ nStack ].Push( rAttr );
+ m_aAttrStack[ nStack ].Push( rAttr );
return true;
}
- const sal_uInt16 nPos = aAttrStack[ nStack ].Count();
+ const sal_uInt16 nPos = m_aAttrStack[ nStack ].Count();
OSL_ENSURE( nPos, "empty stack?" );
- aAttrStack[ nStack ].Insert( rAttr, nPos - 1 );
+ m_aAttrStack[ nStack ].Insert( rAttr, nPos - 1 );
return false;
}
@@ -507,7 +506,7 @@ void SwAttrHandler::PopAndChg( const SwTextAttr& rAttr, SwFont& rFnt )
{
// we remove rAttr from the appropriate stack
const sal_uInt16 nStackPos = StackPos[ i ];
- aAttrStack[ nStackPos ].Remove( rAttr );
+ m_aAttrStack[ nStackPos ].Remove( rAttr );
// reset font according to attribute on top of stack
// or default value
ActivateTop( rFnt, i );
@@ -518,7 +517,7 @@ void SwAttrHandler::PopAndChg( const SwTextAttr& rAttr, SwFont& rFnt )
// stack and reset the font
else
{
- aAttrStack[ StackPos[ rAttr.Which() ] ].Remove( rAttr );
+ m_aAttrStack[ StackPos[ rAttr.Which() ] ].Remove( rAttr );
// reset font according to attribute on top of stack
// or default value
ActivateTop( rFnt, rAttr.Which() );
@@ -533,7 +532,7 @@ void SwAttrHandler::Pop( const SwTextAttr& rAttr )
if ( rAttr.Which() < RES_TXTATR_WITHEND_END )
{
- aAttrStack[ StackPos[ rAttr.Which() ] ].Remove( rAttr );
+ m_aAttrStack[ StackPos[ rAttr.Which() ] ].Remove( rAttr );
}
}
@@ -543,7 +542,7 @@ void SwAttrHandler::ActivateTop( SwFont& rFnt, const sal_uInt16 nAttr )
"I cannot activate this attribute, nWhich >= RES_TXTATR_WITHEND_END" );
const sal_uInt16 nStackPos = StackPos[ nAttr ];
- const SwTextAttr* pTopAt = aAttrStack[ nStackPos ].Top();
+ const SwTextAttr* pTopAt = m_aAttrStack[ nStackPos ].Top();
if ( pTopAt )
{
const SfxPoolItem* pItemNext(nullptr);
@@ -561,7 +560,7 @@ void SwAttrHandler::ActivateTop( SwFont& rFnt, const sal_uInt16 nAttr )
if (pItemNext)
{
Color aColor;
- if ( lcl_ChgHyperLinkColor( *pTopAt, *pItemNext, mpShell, &aColor ) )
+ if (lcl_ChgHyperLinkColor(*pTopAt, *pItemNext, m_pShell, &aColor))
{
SvxColorItem aItemNext( aColor, RES_CHRATR_COLOR );
FontChg( aItemNext, rFnt, false );
@@ -575,7 +574,7 @@ void SwAttrHandler::ActivateTop( SwFont& rFnt, const sal_uInt16 nAttr )
// default value has to be set, we only have default values for char attribs
else if ( nStackPos < NUM_DEFAULT_VALUES )
- FontChg( *pDefaultArray[ nStackPos ], rFnt, false );
+ FontChg( *m_pDefaultArray[ nStackPos ], rFnt, false );
else if ( RES_TXTATR_REFMARK == nAttr )
rFnt.GetRef()--;
else if ( RES_TXTATR_TOXMARK == nAttr )
@@ -590,7 +589,7 @@ void SwAttrHandler::ActivateTop( SwFont& rFnt, const sal_uInt16 nAttr )
// check, if an rotation attribute has to be applied
const sal_uInt16 nTwoLineStack = StackPos[ RES_CHRATR_TWO_LINES ];
bool bTwoLineAct = false;
- const SwTextAttr* pTwoLineAttr = aAttrStack[ nTwoLineStack ].Top();
+ const SwTextAttr* pTwoLineAttr = m_aAttrStack[ nTwoLineStack ].Top();
if ( pTwoLineAttr )
{
@@ -599,25 +598,25 @@ void SwAttrHandler::ActivateTop( SwFont& rFnt, const sal_uInt16 nAttr )
}
else
bTwoLineAct =
- static_cast<const SvxTwoLinesItem*>(pDefaultArray[ nTwoLineStack ])->GetValue();
+ static_cast<const SvxTwoLinesItem*>(m_pDefaultArray[ nTwoLineStack ])->GetValue();
if ( bTwoLineAct )
return;
// eventually, an rotate attribute has to be activated
const sal_uInt16 nRotateStack = StackPos[ RES_CHRATR_ROTATE ];
- const SwTextAttr* pRotateAttr = aAttrStack[ nRotateStack ].Top();
+ const SwTextAttr* pRotateAttr = m_aAttrStack[ nRotateStack ].Top();
if ( pRotateAttr )
{
const SfxPoolItem* pRotateItem = CharFormat::GetItem( *pRotateAttr, RES_CHRATR_ROTATE );
rFnt.SetVertical( static_cast<const SvxCharRotateItem*>(pRotateItem)->GetValue(),
- bVertLayout );
+ m_bVertLayout );
}
else
rFnt.SetVertical(
- static_cast<const SvxCharRotateItem*>(pDefaultArray[ nRotateStack ])->GetValue(),
- bVertLayout
+ static_cast<const SvxCharRotateItem*>(m_pDefaultArray[ nRotateStack ])->GetValue(),
+ m_bVertLayout
);
}
else if ( RES_TXTATR_INPUTFIELD == nAttr )
@@ -674,13 +673,13 @@ void SwAttrHandler::FontChg(const SfxPoolItem& rItem, SwFont& rFnt, bool bPush )
case RES_CHRATR_UNDERLINE :
{
const sal_uInt16 nStackPos = StackPos[ RES_CHRATR_HIDDEN ];
- const SwTextAttr* pTopAt = aAttrStack[ nStackPos ].Top();
+ const SwTextAttr* pTopAt = m_aAttrStack[ nStackPos ].Top();
const SfxPoolItem* pTmpItem = pTopAt ?
CharFormat::GetItem( *pTopAt, RES_CHRATR_HIDDEN ) :
- pDefaultArray[ nStackPos ];
+ m_pDefaultArray[ nStackPos ];
- if( (mpShell && !mpShell->GetWin()) ||
+ if ((m_pShell && !m_pShell->GetWin()) ||
(pTmpItem && !static_cast<const SvxCharHiddenItem*>(pTmpItem)->GetValue()) )
{
rFnt.SetUnderline( static_cast<const SvxUnderlineItem&>(rItem).GetLineStyle() );
@@ -722,8 +721,8 @@ void SwAttrHandler::FontChg(const SfxPoolItem& rItem, SwFont& rFnt, bool bPush )
case RES_CHRATR_AUTOKERN :
if( static_cast<const SvxAutoKernItem&>(rItem).GetValue() )
{
- rFnt.SetAutoKern( ( !mpIDocumentSettingAccess ||
- !mpIDocumentSettingAccess->get(DocumentSettingId::KERN_ASIAN_PUNCTUATION) ) ?
+ rFnt.SetAutoKern( (!m_pIDocumentSettingAccess ||
+ !m_pIDocumentSettingAccess->get(DocumentSettingId::KERN_ASIAN_PUNCTUATION)) ?
FontKerning::FontSpecific :
FontKerning::Asian );
}
@@ -789,7 +788,7 @@ void SwAttrHandler::FontChg(const SfxPoolItem& rItem, SwFont& rFnt, bool bPush )
rFnt.SetRelief( static_cast<const SvxCharReliefItem&>(rItem).GetValue() );
break;
case RES_CHRATR_HIDDEN :
- if( mpShell && mpShell->GetWin())
+ if (m_pShell && m_pShell->GetWin())
{
if ( static_cast<const SvxCharHiddenItem&>(rItem).GetValue() )
rFnt.SetUnderline( LINESTYLE_DOTTED );
@@ -804,14 +803,14 @@ void SwAttrHandler::FontChg(const SfxPoolItem& rItem, SwFont& rFnt, bool bPush )
// 2. top of two line stack ( or default attribute )is an
// deactivated two line attribute
const bool bRuby =
- 0 != aAttrStack[ StackPos[ RES_TXTATR_CJK_RUBY ] ].Count();
+ 0 != m_aAttrStack[ StackPos[ RES_TXTATR_CJK_RUBY ] ].Count();
if ( bRuby )
break;
const sal_uInt16 nTwoLineStack = StackPos[ RES_CHRATR_TWO_LINES ];
bool bTwoLineAct = false;
- const SwTextAttr* pTwoLineAttr = aAttrStack[ nTwoLineStack ].Top();
+ const SwTextAttr* pTwoLineAttr = m_aAttrStack[ nTwoLineStack ].Top();
if ( pTwoLineAttr )
{
@@ -820,25 +819,25 @@ void SwAttrHandler::FontChg(const SfxPoolItem& rItem, SwFont& rFnt, bool bPush )
}
else
bTwoLineAct =
- static_cast<const SvxTwoLinesItem*>(pDefaultArray[ nTwoLineStack ])->GetValue();
+ static_cast<const SvxTwoLinesItem*>(m_pDefaultArray[ nTwoLineStack ])->GetValue();
if ( !bTwoLineAct )
rFnt.SetVertical( static_cast<const SvxCharRotateItem&>(rItem).GetValue(),
- bVertLayout );
+ m_bVertLayout );
break;
}
case RES_CHRATR_TWO_LINES :
{
bool bRuby = 0 !=
- aAttrStack[ StackPos[ RES_TXTATR_CJK_RUBY ] ].Count();
+ m_aAttrStack[ StackPos[ RES_TXTATR_CJK_RUBY ] ].Count();
// two line is activated, if
// 1. no ruby attribute is set and
// 2. attribute is active
if ( !bRuby && static_cast<const SvxTwoLinesItem&>(rItem).GetValue() )
{
- rFnt.SetVertical( 0, bVertLayout );
+ rFnt.SetVertical( 0, m_bVertLayout );
break;
}
@@ -848,23 +847,23 @@ void SwAttrHandler::FontChg(const SfxPoolItem& rItem, SwFont& rFnt, bool bPush )
break;
const sal_uInt16 nRotateStack = StackPos[ RES_CHRATR_ROTATE ];
- const SwTextAttr* pRotateAttr = aAttrStack[ nRotateStack ].Top();
+ const SwTextAttr* pRotateAttr = m_aAttrStack[ nRotateStack ].Top();
if ( pRotateAttr )
{
const SfxPoolItem* pRotateItem = CharFormat::GetItem( *pRotateAttr, RES_CHRATR_ROTATE );
rFnt.SetVertical( static_cast<const SvxCharRotateItem*>(pRotateItem)->GetValue(),
- bVertLayout );
+ m_bVertLayout );
}
else
rFnt.SetVertical(
- static_cast<const SvxCharRotateItem*>(pDefaultArray[ nRotateStack ])->GetValue(),
- bVertLayout
+ static_cast<const SvxCharRotateItem*>(m_pDefaultArray[ nRotateStack ])->GetValue(),
+ m_bVertLayout
);
break;
}
case RES_TXTATR_CJK_RUBY :
- rFnt.SetVertical( 0, bVertLayout );
+ rFnt.SetVertical( 0, m_bVertLayout );
break;
case RES_TXTATR_REFMARK :
if ( bPush )
@@ -898,11 +897,11 @@ void SwAttrHandler::FontChg(const SfxPoolItem& rItem, SwFont& rFnt, bool bPush )
void SwAttrHandler::GetDefaultAscentAndHeight( SwViewShell const * pShell, OutputDevice const & rOut,
sal_uInt16& nAscent, sal_uInt16& nHeight ) const
{
- OSL_ENSURE( pFnt, "No font available for GetDefaultAscentAndHeight" );
+ OSL_ENSURE(m_pFnt, "No font available for GetDefaultAscentAndHeight");
- if ( pFnt )
+ if (m_pFnt)
{
- SwFont aFont( *pFnt );
+ SwFont aFont( *m_pFnt );
nHeight = aFont.GetHeight( pShell, rOut );
nAscent = aFont.GetAscent( pShell, rOut );
}
commit 750fe9560b50e96beae79affa6a9a9791fce919c
Author: Michael Stahl <Michael.Stahl at cib.de>
Date: Tue May 15 13:59:59 2018 +0200
sw: prefix members in SwAttrHandler::SwAttrStack
Change-Id: I64c4bfa709b6cd12793efba1d9773731fb66939d
Reviewed-on: https://gerrit.libreoffice.org/54369
Tested-by: Jenkins <ci at libreoffice.org>
Reviewed-by: Michael Stahl <Michael.Stahl at cib.de>
diff --git a/sw/source/core/text/atrhndl.hxx b/sw/source/core/text/atrhndl.hxx
index 23975bc71e04..2949206d9a22 100644
--- a/sw/source/core/text/atrhndl.hxx
+++ b/sw/source/core/text/atrhndl.hxx
@@ -45,22 +45,23 @@ private:
class SwAttrStack
{
private:
- SwTextAttr* pInitialArray[ INITIAL_NUM_ATTR ];
- SwTextAttr** pArray;
- sal_uInt16 nCount; // number of elements on stack
- sal_uInt16 nSize; // number of positions in Array
+ SwTextAttr* m_pInitialArray[ INITIAL_NUM_ATTR ];
+ SwTextAttr** m_pArray;
+ sal_uInt16 m_nCount; // number of elements on stack
+ sal_uInt16 m_nSize; // number of positions in Array
public:
// Ctor, Dtor
inline SwAttrStack();
~SwAttrStack() {
- if ( nSize > INITIAL_NUM_ATTR ) delete [] pArray; }
+ if (m_nSize > INITIAL_NUM_ATTR) delete [] m_pArray;
+ }
// reset stack
- void Reset() { nCount = 0; };
+ void Reset() { m_nCount = 0; };
// insert on top
- void Push( const SwTextAttr& rAttr ) { Insert( rAttr, nCount ); };
+ void Push( const SwTextAttr& rAttr ) { Insert(rAttr, m_nCount); };
// insert at specified position, take care for not inserting behind
// the value returned by Count()
void Insert( const SwTextAttr& rAttr, const sal_uInt16 nPos );
@@ -72,7 +73,7 @@ private:
const SwTextAttr* Top() const;
// number of elements on stack
- sal_uInt16 Count() const { return nCount; };
+ sal_uInt16 Count() const { return m_nCount; };
// returns position of rAttr on Stack if found, otherwise USHRT_MAX
// can be used for Remove of an attribute
diff --git a/sw/source/core/text/atrstck.cxx b/sw/source/core/text/atrstck.cxx
index a252302b743a..ee43d211bd79 100644
--- a/sw/source/core/text/atrstck.cxx
+++ b/sw/source/core/text/atrstck.cxx
@@ -265,76 +265,76 @@ static bool lcl_ChgHyperLinkColor( const SwTextAttr& rAttr,
}
inline SwAttrHandler::SwAttrStack::SwAttrStack()
- : nCount( 0 ), nSize( INITIAL_NUM_ATTR )
+ : m_nCount( 0 ), m_nSize( INITIAL_NUM_ATTR )
{
- pArray = pInitialArray;
+ m_pArray = m_pInitialArray;
}
void SwAttrHandler::SwAttrStack::Insert( const SwTextAttr& rAttr, const sal_uInt16 nPos )
{
// do we still have enough space?
- if ( nCount >= nSize )
+ if (m_nCount >= m_nSize)
{
// we are still in our initial array
- if ( INITIAL_NUM_ATTR == nSize )
+ if (INITIAL_NUM_ATTR == m_nSize)
{
- nSize += STACK_INCREMENT;
- pArray = new SwTextAttr*[ nSize ];
+ m_nSize += STACK_INCREMENT;
+ m_pArray = new SwTextAttr*[ m_nSize ];
// copy from pInitArray to new Array
- memcpy( pArray, pInitialArray,
+ memcpy( m_pArray, m_pInitialArray,
INITIAL_NUM_ATTR * sizeof(SwTextAttr*)
);
}
// we are in new memory
else
{
- nSize += STACK_INCREMENT;
- SwTextAttr** pTmpArray = new SwTextAttr*[ nSize ];
- // copy from pArray to new Array
- memcpy( pTmpArray, pArray, nCount * sizeof(SwTextAttr*) );
+ m_nSize += STACK_INCREMENT;
+ SwTextAttr** pTmpArray = new SwTextAttr*[ m_nSize ];
+ // copy from m_pArray to new Array
+ memcpy( pTmpArray, m_pArray, m_nCount * sizeof(SwTextAttr*) );
// free old array
- delete [] pArray;
- pArray = pTmpArray;
+ delete [] m_pArray;
+ m_pArray = pTmpArray;
}
}
- OSL_ENSURE( nPos <= nCount, "wrong position for insert operation");
+ OSL_ENSURE(nPos <= m_nCount, "wrong position for insert operation");
- if ( nPos < nCount )
- memmove( pArray + nPos + 1, pArray + nPos,
- ( nCount - nPos ) * sizeof(SwTextAttr*)
+ if (nPos < m_nCount)
+ memmove( m_pArray + nPos + 1, m_pArray + nPos,
+ (m_nCount - nPos) * sizeof(SwTextAttr*)
);
- pArray[ nPos ] = const_cast<SwTextAttr*>(&rAttr);
+ m_pArray[ nPos ] = const_cast<SwTextAttr*>(&rAttr);
- nCount++;
+ m_nCount++;
}
void SwAttrHandler::SwAttrStack::Remove( const SwTextAttr& rAttr )
{
sal_uInt16 nPos = Pos( rAttr );
- if ( nPos < nCount )
+ if (nPos < m_nCount)
{
- memmove( pArray + nPos, pArray + nPos + 1,
- ( nCount - 1 - nPos ) * sizeof(SwTextAttr*)
+ memmove( m_pArray + nPos, m_pArray + nPos + 1,
+ (m_nCount - 1 - nPos) * sizeof(SwTextAttr*)
);
- nCount--;
+ m_nCount--;
}
}
const SwTextAttr* SwAttrHandler::SwAttrStack::Top() const
{
- return nCount ? pArray[ nCount - 1 ] : nullptr;
+ return m_nCount ? m_pArray[ m_nCount - 1 ] : nullptr;
}
sal_uInt16 SwAttrHandler::SwAttrStack::Pos( const SwTextAttr& rAttr ) const
{
- if ( ! nCount )
+ if ( ! m_nCount )
// empty stack
return USHRT_MAX;
- for ( sal_uInt16 nIdx = nCount; nIdx > 0; )
+ for (sal_uInt16 nIdx = m_nCount; nIdx > 0;)
{
- if ( &rAttr == pArray[ --nIdx ] )
+ if (&rAttr == m_pArray[ --nIdx ])
return nIdx;
}
More information about the Libreoffice-commits
mailing list