sw/source/core/access/accfrmobjslist.cxx | 74 +++++++++++++++---------------- sw/source/core/access/accfrmobjslist.hxx | 20 ++++---- 2 files changed, 47 insertions(+), 47 deletions(-)
New commits: commit fc53cce64400430cdc21f79c959d75fb9a26d13d Author: Miklos Vajna <vmik...@collabora.co.uk> Date: Mon Jan 30 08:56:35 2017 +0100 sw: prefix members of SwAccessibleChildSList_const_iterator Change-Id: I1420a8c655dac568f6227dbea21506a0c5806a42 Reviewed-on: https://gerrit.libreoffice.org/33685 Reviewed-by: Miklos Vajna <vmik...@collabora.co.uk> Tested-by: Jenkins <c...@libreoffice.org> diff --git a/sw/source/core/access/accfrmobjslist.cxx b/sw/source/core/access/accfrmobjslist.cxx index 36d6a880..5741b41 100644 --- a/sw/source/core/access/accfrmobjslist.cxx +++ b/sw/source/core/access/accfrmobjslist.cxx @@ -30,20 +30,20 @@ using namespace ::sw::access; SwAccessibleChildSList_const_iterator::SwAccessibleChildSList_const_iterator( const SwAccessibleChildSList& rLst, SwAccessibleMap& rAccMap ) - : rList( rLst ), - aCurr( rList.GetFrame().GetLower() ), - nNextObj( 0 ) + : m_rList( rLst ), + m_aCurr( m_rList.GetFrame().GetLower() ), + m_nNextObj( 0 ) { - if( !aCurr.GetSwFrame() ) + if( !m_aCurr.GetSwFrame() ) { - const SwFrame& rFrame = rList.GetFrame(); + const SwFrame& rFrame = m_rList.GetFrame(); if( rFrame.IsPageFrame() ) { const SwPageFrame& rPgFrame = static_cast< const SwPageFrame& >( rFrame ); const SwSortedObjs *pObjs = rPgFrame.GetSortedObjs(); if( pObjs && pObjs->size() ) { - aCurr = (*pObjs)[nNextObj++]->GetDrawObj(); + m_aCurr = (*pObjs)[m_nNextObj++]->GetDrawObj(); } } else if( rFrame.IsTextFrame() ) @@ -51,34 +51,34 @@ SwAccessibleChildSList_const_iterator::SwAccessibleChildSList_const_iterator( const SwSortedObjs *pObjs = rFrame.GetDrawObjs(); if ( pObjs && pObjs->size() ) { - aCurr = (*pObjs)[nNextObj++]->GetDrawObj(); - while( aCurr.IsValid() && !aCurr.IsBoundAsChar() ) + m_aCurr = (*pObjs)[m_nNextObj++]->GetDrawObj(); + while( m_aCurr.IsValid() && !m_aCurr.IsBoundAsChar() ) { - aCurr = (nNextObj < pObjs->size()) - ? (*pObjs)[nNextObj++]->GetDrawObj() + m_aCurr = (m_nNextObj < pObjs->size()) + ? (*pObjs)[m_nNextObj++]->GetDrawObj() : static_cast< const SdrObject *>( nullptr ); } } - if ( !aCurr.IsValid() ) + if ( !m_aCurr.IsValid() ) { ::rtl::Reference < SwAccessibleContext > xAccImpl = rAccMap.GetContextImpl( &rFrame, false ); if( xAccImpl.is() ) { SwAccessibleContext* pAccImpl = xAccImpl.get(); - aCurr = SwAccessibleChild( pAccImpl->GetAdditionalAccessibleChild( 0 ) ); - ++nNextObj; + m_aCurr = SwAccessibleChild( pAccImpl->GetAdditionalAccessibleChild( 0 ) ); + ++m_nNextObj; } } } } - if( rList.IsVisibleChildrenOnly() ) + if( m_rList.IsVisibleChildrenOnly() ) { // Find the first visible - while( aCurr.IsValid() && - !aCurr.AlwaysIncludeAsChild() && - !aCurr.GetBox( rAccMap ).IsOver( rList.GetVisArea() ) ) + while( m_aCurr.IsValid() && + !m_aCurr.AlwaysIncludeAsChild() && + !m_aCurr.GetBox( rAccMap ).IsOver( m_rList.GetVisArea() ) ) { next(); } @@ -88,14 +88,14 @@ SwAccessibleChildSList_const_iterator::SwAccessibleChildSList_const_iterator( SwAccessibleChildSList_const_iterator& SwAccessibleChildSList_const_iterator::next() { bool bNextTaken( true ); - if( aCurr.GetDrawObject() || aCurr.GetWindow() ) + if( m_aCurr.GetDrawObject() || m_aCurr.GetWindow() ) { bNextTaken = false; } - else if( aCurr.GetSwFrame() ) + else if( m_aCurr.GetSwFrame() ) { - aCurr = aCurr.GetSwFrame()->GetNext(); - if( !aCurr.GetSwFrame() ) + m_aCurr = m_aCurr.GetSwFrame()->GetNext(); + if( !m_aCurr.GetSwFrame() ) { bNextTaken = false; } @@ -103,37 +103,37 @@ SwAccessibleChildSList_const_iterator& SwAccessibleChildSList_const_iterator::ne if( !bNextTaken ) { - const SwFrame& rFrame = rList.GetFrame(); + const SwFrame& rFrame = m_rList.GetFrame(); if( rFrame.IsPageFrame() ) { const SwPageFrame& rPgFrame = static_cast< const SwPageFrame& >( rFrame ); const SwSortedObjs *pObjs = rPgFrame.GetSortedObjs(); - aCurr = ( pObjs && nNextObj < pObjs->size() ) - ? (*pObjs)[nNextObj++]->GetDrawObj() + m_aCurr = ( pObjs && m_nNextObj < pObjs->size() ) + ? (*pObjs)[m_nNextObj++]->GetDrawObj() : static_cast< const SdrObject *>( nullptr ); } else if( rFrame.IsTextFrame() ) { const SwSortedObjs* pObjs = rFrame.GetDrawObjs(); const size_t nObjsCount = pObjs ? pObjs->size() : 0; - aCurr = ( pObjs && nNextObj < nObjsCount ) - ? (*pObjs)[nNextObj++]->GetDrawObj() + m_aCurr = ( pObjs && m_nNextObj < nObjsCount ) + ? (*pObjs)[m_nNextObj++]->GetDrawObj() : static_cast< const SdrObject *>( nullptr ); - while( aCurr.IsValid() && !aCurr.IsBoundAsChar() ) + while( m_aCurr.IsValid() && !m_aCurr.IsBoundAsChar() ) { - aCurr = ( nNextObj < nObjsCount ) - ? (*pObjs)[nNextObj++]->GetDrawObj() + m_aCurr = ( m_nNextObj < nObjsCount ) + ? (*pObjs)[m_nNextObj++]->GetDrawObj() : static_cast< const SdrObject *>( nullptr ); } - if ( !aCurr.IsValid() ) + if ( !m_aCurr.IsValid() ) { ::rtl::Reference < SwAccessibleContext > xAccImpl = - rList.GetAccMap().GetContextImpl( &rFrame, false ); + m_rList.GetAccMap().GetContextImpl( &rFrame, false ); if( xAccImpl.is() ) { SwAccessibleContext* pAccImpl = xAccImpl.get(); - aCurr = SwAccessibleChild( pAccImpl->GetAdditionalAccessibleChild( nNextObj - nObjsCount ) ); - ++nNextObj; + m_aCurr = SwAccessibleChild( pAccImpl->GetAdditionalAccessibleChild( m_nNextObj - nObjsCount ) ); + ++m_nNextObj; } } } @@ -145,9 +145,9 @@ SwAccessibleChildSList_const_iterator& SwAccessibleChildSList_const_iterator::ne SwAccessibleChildSList_const_iterator& SwAccessibleChildSList_const_iterator::next_visible() { next(); - while( aCurr.IsValid() && - !aCurr.AlwaysIncludeAsChild() && - !aCurr.GetBox( rList.GetAccMap() ).IsOver( rList.GetVisArea() ) ) + while( m_aCurr.IsValid() && + !m_aCurr.AlwaysIncludeAsChild() && + !m_aCurr.GetBox( m_rList.GetAccMap() ).IsOver( m_rList.GetVisArea() ) ) { next(); } @@ -157,7 +157,7 @@ SwAccessibleChildSList_const_iterator& SwAccessibleChildSList_const_iterator::ne SwAccessibleChildSList_const_iterator& SwAccessibleChildSList_const_iterator::operator++() { - return rList.IsVisibleChildrenOnly() ? next_visible() : next(); + return m_rList.IsVisibleChildrenOnly() ? next_visible() : next(); } /* vim:set shiftwidth=4 softtabstop=4 expandtab: */ diff --git a/sw/source/core/access/accfrmobjslist.hxx b/sw/source/core/access/accfrmobjslist.hxx index 863d86e..0ade63d 100644 --- a/sw/source/core/access/accfrmobjslist.hxx +++ b/sw/source/core/access/accfrmobjslist.hxx @@ -31,13 +31,13 @@ class SwAccessibleChildSList_const_iterator private: friend class SwAccessibleChildSList; - const SwAccessibleChildSList& rList; // The frame we are iterating over - sw::access::SwAccessibleChild aCurr; // The current object - size_t nNextObj; // The index of the current sdr object + const SwAccessibleChildSList& m_rList; // The frame we are iterating over + sw::access::SwAccessibleChild m_aCurr; // The current object + size_t m_nNextObj; // The index of the current sdr object inline SwAccessibleChildSList_const_iterator( const SwAccessibleChildSList& rLst ) - : rList( rLst ) - , nNextObj( 0 ) + : m_rList( rLst ) + , m_nNextObj( 0 ) {} SwAccessibleChildSList_const_iterator( const SwAccessibleChildSList& rLst, @@ -48,14 +48,14 @@ private: public: inline SwAccessibleChildSList_const_iterator( const SwAccessibleChildSList_const_iterator& rIter ) - : rList( rIter.rList ) - , aCurr( rIter.aCurr ) - , nNextObj( rIter.nNextObj ) + : m_rList( rIter.m_rList ) + , m_aCurr( rIter.m_aCurr ) + , m_nNextObj( rIter.m_nNextObj ) {} inline bool operator==( const SwAccessibleChildSList_const_iterator& r ) const { - return aCurr == r.aCurr; + return m_aCurr == r.m_aCurr; } inline bool operator!=( @@ -68,7 +68,7 @@ public: inline const sw::access::SwAccessibleChild& operator*() const { - return aCurr; + return m_aCurr; } }; _______________________________________________ Libreoffice-commits mailing list libreoffice-comm...@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits