Age | Commit message (Collapse) | Author |
|
I went through all files manually and checked with the tool.
Change-Id: I2eba5e61a0814dd88d7c2c001cd85db8032f1b9f
Reviewed-on: https://gerrit.libreoffice.org/10545
Reviewed-by: Eike Rathke <erack@redhat.com>
Tested-by: Eike Rathke <erack@redhat.com>
|
|
Change-Id: I1dd003bc984a11d6d71c92aae44accc9d358db0c
|
|
rStr is NaN
Change-Id: I8206ab322a08d558cf5f05750efd700875b2134c
|
|
Change-Id: I0f9cf74550e43d174bf6ac75e70c51ab7f51ccf8
|
|
... to prevent confusion of #.### input with D.M that then later is
discarded as invalid date input instead of accepted as valid numeric
input.
Change-Id: I178e28b7ef3b26d04eecc73e5e5c61ee41f89e32
|
|
...to avoid -fsanitize=function warnings in Link::Call.
Change-Id: I837b35bd3052716fbb41bf4f893961257d1f9b2c
|
|
Change-Id: I7777ec51b00e11204457dd11359d14c2bcf0e283
|
|
a5f62d96b124564f522eac7925e9c3d22c37e503 would had liked to let
usage="DATE_TIME" formatindex="55" define the default format of that
usage, which wasn't possible. There is no reason to prevent this.
Change-Id: Ibef453c8a07aa03b0a8bb626b9cf71d362be0699
|
|
Force ##-MMM-## to be ##-MMM-#### if the first number is a day value.
Note that the previous implementation of
(bDay1 && bDay2 && ((bYear1 && !bYear2) || (!bYear1 && bYear2)))
never matched anyway as the detection of bDay1 and bDay2 were changed to
only yield true if bYear1==false or bYear2==false.
Change-Id: If6bb6d3e1757015552eaa81c1f58d53c0794846e
|
|
No need to create cached versions of stringified sheet tokens for
a given grammar at great for every formula that we compile; defer
until use. Is this a large cost on save ?
Change-Id: I8058ed564dbdc00ff45c02cb483c1a20a48af272
|
|
Change-Id: I991801b9a0b7be6024612c8f29262f2ebd8e6a64
|
|
Change-Id: I2cf02a26b81fa634c842df455de669f1c60241bc
|
|
This reverts commit 3b3b690c1f0479cfbebcfa68595f75a4994e7a5f.
With the use of StringContainsWord() it is unnecessary to loop all
available names.
|
|
Follow-up, check for ASCII first to avoid calls to i18n, and check the
type flags instead of calls to CharClass methods that give unexpected
results with their masks.
Change-Id: I10e685998299dceb2dbcf1d87ae1de09680b8a99
|
|
Change-Id: I897dbcee47de574d91ba3e3b40a39a35b779fef8
|
|
Change-Id: I18fb7a677e2bb98c83f43f5af3fa988393dc23f9
|
|
Change-Id: Idad7b083ce5528f5c735f3a8bd091819bf043fc8
|
|
Change-Id: I3c932e8386d994450f05a876b6ad76ba141bd4e9
|
|
Change-Id: I67acda35fa127547dcea0cd18c9dc16db7c00294
|
|
Change-Id: Ie16923d17541e84e0d7424fffe37caf410786abf
|
|
Change-Id: I9906b4e58fd8672e51f6aa2e56ec68bb28772500
|
|
Look for code like:
xxx ? yyy : false;
Which can be simplified to:
xxx && yyy
Change-Id: Ia33c0e452aa28af3f0658a5382895aaad0246b4d
|
|
Change-Id: I997dad13c3b8931d1fd4eae09218b51cb4621263
|
|
Change-Id: Id7ecd49ea03f225fc350a31437c32b4a738d7199
|
|
Change-Id: Iffad31bc60748fe58fe3524776dd56e907586342
|
|
It appears that the C++ standard allows overriding destructors to be marked
"override," but at least some MSVC versions complain about it, so at least make
sure such destructors are explicitly marked "virtual."
Change-Id: I0e1cafa7584fd16ebdce61f569eae2373a71b0a1
|
|
Change-Id: I95ef44f12ad478677e761edb19528829d33363c9
|
|
Change-Id: I386944df28c7e7464e596144a5612a205f5cb967
Reviewed-on: https://gerrit.libreoffice.org/8789
Reviewed-by: Eike Rathke <erack@redhat.com>
Tested-by: Eike Rathke <erack@redhat.com>
|
|
Change-Id: I08683ea7dc48f5ac91d56ee4de6c4e52d1c6f058
|
|
Change-Id: I13d0cb525f3ca22b1b9158b2ac20b8b59d645463
|
|
Change-Id: I402f64fdab57508df20d4dffa7bdfdf9473d8b45
|
|
...mostly done with a rewriting Clang plugin, with just some manual tweaking
necessary to fix poor macro usage.
Change-Id: I71fa20213e86be10de332ece0aa273239df7b61a
|
|
Change-Id: I59ab0c6883e7fc02bed2f4e286a83b6054d8ef3b
|
|
...that causes a call to std::abs with a short argument to effectively select
the double overload (via a template added with <http://gcc.gnu.org/git/
?p=gcc.git;a=commit;h=8c0edf5c2aad076cbc805299ed82845ae049f4f6>
"include/c_std/cmath (abs(_Tp)): Add"), see
<http://www.open-std.org/jtc1/sc22/wg21/docs/lwg-toc.html#2192> "Validity and
return type of std::abs(0u) is unclear" for details.
The workaround is to use int instead of short arguments (and make sure the
integral overloads of std::abs from cstdlib are available).
Change-Id: If95363f9a54abb9cb1a57fd4828c4b91c12bc4be
|
|
Change-Id: I56e32131b7991ee9948ce46765632eb823d463b3
|
|
Change-Id: Ib914ec8d161e24f4e3e1fb8e7684a780d3ae6209
Reviewed-on: https://gerrit.libreoffice.org/8319
Reviewed-by: Björn Michaelsen <bjoern.michaelsen@canonical.com>
Tested-by: Björn Michaelsen <bjoern.michaelsen@canonical.com>
|
|
Change-Id: Ib03c97a52df120bac1ac9b2b9d2e52431ead1027
|
|
Change-Id: I9dc0525e04de5ae79205872b779dcd0115a9cc14
|
|
Change-Id: Ic31455a1f5ffffa35d4fdde901dd70734207b6f4
|
|
Convert the template based read_lenPrefixed methods to regular
methods.
Change-Id: Ifd0e93aca055e55a0575e4377ec2b8e266dfb019
Reviewed-on: https://gerrit.libreoffice.org/7895
Reviewed-by: Michael Stahl <mstahl@redhat.com>
Tested-by: Michael Stahl <mstahl@redhat.com>
|
|
First, I updated the clang rewriter to do the conversion.
Then I lightly hand-tweaked the output for the few places where
the rewriter messed up, mostly when dealing with calls on "this".
Change-Id: I40a6a977959cd97415c678eafc8507de8aa3b1a9
Reviewed-on: https://gerrit.libreoffice.org/7879
Reviewed-by: Michael Stahl <mstahl@redhat.com>
Tested-by: Michael Stahl <mstahl@redhat.com>
|
|
.. to more explicit SvStream::Write* calls
This was done using another run of the clang rewriter, and then
a lot of hand tweaking to fix all the places where the rewriter
did not play nice with various macros.
Change-Id: I7bcab93851c8dfb59cde6bc76290c6484d88fb18
Reviewed-on: https://gerrit.libreoffice.org/7494
Reviewed-by: Michael Stahl <mstahl@redhat.com>
Tested-by: Michael Stahl <mstahl@redhat.com>
|
|
...final season
Change-Id: Ie51ac231def4a8a6e2681b81de7580798c95eed2
|
|
This is the actual re-write.
Use a clang rewriter to rewrite SvStream::operator<< to methods
like WriteuInt32.
Note that the rewriter is not perfect, and I hand-tweaked the output.
In particular, I had to adjust places doing things like
(*this) << 1;
Change-Id: I5923eda3f4ebaa8b452b6ef109e726e116235a2a
Reviewed-on: https://gerrit.libreoffice.org/7342
Tested-by: LibreOffice gerrit bot <gerrit@libreoffice.org>
Reviewed-by: Michael Stahl <mstahl@redhat.com>
|
|
Convert code like:
buf.append( static_cast<sal_Unicode>('!') );
to:
buf.append( '!' );
Change-Id: Iacb03a61de65a895540940953b49620677b3d051
|
|
Change-Id: Ib2690e3ec9987b97363687b61fe8ddae4ace9058
|
|
...in comphelper::PropertyMapEntry and SfxItemPropertyMapEntry. And as the
arrays of such need to be initialized dynamically anyway, also change their name
members to proper OUStrings while at it. Plus some const clean-up.
Change-Id: I67d4d7b5773fb020605f369daf39528bec930606
|
|
This reverts commit 90f91088d238469b4a2262c91de3117ba40f5bde for now:
Ach, old GCC doesn't like plain string literals to initialize members
of OUString type...
Change-Id: I50563a00406259bb5d41831e2a2796762450d097
|
|
...in comphelper::PropertyMapEntry and SfxItemPropertyMapEntry. And as the
arrays of such need to be initialized dynamically anyway, also change their name
members to proper OUStrings while at it. Plus some const clean-up.
Change-Id: I67d4d7b5773fb020605f369daf39528bec930606
|
|
Change-Id: Ifeecd82f98f9be9f90d293a7b68691f91d7bbc34
|