Hello Willam, On Thu, 2014-05-29 at 11:34 +0200, William Bonnet wrote: > Hello Kohei, > > Thanks for your feedback. I'll do my best to be C++03 compliant for my > next patches . So far i haven't found an equivalent to shrink_to_fit, > but i think i have another to do it... i'll given it a look during this > week end.
With pre-C++11 C++, there is a way to achieve that though it's not as clean as shrink_to_fit() of C++11. Referencing Item 17 from "Effective STL" by Scott Meyers, the following idiom vector<T> vec; ... (do stuff to increase vec's capacity) vector<T>(vec).swap(vec); cuts any excess capacity the vec container may have, because std::vector's copy constructor only allocates to the actual size of the container and disregards the capacity of the source container. And the following swap call puts the copied content back to the original. > Meanwhile, i think we can improve things a little with the patch i > attached to this email. ... > If you agree with the solution i proposed, i'll check if other method > need the same kind of optimization and i will submit a patch. Yeah, I think it's reasonable. So, let's go for that. It may be worthwhile to see if we could share this code into a common method if that's doable. Also, it would be great to have you adopt the existing brace styles of mdds. That would improve the readability of the code in general. Best, Kohei _______________________________________________ LibreOffice mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/libreoffice
