[Libreoffice] [PATCH V2] convert SdCustomShow from tools/list to vector
Noel Power
nopower at suse.com
Tue Jan 31 09:06:37 PST 2012
On 31/01/12 15:55, Ivan Timofeev wrote:
> 31.01.2012 19:42, Noel Grandin пишет:
>> So I should be calling
>> maPages.erase( std::find(maPages.begin(), maPages.end(), pOldPage) );
>> ??
>
> maPages.erase( std::remove(maPages.begin(), maPages.end(), pOldPage),
> maPages.end() );
>
> I'll fix that.
>
> Ivan
>
nice catch, I admit not being familiar with this but I did read
"http://en.cppreference.com/w/cpp/algorithm/remove" and even read the
second paragraph where it mentions the items are removed but fails ( imo
) to clearly indicate the removed items are not removed. Sure if you
read it *very carefully* you get it :-/ ( appears I am not careful enough )
More information about the LibreOffice
mailing list