[Libreoffice] [PATCH V2] replace open-coded array with std::vector in ScColumn
noelgrandin at gmail.com
Fri Jan 20 14:01:16 PST 2012
I hereby confirm that this and my future contributions are licensed
The problem was in ScColumn::Resize
I was calling vector::resize when the correct call is vector::reserve
At some point I should rename the ScColumn::Resize method to match the
-- Noel Grandin
On Fri, Jan 20, 2012 at 19:07, Kohei Yoshida <kohei.yoshida at suse.com> wrote:
> On Fri, 2012-01-20 at 15:15 +0200, Noel Grandin wrote:
>> this patch passes valgrind tests in the sc module, and passes a global
>> make check.
> Yup. All looks green. Nice work. :-)
> Before I push, could you verify that you are submitting this patch under
> GPLv3+/LGPLv3+/MPL1.1+ licenses?
> Out of curiosity, what was the problem that was causing the unit test to
> fail with your earlier patch?
> Kohei Yoshida, LibreOffice hacker, Calc
> LibreOffice mailing list
> LibreOffice at lists.freedesktop.org
More information about the LibreOffice