Age | Commit message (Collapse) | Author |
|
Change-Id: I6d5a952901648e01904ef5c37f953c517304d31e
|
|
...and replace gb_CppunitTest_unset_headless with
gb_CppunitTest_use_vcl_non_headless.
Change-Id: Ibd859f98ed8df09988ca26ffcee26e06e25d8313
|
|
Change-Id: I9d4255f0ddcaf375706556d0c0baf2a5a0762bbf
|
|
connectivity: 3m52s -> 2m47s
cppcanvas: 28s -> 13s
cppuhelper: 20s -> 14s
dbaccess: 2m38s -> 2m01s
hwpfilter: 16s -> 13s
sot: 21s -> 16s
Change-Id: I49286bfe6be73dd1b861be632b95e17a99e82f8a
|
|
This required some changes to the framework:
* Init-/DeInitVCL is no longer done per individual test in BootstrapFixture, but
once per CppunitTest invocation in a new vclbootstrapprotector (similarly to
the exisiting unobootstrapprotector). CppunitTests that need VCL now need to
declare gb_CppunitTest_use_vcl.
* For things to work properly, the UNO component context needs to be disposed
from within DeInitVCL (cf. Desktop's Application::DeInit called from
DeInitVCL). The easiest solution was to introduce an
Application::setDeInitHook (where the hook is called from DeInitVCL)
specifically for vclbootstrapprotector to call.
* PythonTests don't (yet) call DeInitVCL; they still hook into
BootstrapFixture's original test_init functionality (to call InitVCL), and do
not make use of the vclbootstrapprotector.
Change-Id: I4f3a3c75db30b58c1cd49d81c51db14902ed68b2
|
|
Change-Id: I318878b3fa883eebbc081a6faf6a594dad2c6e26
|
|
Change-Id: Ie453e1a95b54de07463b6f5052a23d6632400b8d
|
|
Change-Id: I5bd3f808a0ca61731c5ceb57100d1b30a3163e98
|
|
Change-Id: I3e51a62710bb46c8255fd228d41d9300c90a1fb5
Reviewed-on: https://gerrit.libreoffice.org/9360
Reviewed-by: Thomas Arnhold <thomas@arnhold.org>
Tested-by: Thomas Arnhold <thomas@arnhold.org>
|
|
Change-Id: I590a682af7fc1879ae63cda3aea73063ba4c84f4
|
|
Change-Id: I1a52b9ce7b590e8cf274a2daaa1c55cfc1efe48b
|
|
Change-Id: Id945cf9549b7f633f7deb1f237a2bf6536f221c1
|
|
Change-Id: Ib65f0e95ef2a837b6c5791d9e2dce5fe8025da33
|
|
Change-Id: Ia7cca6e2a2bc3240a32509697a999461491a2747
|
|
Change-Id: I8b112fa0095cc6ed15c9198a312a0b6fce3a8d28
|
|
Change-Id: I91f6e861f6d0dbea1b57ac73857e0c706b27f681
|
|
This test is already disabled on non-Linux, but it fails in case there
is no display to use. For now just disable it in that case.
Change-Id: I29c52e803a1fca5f2bdeeb655c573ad8fef622e8
|
|
We didn't have EMF+ rendering testcases so far, let's see if it works
out to render into a bitmap and then just assert pixel position colors
there. It's better than nothing for missing shapes at least.
Change-Id: I2d1c63fef1127f69af7156ed6c99553845f77c9f
|
|
This is still not perfect, but at least we now don't do the opposite of
what was asked.
Change-Id: I5e144c5ec2987902e65b2eb472259d9c39bbbd11
|
|
Change-Id: I54f8a16cd64375d1398b6582175b0941ba68696e
|
|
Change-Id: I3dc076f3593d4d083d5e25e41e007eadc97d2978
|
|
Change-Id: I0ef2e67f6d61e0ce118c0f5e926b8194ef9d8058
|
|
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
|
|
...mostly done with a rewriting Clang plugin, with just some manual tweaking
necessary to fix poor macro usage.
Change-Id: I71fa20213e86be10de332ece0aa273239df7b61a
|
|
It's not very efficient, because we generally end up copying it twice -
once into the parameter and again into the destination OUString.
So I create a clang plugin that finds such places and generates a
warning so that we can convert them to pass-by-reference.
Change-Id: I5341a6ea9e3190f4b4c05c42c85595e3dcd83361
|
|
Change-Id: I56e32131b7991ee9948ce46765632eb823d463b3
|
|
Change-Id: I961e6a9e227962096b57569a8b8160841cacd950
|
|
rSt >> pPtr[3] >> pPtr[2] >> pPtr[1] >> pPtr[0] results in error:
no match for 'operator>>' in 'rSt >> *(pPtr + 3u)'
Change-Id: I1820bac8c72f7633ab01a17bf5d073b988e1b58c
Reviewed-on: https://gerrit.libreoffice.org/8207
Tested-by: Michael Stahl <mstahl@redhat.com>
Reviewed-by: Michael Stahl <mstahl@redhat.com>
|
|
Change-Id: I2e472aa0279d0763762d3c660207cd74da512626
|
|
Change-Id: I939160ae72fecbe3d4a60ce755730bd4c38497fb
Reviewed-on: https://gerrit.libreoffice.org/8182
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Tested-by: Caolán McNamara <caolanm@redhat.com>
|
|
Change-Id: I2a3e3d3e3266ea0f0fafdd91362076a4aa160f0e
|
|
Change-Id: Id7c6db5c6cbefef427d9c2cc4bd210811b5a7991
|
|
in favour of ReadXXX methods.
Change-Id: Ic2c0a7b6b92ff4c236ae99b39d77f3d935b301e3
Reviewed-on: https://gerrit.libreoffice.org/7915
Tested-by: Michael Stahl <mstahl@redhat.com>
Reviewed-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>
|
|
as preparation for converting the SvStream::operator>> methods on
primitive types
Change-Id: I62f134bced15c687d6e0d46924f56e8d1c3d95b9
Reviewed-on: https://gerrit.libreoffice.org/7798
Tested-by: LibreOffice gerrit bot <gerrit@libreoffice.org>
Reviewed-by: Michael Stahl <mstahl@redhat.com>
|
|
Change-Id: I4520eea08e43ccd657c1db03b258ef84612da971
Reviewed-on: https://gerrit.libreoffice.org/7726
Reviewed-by: Jan Holesovsky <kendy@collabora.com>
Tested-by: Jan Holesovsky <kendy@collabora.com>
|
|
Change-Id: Id4dfa3d4676b50a86fd6044d501de15b122bb7f1
|
|
Change-Id: Ibe8600f8ad38859cd79d52528f970d99f0b8f536
|
|
Change-Id: Ie2de4680e07e26416fc52d4c22b5bececb190cc1
|
|
we can use -1 as the failure return code as return value is always checked and
isolated from the rest of the code.
Change-Id: I87a743de82d681a7e4eabc61a576c4e50f1279a8
|
|
Change-Id: I5b369703333332598353d9817f0253bbe5fc3601
|
|
Otherwise (i.e. with non-vertical lines) the width will be incorrect,
and can result in lines disappearing as the resulting X component
can be tiny/approaching zero.
Change-Id: Icf3b7c10c627594600b517b8ff445f8df87c56f8
|
|
Change-Id: Iff1ae9806ee05654762b313ed00ada206e51337f
|
|
Change-Id: I5babcec50d48dc2a6288a01685db61fbb7058680
|
|
a new log section (sal.rtl.xub) is used to display alert in case of
suspicious len == 0xFFFF (aka STRING_LEN)
Change-Id: I3ed2aa7896e12592be9e003580dd6c8eda4add5e
Reviewed-on: https://gerrit.libreoffice.org/7117
Tested-by: LibreOffice gerrit bot <gerrit@libreoffice.org>
Reviewed-by: Norbert Thiebaud <nthiebaud@gmail.com>
|
|
Change-Id: I56f38bd786f3a026cb2908f28540dc9c4003af83
|
|
Change-Id: I7b53a8f18e1fb24b1ae0322bdf0980e431a0725f
|
|
Change-Id: I903d8e5629c578f292307bf3abdb2f58ef9dd92c
|
|
Change-Id: I4f861c7653c89d8f1ca73696e885f33f7bfc3b06
|
|
It seems I missed a few places in
21ec9beae29b19b8ec6f0a16fd0e708e4f210208, getting that straight
now hopefully.
Change-Id: I34fc8df2d1ee65bef0639b3de9487ff311bed89a
|