[Libreoffice-commits] .: accessibility/source
Libreoffice Gerrit user
logerrit at kemper.freedesktop.org
Fri Nov 2 15:00:30 PDT 2012
accessibility/source/extended/textwindowaccessibility.cxx | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
New commits:
commit 465ab1c1263b76e3284f18219678e46a211c5605
Author: Julien Nabet <serval2412 at yahoo.fr>
Date: Thu Nov 1 20:19:55 2012 +0100
Related: fdo#56473 IDE: Crash when inserting a simple macro
After having discussed with Michael Meeks, a better way would be to be iterator free
Now, should all textwindowaccessibility part be iterator free?
Change-Id: I8079b3ffbc9d37bc2c3b9ede088485dd3a7e410e
diff --git a/accessibility/source/extended/textwindowaccessibility.cxx b/accessibility/source/extended/textwindowaccessibility.cxx
index 620ff0f..1e2da2e 100644
--- a/accessibility/source/extended/textwindowaccessibility.cxx
+++ b/accessibility/source/extended/textwindowaccessibility.cxx
@@ -2127,11 +2127,11 @@ void Document::notifySelectionChange( sal_Int32 nFirst, sal_Int32 nLast )
if ( nFirst < nLast )
{
Paragraphs::iterator aItBound1 = m_xParagraphs->begin();
- for (sal_Int32 i = 0; i < nLast && aItBound1 != m_xParagraphs->end() ; ++aItBound1);
+ for (sal_Int32 i = 0; i < nLast && aItBound1 != m_xParagraphs->end() ; ++aItBound1, ++i);
Paragraphs::iterator aEnd( ::std::min( aItBound1, m_aVisibleEnd ) );
Paragraphs::iterator aItBound2 = m_xParagraphs->begin();
- for (sal_Int32 i = 0; i < nFirst && aItBound2 != m_xParagraphs->end() ; ++aItBound2);
+ for (sal_Int32 i = 0; i < nFirst && aItBound2 != m_xParagraphs->end() ; ++aItBound2, ++i);
for ( Paragraphs::iterator aIt = ::std::max( aItBound2, m_aVisibleBegin ); aIt != aEnd; ++aIt )
{
More information about the Libreoffice-commits
mailing list