Age | Commit message (Collapse) | Author |
|
...which (in LIBO_INTERNAL_ONLY) for Clang expands to [[clang::fallthrough]] in
preparation of enabling -Wimplicit-fallthrough. (This is only relevant for
C++11, as neither C nor old C++ has a way to annotate intended fallthroughs.)
Could use BOOST_FALLTHROUGH instead of introducing our own SAL_FALLTHROUGH, but
that would require adding back in dependencies on boost_headers to many
libraries where we carefully removed any remaining Boost dependencies only
recently. (At least make SAL_FALLTHROUGH strictly LIBO_INTERNAL_ONLY, so its
future evolution will not have any impact on the stable URE interface.) C++17
will have a proper [[fallthroug]], eventually removing the need for a macro
altogether.
(cherry picked from commit 14cd5182c5f64c43581c82db8c958369152226ac)
Change-Id: I342a7610a107db7d7a344ea9cbddfd9714d7e9ca
|
|
Change-Id: Idd31b0a53c8318af69bbcd32f6798721ec8eb8e1
Reviewed-on: https://gerrit.libreoffice.org/21945
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
(cherry picked from commit 1ef9f3988ee4dcbc77e1fdefa20442e044a67d4d)
|
|
Reviewed-on: https://gerrit.libreoffice.org/21209
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Ashod Nakashian <ashnakash@gmail.com>
(cherry picked from commit 64d624b65124ac02d8ee59b135593fd9d8eb9067)
Change-Id: I9a5940027423ff0791fa7da0b79b617412ce6b86
|
|
A ridiculously fast way of doing this is:
for i in $(pcregrep -l -M -r --include='.*[hc]xx$' \
--exclude-dir=workdir --exclude-dir=instdir '^
{3,}' .)
do
perl -0777 -i -pe 's/^
{3,}/
/gm' $i
done
Reviewed-on: https://gerrit.libreoffice.org/22224
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Chris Sherlock <chris.sherlock79@gmail.com>
(cherry picked from commit a238b1f8d304bf1e2ffb357937f3ec888ee8ac89)
Change-Id: Iebb93eccbee9e4fc5c4380474ba595858a27ac2c
|
|
Change-Id: I6f2d1c2c947e01a686fdb7a7f175dd7541924afa
Reviewed-on: https://gerrit.libreoffice.org/25805
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Eike Rathke <erack@redhat.com>
Tested-by: Eike Rathke <erack@redhat.com>
(cherry picked from commit eee5256e6649c5ccbd26f30b53351c808f6c56e4)
Reviewed-on: https://gerrit.libreoffice.org/25814
|
|
(cherry picked from commit c5c2181a14c6ca9ed47729e6d8dbda2cc3552221)
Change-Id: I53968fb15b6e0d6e1ab5cedb7a9b6c4ffae8e534
Reviewed-on: https://gerrit.libreoffice.org/25777
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Tested-by: Caolán McNamara <caolanm@redhat.com>
|
|
Caveat, the space needs to be quoted, otherwise it is taken as the group
separator that when trailing a number divides the number by 1000.
Change-Id: Ic24b90d500a6c9351ef7463b2b1adf9d066442df
(cherry picked from commit 5868d927bccfc9594c4f791d0800337a0ce60c18)
Reviewed-on: https://gerrit.libreoffice.org/25738
Reviewed-by: Christian Lohmaier <lohmaier+LibreOffice@googlemail.com>
Tested-by: Christian Lohmaier <lohmaier+LibreOffice@googlemail.com>
|
|
Change-Id: Ie872c593c531b99360efe7fcf807e76cb88acb79
(cherry picked from commit 91244b95b7826ec5c1075c00e6b4013804c4ab2f)
Reviewed-on: https://gerrit.libreoffice.org/25589
Reviewed-by: Adolfo Jayme Barrientos <fitojb@ubuntu.com>
Tested-by: Adolfo Jayme Barrientos <fitojb@ubuntu.com>
|
|
Change-Id: If77055d6b292d5126165252d7198e8abf9d2c44d
Reviewed-on: https://gerrit.libreoffice.org/24029
Reviewed-by: Eike Rathke <erack@redhat.com>
Tested-by: Eike Rathke <erack@redhat.com>
(cherry picked from commit 087162b7142cf666478986461cdfa15c3d7fe382)
Reviewed-on: https://gerrit.libreoffice.org/24033
|
|
(cherry picked from commit 72c5e230f9cda8e18f63f7bbc6567487b4c5a5e0)
i18npool: fix build
Looks like 72c5e230f9cda8e18f63f7bbc6567487b4c5a5e0 missed the
makefile change.
(cherry picked from commit a6c8b41c4df79183c5d2c1450c010768ac7d82f6)
2d887ca6cb6c698fdb2d24029f38cce6050e0881
Change-Id: I89641fa82f0d0129aeff0d86a10be26cdc721dfc
Reviewed-on: https://gerrit.libreoffice.org/21725
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: David Tardon <dtardon@redhat.com>
|
|
The UTF-16 code unit limitation was mangling starmath import.
Change-Id: I087e5c5b7954799fdb73e7ee1a8d3d02669f8831
(cherry picked from commit fefd1221be844a033e409a18e05e8c6e98f6d1a7)
Reviewed-on: https://gerrit.libreoffice.org/21784
Reviewed-by: Eike Rathke <erack@redhat.com>
Tested-by: Eike Rathke <erack@redhat.com>
|
|
_length must at the tail of filename.
Change-Id: I434e89bf052d8f3130bfca62d71eaf8588c2bac9
Reviewed-on: https://gerrit.libreoffice.org/21213
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Eike Rathke <erack@redhat.com>
Tested-by: Eike Rathke <erack@redhat.com>
(cherry picked from commit 1b683de366f4caf82225c632f02cf0f58ae76490)
Reviewed-on: https://gerrit.libreoffice.org/21367
|
|
Change-Id: I77f49f09ce9fe5897f78044c0522210978c904d1
(cherry picked from commit d01888302b03a79bf7ccde6cbe8434b427164a75)
Reviewed-on: https://gerrit.libreoffice.org/21130
Reviewed-by: Michael Stahl <mstahl@redhat.com>
Tested-by: Michael Stahl <mstahl@redhat.com>
|
|
...instead of a local function using a static (!) buffer, which just cannot work
in a multi-threaded environment.
Change-Id: Id28e1c55c32aa2e35cd3c1756f5d392241092a30
(cherry picked from commit c83b55564dab042e3fc1889920cbb4db8fe40117)
Reviewed-on: https://gerrit.libreoffice.org/21131
Reviewed-by: Michael Stahl <mstahl@redhat.com>
Tested-by: Michael Stahl <mstahl@redhat.com>
|
|
Change-Id: I054786a97b6f0e6ba3fe3b824ff9ff9748d8be4a
Reviewed-on: https://gerrit.libreoffice.org/20013
Reviewed-by: Eike Rathke <erack@redhat.com>
Tested-by: Eike Rathke <erack@redhat.com>
(cherry picked from commit 33a774bb1ff7b1af48b3a534b53dd3843a279eac)
Signed-off-by: Eike Rathke <erack@redhat.com>
|
|
"(abc)|(def)" matches "def" with result offset sequences
{0,-1,0},{3,-1,3}
And thus the assert() calls introduced with
4cf1d290bab29e18e1312b63ff862f5102e00387 and
ce91f3c1292f3e9b84157acf10b67ad9ca16719d were hit.
Change-Id: I571b6c7d47349a2cc7b1d1e34193b2865012a49f
(cherry picked from commit 4dd2d40673299966ad639d799e925e64ae5560cf)
|
|
Change-Id: Ibb34f4a655cce13a24cc7ec22a29fa40e3a0286a
|
|
Change-Id: I1934441858baeeb41a46f694dbcef2d846b308b7
|
|
Change-Id: Icb5a45acaad9029e0998594de8b3dc2e9211866e
|
|
nStart should not become <0, not even when searching for $ end.
Change-Id: I79586471f1a7908372fa3ee044a6f66499b1790f
|
|
With simple transliteration, TextSearch::searchForward used to use
whole string to perform the search, then started to create substring
to search. But it left the precautions from
commit c00601dab0f5533b152cd63cec0a89bfec1ba95f by Eike Rathke:
searching for $ may actually return a result set pointing behind the
search string which it does with the ICU regex engine.
The precaution made it to skip reverse mapping if index was 0.
Commit 9aae521b451269007f03527c83645b8b935eb419 by Michael Stahl
didn't consider the case when nStop is 0, and startPos > 0. Then it
tried to get offset[-1].
Anyway, using value of startPos in those conditions seems illogical.
Removed those precautions (and made assertions for that).
Fixed handling zero indexes.
Change-Id: I2066abc51fff7fb7323bc7f6198bdea06439d4f3
Reviewed-on: https://gerrit.libreoffice.org/19840
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Eike Rathke <erack@redhat.com>
Tested-by: Eike Rathke <erack@redhat.com>
|
|
Change-Id: I09dad8485f7a558dd389032a6052b88a8285afea
|
|
Change-Id: I721f53d79cd3e2c24542382d519a909a86a31111
|
|
update the plugin with lessons learned from the mergeclasses plugin and
re-run it
Change-Id: I9d622eb3d05fceaf8fa764c533c8fa5dfb4c7711
Reviewed-on: https://gerrit.libreoffice.org/20015
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
|
|
Change-Id: I727951614587654c11af5f6635a45c52ebec1b5a
|
|
Change-Id: I08c98ed0f81894444a75adcf7da3a952c7f418cf
|
|
Change-Id: Ia5e47261d1fc6fac2d046656c05a1c5eedb07e02
Reviewed-on: https://gerrit.libreoffice.org/19978
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
|
|
replaced using the script:
git grep -lP 'Sequence.*OUString.*\(1\)'
| xargs perl -0777 -pi
-e "s/Sequence< OUString > (\w+)\(1\);
.*\[0\] = (\S+);/Sequence< OUString > \1 { \2 };/g"
Change-Id: I23688a91562051a8eed11fc2a85599545c285c34
Reviewed-on: https://gerrit.libreoffice.org/19967
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
|
|
Change-Id: Ic8b4c6a1718a10d0b9a5a99b069ef4e98de87518
|
|
Change-Id: I1bcb4a70a2b2621a8718e5d4d0741e8eed3848cf
|
|
Change-Id: I22047ed53eb59716e3da7c6e799867a666d35921
|
|
Change-Id: If1c2f5e8527855e8e537d082156bcd6582e50c58
|
|
Change-Id: I4a257a4eed1064a3fc09f6e2d37dbbc6be64a94c
|
|
Change-Id: I255273601fd460b22aa22e3e366dc040e0795a87
|
|
Change-Id: I81c71c02888e9c5cab28e77c1464dd5f6d4c1ee5
|
|
Change-Id: Ib2738355c0ffc1342db51a4f17185503ad7f0eb7
|
|
Change-Id: I7af8d14bfe7337fea3f18dbbd41eb32e0f7ae331
|
|
Change-Id: Idbe31956d8627ff51f90d2e4a52666d33143b8c0
|
|
... same as in other ar_* locale data.
Change-Id: If5bd17d710fc6a9b40aac87eb49a8615f59a7a31
|
|
Change-Id: Iefb671f4e7130523edde1d0495d36fb57aa7528c
|
|
Change-Id: Id4be326683f4be745284d0951768047d830570c1
|
|
Change-Id: Ie7573d79f74d601abd340d028080dcb50a78917a
|
|
Change-Id: I55eea19b0302053691eb76b7785276a89bcd136a
|
|
Change-Id: Ice269b722e219ae7b51aa6ea11dea93cfd31f413
|
|
Change-Id: I2c65eac8aa0b24be7f0c8a083b21904c842ca1ee
|
|
Change-Id: I11822c50fa66d038a3d6f38054ab35c2e613f077
|
|
Change-Id: I178545792c7354a362658ac7ef8b1d4cf0865797
Signed-off-by: Michael Stahl <mstahl@redhat.com>
|
|
* always wrap the sub-command in ( ... )
* if no 2nd arg is given, use a temp file
Change-Id: I7b43214dda1e4f8ee5e9b9a7310d58adee2a6500
|
|
Change-Id: I37da28539b94706574116d0fff5f008aabfb5526
Reviewed-on: https://gerrit.libreoffice.org/19682
Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
Tested-by: Noel Grandin <noelgrandin@gmail.com>
|
|
Change-Id: If8503566d02b2e2de51acc21f33042b9cc23b059
|