[Libreoffice] [PATCH V2] replace open-coded array with std::vector in ScColumn

Kohei Yoshida kohei.yoshida at suse.com
Fri Jan 20 14:19:06 PST 2012


Hi Noel,

On Sat, 2012-01-21 at 00:01 +0200, Noel Grandin wrote:
> I hereby confirm that this and my future contributions are licensed
> under GPLv3+/LGPLv3+/MPL1.1+

Thanks for the confirmation.  Just pushed to master, with my follow-up
changes (as I mentioned in my earlier post).

> The problem was in ScColumn::Resize
> I was calling vector::resize when the correct call is vector::reserve

Ah! Ok.  I didn't spot that either.  Good catch.

> At some point I should rename the ScColumn::Resize method to match the
> std::vector naming

Yup.  That makes sense.  No objection with that.

Regards,

Kohei

-- 
Kohei Yoshida, LibreOffice hacker, Calc



More information about the LibreOffice mailing list