diff options
| author | Matteo Casalin <matteo.casalin@yahoo.com> | 2014-08-18 20:18:40 +0200 |
|---|---|---|
| committer | Matteo Casalin <matteo.casalin@yahoo.com> | 2014-08-18 20:26:24 +0200 |
| commit | 5d5c102e36bcc7eac6ae89d44a635e5d9b680e40 (patch) | |
| tree | 8cffaef1f1ecbdf30fd983dae540d8c08bfccb41 | |
| parent | b4454c19c424d50b75fff8ef2965c5cf379c8558 (diff) | |
Typo: loop on all Marks instead of processing Mark(0) over and over
See:
http://nabble.documentfoundation.org/Suspicious-loop-any-idea-td4119239.html
Since the code was like this since initial import (2000), if this patch proves
to be wrong then the other option could be to just don't loop and process
Mark(0) just once.
Change-Id: I41ac18604e916740dcc68bf69af37529842f774d
| -rw-r--r-- | sw/source/core/frmedt/feshview.cxx | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/sw/source/core/frmedt/feshview.cxx b/sw/source/core/frmedt/feshview.cxx index 49dd779dd40e..97c064c35d80 100644 --- a/sw/source/core/frmedt/feshview.cxx +++ b/sw/source/core/frmedt/feshview.cxx @@ -731,7 +731,7 @@ static void lcl_NotifyNeighbours( const SdrMarkList *pLst ) bool bCheckNeighbours = false; sal_Int16 aHori = text::HoriOrientation::NONE; SwRect aRect; - SdrObject *pO = pLst->GetMark( 0 )->GetMarkedSdrObj(); + SdrObject *pO = pLst->GetMark( j )->GetMarkedSdrObj(); if ( pO->ISA(SwVirtFlyDrawObj) ) { SwFlyFrm *pFly = ((SwVirtFlyDrawObj*)pO)->GetFlyFrm(); |