Age | Commit message (Collapse) | Author |
|
Instead of having a lot of it hardcoded, which brings problems like:
- Clang-to-be-10 has switched to -std=c++14, so our hardcoded c++11
makes the build fail
- I cannot compile with my openSUSE-shipped clang, because it ships
only libclang-cpp and not the other libClangSomething libs
The possibility to explicitly set the necessary variables is still there.
Change-Id: I58d401d4584fa064f1c1351a8a06ff4e29643063
Reviewed-on: https://gerrit.libreoffice.org/80300
Tested-by: Jenkins
Reviewed-by: Luboš Luňák <l.lunak@collabora.com>
|
|
These are triggered when using llvm-config --cxxflags for building,
and sometimes there's -Werror. The warnings were mostly unused
variables because of being used only in assert(), or default case
in switch that covers all enums (it's better to not handle default
to get warning if a case is not handled).
Change-Id: I0ecdd1f27390aadf033852b0d1ee0ca424ae3c37
Reviewed-on: https://gerrit.libreoffice.org/80317
Tested-by: Jenkins
Reviewed-by: Luboš Luňák <l.lunak@collabora.com>
|
|
Change-Id: Id05f2741763ef0d9e87d9804339d29aee6bc068f
Reviewed-on: https://gerrit.libreoffice.org/80380
Tested-by: Jenkins
Reviewed-by: Luboš Luňák <l.lunak@collabora.com>
|
|
Change-Id: I97ff0418e25aeaea4cae349f2d228fb35219b5c2
Reviewed-on: https://gerrit.libreoffice.org/80374
Tested-by: Jenkins
Reviewed-by: Luboš Luňák <l.lunak@collabora.com>
|
|
Change-Id: I0389311323175fbbc58ee0b9fbb6353061b21e2f
Reviewed-on: https://gerrit.libreoffice.org/80395
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Tested-by: Caolán McNamara <caolanm@redhat.com>
|
|
Change-Id: Iee2254dc13ccc31d266009ba1eb00f2517484cc7
Reviewed-on: https://gerrit.libreoffice.org/79814
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
|
|
Change-Id: Ia83cc372d53db41edc8199077db91cb7b1d06e24
Reviewed-on: https://gerrit.libreoffice.org/80386
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
|
|
Change-Id: Iab0ff79ba1f94f087b57faefa3945aba1ed42448
Reviewed-on: https://gerrit.libreoffice.org/80387
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
|
|
(See the discussion in the comments to
<https://gerrit.libreoffice.org/#/c/80317/> "fix various warnings in
compilerplugins".)
Change-Id: Ia6ba32206ddf44bba9bb5acdd929c0750c48cca9
Reviewed-on: https://gerrit.libreoffice.org/80372
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
|
|
Change-Id: Id8b7e4dfb021b836d5d14ce9ca73f3835288a7a4
Reviewed-on: https://gerrit.libreoffice.org/80343
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Tested-by: Caolán McNamara <caolanm@redhat.com>
|
|
Change-Id: I94e42c1d10922170f53af917bba6848c9f006a11
Reviewed-on: https://gerrit.libreoffice.org/80383
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Tested-by: Caolán McNamara <caolanm@redhat.com>
|
|
Change-Id: I81084e169722f6abcc660e360d6e39aff7446240
Reviewed-on: https://gerrit.libreoffice.org/80370
Tested-by: Jenkins
Reviewed-by: Julien Nabet <serval2412@yahoo.fr>
|
|
Change-Id: Ica2fb688d3fcf6f9f74399215e3092bf4a2ac9c8
Reviewed-on: https://gerrit.libreoffice.org/80337
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Tested-by: Caolán McNamara <caolanm@redhat.com>
|
|
Change-Id: I7762d95290a0b35de4f909895a4e28b52c436ed9
Reviewed-on: https://gerrit.libreoffice.org/80307
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Tested-by: Caolán McNamara <caolanm@redhat.com>
|
|
Change-Id: I887b0371309a897848eed5d72c55458f78ed1f68
Reviewed-on: https://gerrit.libreoffice.org/80361
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Tested-by: Caolán McNamara <caolanm@redhat.com>
|
|
GetPrefSize() at this point is Size(0,0), so the test was testing
topleft corner. And getPixel() takes Y,X , so even then it wasn't
testing the center.
Change-Id: Ie2e489a693fae2b7ea41075ebe6fa3ef405545d5
Reviewed-on: https://gerrit.libreoffice.org/80351
Tested-by: Jenkins
Reviewed-by: Luboš Luňák <l.lunak@collabora.com>
|
|
Change-Id: Ic331d33e0f0feca1fde4425fdb4106b1a769194d
Reviewed-on: https://gerrit.libreoffice.org/80350
Tested-by: Jenkins
Reviewed-by: Luboš Luňák <l.lunak@collabora.com>
|
|
Change-Id: I7dc06a2754eaa172f112f38a260240f8e2cc0983
Reviewed-on: https://gerrit.libreoffice.org/80345
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Tested-by: Caolán McNamara <caolanm@redhat.com>
|
|
Change-Id: Ibb21567c11adef233061cb53f787a73eb56cc282
Reviewed-on: https://gerrit.libreoffice.org/74848
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Tested-by: Caolán McNamara <caolanm@redhat.com>
|
|
Change-Id: I6febe3d48fc9018b373a940d88d2afeefad7502c
Reviewed-on: https://gerrit.libreoffice.org/80355
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Tested-by: Caolán McNamara <caolanm@redhat.com>
|
|
Change-Id: I3b283a3283d865f5f96d872bad219ce9c3f872df
Reviewed-on: https://gerrit.libreoffice.org/80356
Reviewed-by: László Németh <nemeth@numbertext.org>
Tested-by: Jenkins
|
|
I didn't know that flushClipboard is called for simple text edit
fields for C'n'P operations, and not just on LO shutdown. This way
the simple text is actually secured in the clipboard instandly, as
there aren't complex mime-types to generate and secure.
As a result we also need to protect flushClipboard from loosing
ownership, which wasn't needed for the shutdown-only case, as this
would give up ownership anyway.
Change-Id: Ib3cd4979228fc645a27c658abb3df38ccf8c8956
Reviewed-on: https://gerrit.libreoffice.org/80042
Reviewed-by: Ilmari Lauhakangas <ilmari.lauhakangas@libreoffice.org>
Tested-by: Jenkins
Reviewed-by: Michael Weghorn <m.weghorn@posteo.de>
Reviewed-by: Jan-Marek Glogowski <glogow@fbihome.de>
|
|
Change-Id: I8158bf997de6e34da1f2ab6a2f9b5e3c61267520
Reviewed-on: https://gerrit.libreoffice.org/80354
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Tested-by: Caolán McNamara <caolanm@redhat.com>
|
|
Change-Id: Idc94a2a912d94a8208348a8d6151c6d7dfe3ccb9
Reviewed-on: https://gerrit.libreoffice.org/80352
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Tested-by: Caolán McNamara <caolanm@redhat.com>
|
|
This is a continuation of ff002524c12471668e63837a804b6006f9136a34.
When compiling with icecream, its -frewrite-includes merges
all #include's into one .cxx file and marks them with with line
markers. But SourceManager::getFilename() doesn't take those
into account and so it reports all of those as <stdin>.
So use getFileNameOfSpellingLoc(), which explicitly handles this
case.
And we should probably never ever use SourceManager::getFilename().
Change-Id: Ia194c2e041578e1e199aee2df2f885922ef7e31a
Reviewed-on: https://gerrit.libreoffice.org/80326
Tested-by: Jenkins
Reviewed-by: Luboš Luňák <l.lunak@collabora.com>
|
|
Change-Id: I20baebc41bd833764bbe828dd63df5218a12ce20
Reviewed-on: https://gerrit.libreoffice.org/80349
Tested-by: Jenkins
Reviewed-by: Luboš Luňák <l.lunak@collabora.com>
|
|
Change-Id: Ic499677dad477d1ed194d63929f9374995fccb65
Reviewed-on: https://gerrit.libreoffice.org/79254
Reviewed-by: Tor Lillqvist <tml@collabora.com>
Tested-by: Tor Lillqvist <tml@collabora.com>
(cherry picked from commit 5a742afd3dde74138bda8fd140d3801b3ca900e6)
|
|
Change-Id: Ie83b4ad7a7ab1c26ff081f70ad98eb39a04f1015
(cherry picked from commit 7dd803da92d9b2ff36ded57ea22fbb41ef1db28f)
Reviewed-on: https://gerrit.libreoffice.org/79253
Reviewed-by: Tor Lillqvist <tml@collabora.com>
Tested-by: Tor Lillqvist <tml@collabora.com>
(cherry picked from commit dedf3da4d33a51f63006b125991e1d64d70df381)
|
|
This "comment" was actually interpreted as a shell command. Sure, the
"command" was a comment in shell, too, but still, quite pointless to
run it.
Change-Id: If5abeabb1c10d62fa90dcdaf81b3b54af958ba62
Reviewed-on: https://gerrit.libreoffice.org/79292
Reviewed-by: Tor Lillqvist <tml@collabora.com>
Tested-by: Tor Lillqvist <tml@collabora.com>
(cherry picked from commit d7cbe885daf2e05677b2891c920e5ff125ee15a9)
|
|
Apparently the use of [UIFont systemFontOfSize:10] familyName] to get
a default font family name is a bad idea. Presumably the return value
from this, ".SF UI Text", is matched against the list of font family
names enumerated from the system. (The "SF" apparently stands for "San
Francisco".) That ".SF UI Text" is not among them, so maybe vcl
chooses some arbitrary other font instead that happens to be a serif
one? If we instead use "Helvetica", at least we get a sans-serif font,
even if it doesn't match the system UI font exactly.
Change-Id: I7ff39d8e7893ce3c27f3f12d227f87209bbc7952
(cherry picked from commit 685e91a7aee4a4acc60e33bf1313a394fd15b1ff)
Reviewed-on: https://gerrit.libreoffice.org/79196
Reviewed-by: Tor Lillqvist <tml@collabora.com>
Tested-by: Tor Lillqvist <tml@collabora.com>
(cherry picked from commit ba9f91a909cb52194178ac2ed78dc62bd61c1be3)
|
|
Change-Id: I57b50e72b68b71406d1745304d24b52d75b3d082
Reviewed-on: https://gerrit.libreoffice.org/80342
Tested-by: Jenkins
Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>
|
|
Change-Id: Id36265c5bd50d95105e3abf37e999aba80b4712c
|
|
Change-Id: Ie7c2f75ec66ae4c1f8dc49075624617c294cbc25
Reviewed-on: https://gerrit.libreoffice.org/62945
Reviewed-by: Tor Lillqvist <tml@collabora.com>
Tested-by: Tor Lillqvist <tml@collabora.com>
Reviewed-on: https://gerrit.libreoffice.org/79131
|
|
Nah. We already have
com_sun_star_comp_Writer_DocxExport_get_implementation as it is called
in this branch.
This reverts commit 98deb6e96f547e8eedff40c992fefe3514021447.
Change-Id: Ic5db66420db9ef2477df8d9905ba3525f3b57e5b
Reviewed-on: https://gerrit.libreoffice.org/79010
Reviewed-by: Tor Lillqvist <tml@collabora.com>
Tested-by: Tor Lillqvist <tml@collabora.com>
(cherry picked from commit cb61889e8761837150236c617e806d52c54fd72f)
|
|
Change-Id: Idd7d675872b73454d78576ed231fe90644dbe4c5
Reviewed-on: https://gerrit.libreoffice.org/78142
Reviewed-by: Tor Lillqvist <tml@collabora.com>
Tested-by: Tor Lillqvist <tml@collabora.com>
Reviewed-on: https://gerrit.libreoffice.org/78999
|
|
They are needed for .docx and .rtf export at least. No idea how we
have managed without it previously.
Change-Id: Iff9224f92520e19d9baa691a9e7cd3c944004eed
Reviewed-on: https://gerrit.libreoffice.org/78125
Reviewed-by: Tor Lillqvist <tml@collabora.com>
Tested-by: Tor Lillqvist <tml@collabora.com>
(cherry picked from commit 1a0bd0c6914bf93733c7ab3a27741fc1dc0e3f26)
Reviewed-on: https://gerrit.libreoffice.org/78996
(cherry picked from commit 98deb6e96f547e8eedff40c992fefe3514021447)
|
|
Change-Id: I92110a7a501571d7fd707dc33502ff553f02ae5e
Reviewed-on: https://gerrit.libreoffice.org/77823
Reviewed-by: Tor Lillqvist <tml@collabora.com>
Tested-by: Tor Lillqvist <tml@collabora.com>
(cherry picked from commit e84c42ee32d1a23729c65b534c4418e2043f706d)
Reviewed-on: https://gerrit.libreoffice.org/78994
(cherry picked from commit 1e7a3f82324c3b855c2b3c1f8d4dec73c5162806)
|
|
(I saw warnings about it missing in the iOS app.)
Change-Id: Ib286b107e529e80ec36ffb87059f4f5923867abe
Reviewed-on: https://gerrit.libreoffice.org/77399
Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoffice@gmail.com>
Reviewed-by: Tor Lillqvist <tml@collabora.com>
Tested-by: Tor Lillqvist <tml@collabora.com>
|
|
Change-Id: Id13ddee66a70d8cd293940f7eea8597257d169c5
Reviewed-on: https://gerrit.libreoffice.org/77398
Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoffice@gmail.com>
Reviewed-by: Tor Lillqvist <tml@collabora.com>
(cherry picked from commit aa0b785946ba39bbce419a6b356a7ce07349501c)
|
|
Change-Id: I3f098cf0b364abb1423b5cecb8cca763f053fa4b
Reviewed-on: https://gerrit.libreoffice.org/80336
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Tested-by: Caolán McNamara <caolanm@redhat.com>
|
|
...that was there ever since 7b0b5cdfeed656b279bc32cd929630d5fc25878b "initial
import", while 2e93541d00f9d4aaa6ad2aeb35ee0c901d216cc0 "Resolves: fdo#69220
crash in doc (ww6)" introduced a WW8PLCFxDesc default ctor that initializes all
members (even though it initializes some of them to non-zero, but lets assume
that that doesn't matter here).
Change-Id: I21a75675d59ded81ccb17397f8c464efdfc3d78f
Reviewed-on: https://gerrit.libreoffice.org/80316
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
|
|
Change-Id: Ie2850a38db3d2e343d9fc19e245eb97238196110
Reviewed-on: https://gerrit.libreoffice.org/80339
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Tested-by: Caolán McNamara <caolanm@redhat.com>
|
|
Change-Id: Ideb980a07632e75e2bc7fffa851b8419e727c13b
Reviewed-on: https://gerrit.libreoffice.org/80340
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Tested-by: Caolán McNamara <caolanm@redhat.com>
|
|
On Android, SwAnnotationWin::Rescale leads to invoke
ImpEditEngine::UpdateViews which hides the text cursor. Moreover it
causes sudden document scroll when modifying a commented text.
Not clear the root cause, anyway skipping this method fixes the
problem, and there should be no side effect, since the client has
disabled annotations rendering.
Change-Id: I572a9c6b3fe39473a596209413945d777bd79506
Reviewed-on: https://gerrit.libreoffice.org/80244
Reviewed-by: Michael Meeks <michael.meeks@collabora.com>
Tested-by: Michael Meeks <michael.meeks@collabora.com>
|
|
Newly created or ODF charts were still exported incorrectly, because
ShiftedCategoryPosition was stored in the view code. Now it is removed
to the model using ChartTypeTemplate::adaptScales.
See also commit 6027ec08fd5df2e09e34ff61b3777ad2cc8304b3 'tdf#127777
OOXML chart export: fix X axis position setting "CrossBetween"'
Change-Id: I7cd69e311833bf9f24f3a600ee7c6312285a7738
Reviewed-on: https://gerrit.libreoffice.org/80160
Reviewed-by: László Németh <nemeth@numbertext.org>
Tested-by: László Németh <nemeth@numbertext.org>
|
|
Change-Id: I577cfdfde9076a3c6273c9af1f0b35108a0a0160
Reviewed-on: https://gerrit.libreoffice.org/80338
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Tested-by: Caolán McNamara <caolanm@redhat.com>
|
|
/home/vmiklos/git/libreoffice/master-android/sw/source/core/txtnode/txtedt.cxx:1203:41: error: loop variable 'rImplicitChange' has type 'const std::pair<int, int> &' but is initialized with type 'std::__ndk1::pair<long, long>' resulting in a copy [-Werror,-Wrange-loop-analysis]
Just use auto, the copy that was happening here was not intentional.
Change-Id: I2f98519f7891bd52ccf560ba0707ad8eef415e06
Reviewed-on: https://gerrit.libreoffice.org/80364
Reviewed-by: Miklos Vajna <vmiklos@collabora.com>
Tested-by: Noel Grandin <noel.grandin@collabora.co.uk>
|
|
Change-Id: I111d96da3615f5084f10b4e81dd10bc559c29757
Reviewed-on: https://gerrit.libreoffice.org/80335
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Tested-by: Caolán McNamara <caolanm@redhat.com>
|
|
Change-Id: Ice72d00eb402a291f9a9e262a582493e316864fc
Reviewed-on: https://gerrit.libreoffice.org/80358
Reviewed-by: Julien Nabet <serval2412@yahoo.fr>
Tested-by: Julien Nabet <serval2412@yahoo.fr>
|
|
UNX is defined for Android. Other code in the same file has guards like:
#if !defined(ANDROID) && !defined(IOS)
but this is already in an
#ifndef IOS
block, so just check for ANDROID.
Change-Id: I73f92ef5702ec1f1385e2ffa0ea3dbedf7d3b6a2
|