[PATCH] [PUSHED] convert dptabres.cxx in SC module from SV_DECL_PTRARR_DEL to std::vector

Kohei Yoshida kohei.yoshida at suse.com
Tue Feb 7 09:43:54 PST 2012


Hi Noel,

On Tue, 2012-02-07 at 18:40 +0200, Noel Grandin wrote:

> Note that I did not use boost::ptr_vector because that would have meant 
> switching to using references everywhere, and the existing logic makes 
> use of NULL values, so I left that for a future patch.

There is an easy trick to that problem.  With ptr_vector you can still
return pointer via

  ptr_vector<type> container;
  ....
  type* p = &container[n];

But it's not that critical; we can just keep this in vector for now.
Not a big deal.

Pushed to master with a few changes.  Thanks a lot.

Kohei

-- 
Kohei Yoshida, LibreOffice hacker, Calc



More information about the LibreOffice mailing list