[PATCH] Grouping ScFormulaOptions

Albert Thuswaldner albert.thuswaldner at gmail.com
Mon Apr 9 15:31:09 PDT 2012


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
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Grouping-ScFormulaOptions.patch
Type: text/x-patch
Size: 54458 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/libreoffice/attachments/20120410/f439325b/attachment-0001.bin>


More information about the LibreOffice mailing list