[Libreoffice] [PATCH V2] convert SdCustomShow from tools/list to vector

Ivan Timofeev timofeev.i.s at gmail.com
Tue Jan 31 07:24:02 PST 2012


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


More information about the LibreOffice mailing list