Hi Eike,
Please see my Splinter review in
https://bugs.freedesktop.org/show_bug.cgi?id=50950#c3
I will do that. (I hope my contributions don't cost you too much time in correcting so that they
fit well in the general picture...)
It seems one point didn't make it across: I did not suggest to create yet
another (internal) WEEKNUM function.
I did start with keeping WEEKNUM_ADD in the Add-in module, but as I had diffuculties in using the
Date::getWeekofYear function (getting the null date right), I simply thought keeping WEEKNUM and
ISOWEEKNUM together would be logical - disrepecting pre-ODFF1.2 and interoperability issues.
I will go back to your original plan and upload another diff file to the bugs.feedesktop.org (with
corrections as taken from your Splinter review).
I'll come back to the compiler details later. The problem there may be
mapping from internal (ISOWEEKNUM) to Add-In (WEEKNUM).
I am a bit busy with other projects at the time, so don't hurry for my sake :-)
Winfried
Context
Privacy Policy |
Impressum (Legal Info) |
Copyright information: Unless otherwise specified, all text and images
on this website are licensed under the
Creative Commons Attribution-Share Alike 3.0 License.
This does not include the source code of LibreOffice, which is
licensed under the Mozilla Public License (
MPLv2).
"LibreOffice" and "The Document Foundation" are
registered trademarks of their corresponding registered owners or are
in actual use as trademarks in one or more countries. Their respective
logos and icons are also subject to international copyright laws. Use
thereof is explained in our
trademark policy.