[PATCH] Grouping ScFormulaOptions
Kohei Yoshida
kohei.yoshida at gmail.com
Tue Apr 10 09:31:28 PDT 2012
Hi Albert,
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.
Reviewed it, and pushed to master with some minor modifications.
What I modified are:
1) have the assignment operator return a non-const reference of "this"
object instead of const reference.
2) have the equality and non-equality operators of ScFormulaOptions
return bool instead of int.
Regarding 2), I didn't modify the return type of operator== for
ScTpFormulaItem because it was a virtual function.
Another nice work, Albert. Much appreciated. :-)
Kohei
More information about the LibreOffice
mailing list