Re: [Libreoffice] [PATCH] Replace List with std::vector ImplBmpObj*

2011-07-20 Thread Matúš Kukan
On 20 July 2011 04:51, Joseph Powers jpower...@cox.net wrote: I only compile for Mac OS; thus, I only compile /vcl/aqua /vcl/source. If you watch your compile, you should only compile /vcl/unx /vcl/source. We also have /vcl/win for those brave people who compile on Windows. I'm sorry

Re: [Libreoffice] [PATCH] Replace List with std::vector ImplBmpObj*

2011-07-19 Thread Matúš Kukan
Hi Joe, On 19 July 2011 06:40, Joseph Powers jpower...@cox.net wrote: I'd like someone doing a Unix build to review this for me. I compile Mac and this is Unix only code so I don't just want to push and hope... First I thought it would compile and want just to write something but then I tried

Re: [Libreoffice] [PATCH] Replace List with std::vector ImplBmpObj*

2011-07-19 Thread Joseph Powers
On Jul 19, 2011, at 12:51 AM, Matúš Kukan wrote: Hi Joe, On 19 July 2011 06:40, Joseph Powers jpower...@cox.net wrote: I'd like someone doing a Unix build to review this for me. I compile Mac and this is Unix only code so I don't just want to push and hope... First I thought it would

Re: [Libreoffice] [PATCH] Replace List with std::vector ImplBmpObj*

2011-07-19 Thread Matúš Kukan
On 19 July 2011 14:22, Joseph Powers jpower...@cox.net wrote: A List would  be better; however, it's a list of pointers so the size isn't that big. So why not use it ? I did not mean the actual size in memory but the number of elements. I've seen there around 150 elements when I tried to

Re: [Libreoffice] [PATCH] Replace List with std::vector ImplBmpObj*

2011-07-19 Thread Joseph Powers
On Jul 19, 2011, at 7:34 AM, Matúš Kukan wrote: On 19 July 2011 14:22, Joseph Powers jpower...@cox.net wrote: A List would be better; however, it's a list of pointers so the size isn't that big. So why not use it ? I did not mean the actual size in memory but the number of elements.