Interestingly enough there seem to be two definitions for the frac
function. Your implementation is the one seemingly used by all
programming languages, which implement it. But the correct
mathematical definition seems to be y = x - floor(x)
(http://mathworld.wolfram.com/FractionalPart.html), which gives
different values for positive and negative numbers.

Other than that I believe the patch is fine.

Greetings

eymux

On Fri, Oct 21, 2011 at 5:19 AM, August Sodora <[email protected]> wrote:
> I came across this feature request today and decided to try my hand at
> it. Is the attached patch an appropriate solution?
>
> August Sodora
> [email protected]
> (201) 280-8138
>
> _______________________________________________
> LibreOffice mailing list
> [email protected]
> http://lists.freedesktop.org/mailman/listinfo/libreoffice
>
>
_______________________________________________
LibreOffice mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/libreoffice

Reply via email to