<html> <head> <meta content="text/html; charset=ISO-8859-1" http-equiv="Content-Type"> </head> <body bgcolor="#FFFFFF" text="#000000"> Hi Noel On 31/01/12 14:24, Noel Grandin wrote: <blockquote cite="mid:4F27F98E.2010801@peralex.com" type="cite"> <meta content="text/html; charset=ISO-8859-1" http-equiv="Content-Type"> sorry, that should be "converted to std::vector" </blockquote> you mean boost::ptr_vector I think :-) Hope I didn't miss anything this time, I made some minor changes ( see attached patch ) e.g. I removed the maLayoutNames.clear() from :~SdPresLayoutDlg() as the vector will go out of scope here anyway and delete the container contents. Also I changed slightly the code in IMPL_LINK(SdPresLayoutDlg, ClickLoadHdl, void *, EMPTYARG) and got rid of a String copy and slightly changed the calculation of 'nId' for maVS.SelectItem I am currently running make check on it this time ;-) assuming it works and no other objection to the patch I will commit to master thanks again for the patch. Noel </body> </html>