[PATCH] Grouping ScFormulaOptions

Kohei Yoshida kohei.yoshida at gmail.com
Tue Apr 10 07:26:39 PDT 2012


On Mon, Apr 9, 2012 at 6:31 PM, Albert Thuswaldner
<albert.thuswaldner at gmail.com> wrote:
> Hi Kohei,
> As said back in February:
> http://lists.freedesktop.org/archives/libreoffice/2012-February/027009.html
> here is a follow-on patch that "groups" the formula options in the
> same fashion as the print options.
>
> This is a quite straight-forward re-factoring of the code so it
> shouldn't be too hard to review.
>
> One thing that you might object to is the use of temporary variables in
> ScTpFormulaOptions::FillItemSet
>
> If this is the case, please let me know (and why ;) ) and I re-work it.
>
> Once this is pushed, I'll do the same conversion of the defaults options.
>
> /Albert
>
> _______________________________________________
> LibreOffice mailing list
> LibreOffice at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/libreoffice
>


More information about the LibreOffice mailing list