So I should be calling
   maPages.erase( std::find(maPages.begin(), maPages.end(), pOldPage) );
??

On 2012-01-31 17:24, Ivan Timofeev wrote:
31.01.2012 18:22, Noel Power пишет:
On 30/01/12 13:54, Noel Grandin wrote:
Attached patch converts SdCustomShow to use std::vector.
I hate these remove list etc. type patches as they tend to be tricky and
time consuming to look at ;-) . The patch looked ok for me, hopefully if
I missed something ( 'cause these really are tricky ) then someone else
will spot it.

std::remove doesn't remove elements from the vector. It just moves the elements to the front. So we have to 'vector::erase' garbage when replacing RemovePage by std::remove. Am I right?

Ivan


Disclaimer: http://www.peralex.com/disclaimer.html


_______________________________________________
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice

Reply via email to