Re: Question about iterator management in sw/source/core/fields/cellfml.cxx

Would this patch better ? (I kept the for loop)
diff --git a/sw/source/core/fields/cellfml.cxx
b/sw/source/core/fields/cellfml.cxx
index 33b953e..5c626dd 100644
--- a/sw/source/core/fields/cellfml.cxx
+++ b/sw/source/core/fields/cellfml.cxx
@@ -967,8 +967,8 @@ void SwTableFormula::GetBoxes( const SwTableBox&
rSttBox,
 
                 if( pTbl->IsHeadline( *pLine ) )
                 {
-                    rBoxes.erase( it++ );
-                    --it;
+                    SwSelBoxes::iterator toErase = it;
+                    rBoxes.erase( toErase );
                 }
             }
         } while( sal_False );

I read that to erase a position on a iterator invalidate this iterator from
position to the end
(http://www.cplusplus.com/reference/stl/vector/erase/)
I don't know if to create this temporary variable "toErase" change something
or it's all the same.

Moreover, I don't know if the evolution of the standard had changed the
behavior on this point.

Any idea ?

Julien
(sorry for the nitpicking but I'd like to understand this point)

--
View this message in context: 
http://nabble.documentfoundation.org/Question-about-iterator-management-in-sw-source-core-fields-cellfml-cxx-tp3708331p3711039.html
Sent from the Dev mailing list archive at Nabble.com.