[PATCH] [WIP] fdo#45747 - [EasyHack] remove the limitation to 3 sort entries in calc

Markus Mohrhard markus.mohrhard at googlemail.com
Sat Mar 31 09:36:23 PDT 2012


Hello Albert,

>
> sorry that it took me some time but I had a long fight with our chart
> interpolation algorithm and was not able to update my tree for more
> than a week.
>
>>> - ScSortDescriptor::FillSortParam looks like there might happen some
>>> index out of bounds access
>>
>> So I have updated the patch with respect to your comments above and
>> corrected all the issues except for the last one, which I honestly can
>> not find. There are several ScSortDescriptor::FillSortParam in
>> different files which are touch by my patch. So could help me pointing
>> out the line(s) which you are concern about?
>
> Reviewed and pushed your patch. Fixed the out-of bounds access in
> http://cgit.freedesktop.org/libreoffice/core/commit/?id=d31f5476fddda994cad6920f7383e4c92ccac9a2
>

And had to revert it immediately. We can use initializer lists because
older gcc versions don't know this feature. I got spammed by nearly
every of our tinderbox. I tried to fix them but the whole patch is
build around that.

Can you try to create a version of the patch without initializer list usage?

Regards,
Markus


More information about the LibreOffice mailing list