[PATCH] [PUSHED ] convert SdPresLayoutDlg from tools/list.hxx to std::map
Noel Power
nopower at suse.com
Mon Feb 6 02:06:56 PST 2012
First marking this as pushed ( as I pushed this but forgot to mark it as
so )
On 04/02/12 16:00, Michael Stahl wrote:
[...]
> hi Noel,
>
> a further improvement would be to replace boost::ptr_vector with
> std::vector: our various String classes (at least the 4 most popular
> ones) effectively are already smart pointers to an array and can be
> stored directly in STL containers, so wrapping them in _another_ pointer
> is just, errm, pointless
I suppose you save lots of String copies but.... presumably if the
String implementation really is 'smart' ( which iirc it is ) then it
only points to the existing string and doesn't actually create a 'new'
copy of that. I wonder why the orig implementation didn't store objects
( or didn't List ever support that ) or was it influenced by a heap vs
stack concern ( that probably is irrelevant these days ) Anyway, won't
do any harm leaving it as it is or perhaps someone will go the extra
step and change it.
Noel
More information about the LibreOffice
mailing list