Age | Commit message (Collapse) | Author |
|
O[U]StringView had an odd mixture of uses. For one, it was used like
std::[u16]string_view, for which directly using the latter std types is clearly
the better alternative. For another, it was used in concatenation sequences,
when neither of the two leading terms were of our rtl string-related types.
For that second use case introduce O[U]String::Concat (as std::[u16]string_view
can obviously not be used, those not being one of our rtl string-related types).
Also, O[U]StringLiteral is occasionally used for this, but the planned changes
outlined in the 33ecd0d5c4fff9511a8436513936a3f7044a775a "Change OUStringLiteral
from char[] to char16_t[]" commit message will make that no longer work, so
O[U]String::Concat will be the preferred solution in such use cases going
forward, too.
O[U]StringView was also occasionally used to include O[U]StringBuffer values in
concatenation sequences, for which a more obvious alternative is to make
O[U]StringBuffer participate directly in the ToStringHelper/O[U]StringConcat
machinery.
Change-Id: I1f0e8d836796c9ae01c45f32c518be5f52976622
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/101586
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
|
|
Remove case sensitivity from NON case-sensitive Hungarian collator
in i18npool, added by commit 7b1eb6313c0d2621c364df1724c69d28f8267841
(tdf#116666 fix Hungarian sorting). It seems, this doesn't affect
Writer, where it's possible to choose case-sensitive sorting, too.
Note: Handling of space is still a problem for normal text,
i.e. sorting according to Hungarian orthography, see
commit 7618490d334409c3fc4167f32732537ef738647f
(tdf#123204 hu_HU collation: don't ignore special characters).
Co-authored-by: Tibor Nagy (NISZ)
Change-Id: I667eb5e22401a7fcef0e6e6111c48ce7d9c4aaf3
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/101527
Reviewed-by: László Németh <nemeth@numbertext.org>
Tested-by: László Németh <nemeth@numbertext.org>
|
|
This is a prerequisite for making conversion from OUStringLiteral to OUString
more efficient at least for C++20 (by replacing its internals with a constexpr-
generated sal_uString-compatible layout with a SAL_STRING_STATIC_FLAG refCount,
conditionally for C++20 for now).
For a configure-wise bare-bones build on Linux, size reported by `du -bs
instdir` grew by 118792 bytes from 1155636636 to 1155755428.
In most places just a u"..." string literal prefix had to be added. In some
places
char const a[] = "...";
variables have been changed to char16_t, and a few places required even further
changes to code (which prompted the addition of include/o3tl/string_view.hxx
helper function o3tl::equalsIgnoreAsciiCase and the additional
OUString::createFromAscii overload).
For all uses of macros expanding to string literals, the relevant uses have been
rewritten as
u"" MACRO
instead of changing the macro definitions. It should be possible to change at
least some of those macro definitions (and drop the u"" from their call sites)
in follow-up commits.
Change-Id: Iec4ef1a057d412d22443312d40c6a8a290dc6144
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/101483
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
|
|
Inherited from en-GB, zh-HK
Change-Id: I966d19cfa2da26d3d882af35afe79a2f77eaffa8
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/101139
Reviewed-by: Eike Rathke <erack@redhat.com>
Tested-by: Jenkins
|
|
after
commit 155c056b1d4674d5ff73bbb5e1ad1dcd1e6aae36
i18npool: create instances with uno constructors
this fixes two things
(*) the names of some of the constructo functions were wrong
(*) the native-code.py script needed updating
Change-Id: I5f3cad78eb2f84cb78ba7fed9f98122858fc6b81
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/100599
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
|
|
Change-Id: Ic309fc9b07186ce0b86ca54028d62e0fafd104fc
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/99950
Tested-by: Jenkins
Reviewed-by: László Németh <nemeth@numbertext.org>
|
|
and rename some classes to match their UNO implementation name.
See tdf#74608 for motivation.
Change-Id: I16aa64781d30a500f234029da6f6b00a645c46bb
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/100133
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
|
|
Change-Id: Icea2dac6e4ef6493c2a7fe5f7def0f1708caf6d9
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/95684
Tested-by: Jenkins
Reviewed-by: Tomoyuki Kubota <himajin100000@gmail.com>
Reviewed-by: Eike Rathke <erack@redhat.com>
|
|
Change-Id: I607e1defbd657546bfc017d16d0edfcf0dc2c028
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/99721
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
|
|
See tdf#74608 for motivation.
Change-Id: I814512ccc546bd015558a8122f0d2e3803437e38
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/99722
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
|
|
Follow-up for 4804d969bacd25ad586b3bf70d3dc8c27adb48ef:
* change abbreviations to omit trailing periods, as officially preferred
* add quotes around literal characters in date formats
(so those formats wouldn't be mistakenly detected as "user-defined")
* revert sorting of a few date formats for backwards compatibility's sake:
- when opening files created in 7.0, previous versions shouldn't add
". a" anymore to formats that aren't supposed to have it
Change-Id: I666273aa32e7ca363aa929b8a1fd83bf46533f6f
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/99264
Tested-by: Adolfo Jayme Barrientos <fitojb@ubuntu.com>
Reviewed-by: Christian Lohmaier <lohmaier+LibreOffice@googlemail.com>
|
|
Change-Id: I09c8d3a129c0d1d2fab561add0447869156b193f
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/96693
Tested-by: Andras Timar <andras.timar@collabora.com>
Reviewed-by: Andras Timar <andras.timar@collabora.com>
|
|
If sw text formatting gets the result of the special-case code added to
the "word boundary break" condition, it may call again the next time
with the preceding index, and fall into the "Line boundary break"
condition, which hence also needs to special case '/'.
(regression from ICU 60, 9206a08ada00e8762c4a634f242bd566028964bb)
Change-Id: I2aaefbc7b25af157e0a6ef15fabaa71bff1e8d9a
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/99104
Tested-by: Jenkins
Reviewed-by: Michael Stahl <michael.stahl@cib.de>
|
|
Change-Id: Id5144e95ac0120b3125258cdde46e4f7f6e1690b
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/99109
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
|
|
See tdf#74608 for motivation.
Change-Id: I5f8113393120fdad192f7347109a2186db141acb
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/98708
Tested-by: Noel Grandin <noel.grandin@collabora.co.uk>
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
|
|
Change-Id: I47e920a98833f71bd5c8db0a0590a4f8cdcb83b5
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/98465
Tested-by: Jenkins
Reviewed-by: Eike Rathke <erack@redhat.com>
|
|
Background and motivation:
https://tools.ietf.org/html/draft-knodel-terminology-02
Change-Id: I2f22d455d2a936a85750eaab1fda215ebb6d9d48
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/98182
Tested-by: Thorsten Behrens <Thorsten.Behrens@CIB.de>
Reviewed-by: Thorsten Behrens <Thorsten.Behrens@CIB.de>
|
|
Change-Id: If41314c90397199c14c91fe65ed41243ac385dae
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/97827
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
|
|
Change-Id: I8b93f7610103f4bd8e86e2d9f57929922b6dc92c
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/97568
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
|
|
Change-Id: Ie06a19b902e4dc12914fe0f2328a5abafa85368e
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/96208
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
|
|
See bt here:
https://bugs.documentfoundation.org/attachment.cgi?id=161882
Change-Id: If5a6c5e2b52af9ef97af9522296aaa58352cfa69
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/96161
Tested-by: Jenkins
Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>
Reviewed-by: Julien Nabet <serval2412@yahoo.fr>
|
|
Change-Id: I5644ca7f2ef1b251ce1c262d3001ca48f2ed9edd
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/95482
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
|
|
Single Quotes AutoCorrect, i.e. pressing Ctrl + '
will insert typographic apostrophes (third level
Hungarian quoation marks) instead of » and «, like MSO does.
This reverts commit 0d0c13bfbdff85a18433aee6e94558689f0cb722
(tdf#115382 Hungarian autocorrect: help apostrophe usage), too.
See also commit b9910e87de4eea1cb3684bb7af8e58d681cbe809
(Resolves: tdf#116062 revert [fr-CH] to use previous
single quote characters)
Change-Id: I9ff5cb9a55699c253b6a57589186caa7caa170a8
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/95210
Tested-by: László Németh <nemeth@numbertext.org>
Reviewed-by: László Németh <nemeth@numbertext.org>
|
|
Change-Id: Iedfd492c963eb89fe75fdd73cae630e7e1dae119
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/95100
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
|
|
Change-Id: Ia7870d1d0d91de213727116ccda5b41913223866
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/95097
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
|
|
Change-Id: I19eba57bc6058c317473d0746f06699a09ba2830
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/94608
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
|
|
* fix MinimalDaysInFirstWeek (supposed to be 4, not 1)
* fix FollowPageWord ("jj." = "ja järgmine/järgmised", not "pp.")
* fix Era DefaultAbbrvName & DefaultFullName to official spellings
* add DefaultNarrowName for months (using Roman numerals)
* translate TimeAM, TimePM - not commonly used, but abbreviations exist
* changes to date formats:
- add some that are used, but were missing
- remove those that aren't commonly used or which are ungrammatical
- switch to more sensible defaults
- minor re-sorting into more logical order
Change-Id: I2cf585febe7f100b2b63a94a3fd957a381adc131
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/94256
Tested-by: Jenkins
Reviewed-by: Eike Rathke <erack@redhat.com>
|
|
Change-Id: I2f822a839724c7b4c2129d7f1bd1cce025b9289c
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/94254
Tested-by: Jenkins
Reviewed-by: Eike Rathke <erack@redhat.com>
|
|
Change-Id: Iff68e8f379614a6ab6a6e0d1bad18e70bc76d76a
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/91907
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
|
|
Change-Id: I1de87468b56b86a1eeee09a612551ab119a1be8b
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/93857
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
|
|
See workdir/UnpackedTarball/icu/source/common/utrie2.cpp, the
U_POINTER_MASK_LSB() check:
if( length<=0 || (U_POINTER_MASK_LSB(data, 3)!=0) ||
valueBits<0 || UTRIE2_COUNT_VALUE_BITS<=valueBits
) {
*pErrorCode=U_ILLEGAL_ARGUMENT_ERROR;
return 0;
}
Apparently the data pointer there is always a multiple of four bytes
from the start of the data array that the gencoll_rule code generates.
Change-Id: I9b98b01b49b5800e1db8b077a4221b82d59510bf
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/93507
Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoffice@gmail.com>
Reviewed-by: Tor Lillqvist <tml@collabora.com>
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/93603
Tested-by: Jenkins
Reviewed-by: Andras Timar <andras.timar@collabora.com>
|
|
Change-Id: Ic8e09d31db97c0cf2e1aaf006c96481d12deb2d8
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/93506
Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoffice@gmail.com>
Reviewed-by: Tor Lillqvist <tml@collabora.com>
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/93592
Tested-by: Jenkins
|
|
Improve error repoting in BreakIteratorImpl
Change-Id: I0be64a758ed81b7a720c8b26af14de6b51cc5dbc
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/92955
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
|
|
from
commit c5424e19338a3edaec3f0459c8ac5d53ca92d9fe
loplugin:useuniqueptr in i18npool
which would have resulted in the block at line 245 inside
#if (U_ICU_VERSION_MAJOR_NUM < 58)
never doing anything
spotted while doing improvements to my make_shared plugin
Change-Id: I79c664c7e4a051f3c764cb49d99870b51b19ce55
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/92567
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
|
|
Change-Id: Iac753e528e13cb2565832a484e87f88061bbc91e
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/92239
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
|
|
and workaround a clang crash
Change-Id: Ida94c8abb4b2e997d38a7f430e59f73aadf8fcc8
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/91844
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
|
|
i18npool/source/collator/data/ko_charset.txt
Korean Hangul syllables ordering is wrong.
Some hangul syllables are dissapeared on the text file.
Hangul Syllable ordering is already specified on Unicode Code chart.
Ref. Hangul Syllables Range: AC00–D7AF
https://unicode.org/charts/PDF/UAC00.pdf
That commit applies only Hangul Syllables range.
Korean Hanja[한자/漢字] range will require investigation.
hanja[한자/漢字] is korean name for chinese character.
Change-Id: I31e5cbf04294ee3bd6bff3277f9fe1328530ac3a
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/87018
Reviewed-by: Eike Rathke <erack@redhat.com>
Tested-by: Jenkins
|
|
Change-Id: I865d6b3dcb7f3bff037a4015aa98db2fa2578672
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/91593
Reviewed-by: Eike Rathke <erack@redhat.com>
Tested-by: Jenkins
|
|
Ende -> End
Change-Id: I47faa58be14d9e608a4fad61279026d676c185c4
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/91207
Tested-by: Jenkins
Reviewed-by: Michael Weghorn <m.weghorn@posteo.de>
|
|
According to localeplanet [1] [2], in comparison to other software and people
living it these areas, the LongDateDaySeparator '.' for fr_CH and it_CH in
LibreOffice is not correct. It should be omitted.
[1] http://www.localeplanet.com/icu/it-CH/index.html
[2] http://www.localeplanet.com/icu/fr-CH/index.html
This means for the FormatElement index 22, 23, 24, 25, 26 should be the same in
fr_CH/fr_FR and it_CH/it_IT.
Change-Id: Ief4de0d8728c7a3bbcfac7f6200f37f2d2c647aa
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/90427
Tested-by: Jenkins
Reviewed-by: Eike Rathke <erack@redhat.com>
|
|
This is the last padded numbering type that is supported by Word but was
not supported by Writer.
Change-Id: Ica1a0843897c61a4b569105fd21e5bfe7b5012cb
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/90912
Reviewed-by: Miklos Vajna <vmiklos@collabora.com>
Tested-by: Jenkins
|
|
in i18npool/source/collator/collator_unicode.cxx, we got:
177 // replace algorithm name to implementation name.
178 if (rAlgorithm == "phonetic (alphanumeric first)")
179 aBuf.append("phonetic_alphanumeric_first");
180 else if (rAlgorithm == "phonetic (alphanumeric last)")
181 aBuf.append("phonetic_alphanumeric_last");
182 else
183 aBuf.append(rAlgorithm);
So don't add extra ja_ before "phonetic..."
Also we already add "ja" in buffer with line:
158 aBuf.append("get_").append(rLocale.Language).append("_");
so right functions from ICU will be retrieved
Change-Id: I163c3ca4bb4dcfa1e5d29313190c5ba3e6396c4f
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/90877
Tested-by: Jenkins
Reviewed-by: Eike Rathke <erack@redhat.com>
|
|
This makes the UI work as well.
Change-Id: I4e94b85097cc359b257b07ba7517edfab3011093
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/90827
Reviewed-by: Miklos Vajna <vmiklos@collabora.com>
Tested-by: Jenkins
|
|
This is the actual numbering the customer needed, pad-to-2 and pad-to-3
was added just for compleness (since Word has it and it's related).
Change-Id: I7fdf67488955ab3ee0db169f11fffd21d9cc1e3b
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/90791
Reviewed-by: Miklos Vajna <vmiklos@collabora.com>
Tested-by: Jenkins
|
|
This makes the UI work as well.
Change-Id: I8d64b88e57ba3e4fd61afba892f0ee2267f1c8b2
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/90683
Reviewed-by: Miklos Vajna <vmiklos@collabora.com>
Tested-by: Jenkins
|
|
This is similar to the existing padded numbering, but that one padded to
2. Another difference is pad-to-2 has more file format support:
pad-to-3 is not supported in DOC and RTF.
Change-Id: Ie2ac2691c58a89e181d24d7002cf873ebab380c4
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/90656
Reviewed-by: Miklos Vajna <vmiklos@collabora.com>
Tested-by: Jenkins
|
|
Change-Id: I615ffc60fcc235fcc84848a22fd7bd3b0d8179d1
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/90441
Tested-by: Jenkins
Reviewed-by: Eike Rathke <erack@redhat.com>
|
|
Change-Id: I2a7de5723f8ce82cf10b59ed5f77d2a7dfbb10e2
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/90353
Tested-by: Jenkins
Reviewed-by: Eike Rathke <erack@redhat.com>
|
|
Change-Id: Ic7e609ebac10921b4891877802892fe2cde9ecbe
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/90406
Tested-by: Jenkins
Reviewed-by: Eike Rathke <erack@redhat.com>
|
|
ODF allows any string as style:num-format="...", go with "01, 02, 03,
...", because that seems to be consistent with both
DefaultNumberingProvider::makeNumberingIdentifier()'s fallback mechanism
and with OOXML (which uses "001, 002, 003, ..." for the "pad to 3"
case).
Change-Id: I5c5c7ee5bd61175afc3e682276e69344852106d5
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/89891
Reviewed-by: Miklos Vajna <vmiklos@collabora.com>
Tested-by: Jenkins
|